Move AST node counting to post-pass

Move AST node counting to post-pass

R=mstarzinger@chromium.org, svenpanne@chromium.org
BUG=

Committed: https://code.google.com/p/v8/source/detail?r=24937

[wingo, 2014-10-28 08:26:46]

Patchset #1 (id:1) has been deleted

[wingo, 2014-10-28 08:31:02]

wingo@igalia.com changed required reviewers:
+ mstarzinger@chromium.org

[wingo, 2014-10-28 08:31:02]

PTAL; this is the patch from yesterday with a couple of modifications mentioned
in the comments below.

https://codereview.chromium.org/683023002/diff/20001/src/compiler.cc
File src/compiler.cc (right):

https://codereview.chromium.org/683023002/diff/20001/src/compiler.cc#newcode650
src/compiler.cc:650: if (!Compiler::Analyze(info) ||
!FullCodeGenerator::MakeCode(info)) {
This part changed since the last patch to signal StackOverflow() if analysis
fails and there is no pending exception

https://codereview.chromium.org/683023002/diff/20001/src/compiler/js-inlining.cc
File src/compiler/js-inlining.cc (right):

https://codereview.chromium.org/683023002/diff/20001/src/compiler/js-inlining...
src/compiler/js-inlining.cc:377: // TODO(wingo): ParseAndAnalyze can fail due to
stack overflow.
Here I added this TODO.  Should we just return if parse/analyze fails?  I would
think so.

[Michael Starzinger, 2014-10-28 11:20:21]

LGTM (mostly rubber-stamped, only reviewed the marked changes).

[wingo, 2014-10-28 13:24:11]

Committed patchset #1 (id:20001) manually as 24937 (presubmit successful).