Reviewers: Michael Starzinger,
Message:
mstarzinger, ptal.
Description:
compiler.cc: Remove a comment which is no longer true.
We don't do any automatic PreParsing in Compiler::Compile any more (it was
removed by r10066, > 2 years ago).
[email protected]
BUG=
Please review this at https://codereview.chromium.org/225993002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -8 lines):
M src/compiler.cc
Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index
a012d815ca585a86abd539a28fb7beb6241f1cf0..5967a446291fc3988b07b95b7ebed6a251c7ffc9
100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -956,14 +956,7 @@ Handle<SharedFunctionInfo> Compiler::CompileScript(
}
if (result.is_null()) {
- // No cache entry found. Do pre-parsing, if it makes sense, and compile
- // the script.
- // Building preparse data that is only used immediately after is only a
- // saving if we might skip building the AST for lazily compiled
functions.
- // I.e., preparse data isn't relevant when the lazy flag is off, and
- // for small sources, odds are that there aren't many functions
- // that would be compiled lazily anyway, so we skip the preparse step
- // in that case too.
+ // No cache entry found. Compile the script.
// Create a script object describing the script to be compiled.
Handle<Script> script = isolate->factory()->NewScript(source);
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
