Revision: 20526
Author: ma...@chromium.org
Date: Mon Apr 7 07:40:18 2014 UTC
Log: 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).
R=mstarzin...@chromium.org
BUG=
Review URL: https://codereview.chromium.org/225993002
http://code.google.com/p/v8/source/detail?r=20526
Modified:
/branches/bleeding_edge/src/compiler.cc
=======================================
--- /branches/bleeding_edge/src/compiler.cc Mon Mar 31 16:45:46 2014 UTC
+++ /branches/bleeding_edge/src/compiler.cc Mon Apr 7 07:40:18 2014 UTC
@@ -956,14 +956,7 @@
}
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
v8-dev@googlegroups.com
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 v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.