Reviewers: marja,
Message:
Hi Marja,
Here's a proposed fix for https://code.google.com/p/v8/issues/detail?id=3195
In my testing this signifigantly reduces initial compile times for common
JavaScript libraries which are entirely composed of nested functions (like
jQuery), most signifigantly on mobile (android_arm).
I can share some performance data (timings of
ScriptCompiler::CompileUnbound()
in d8 for common JavaScript libraries) if you'd like.
Thanks,
Ian C
Description:
Support lazy parsing of inner functions
The approach here is for the PreParser to report usage of identifiers in
expression and calls to eval() with the ParserRecorder interface, which
handily also creates the CachedData used to skip this work the next time
we encounter the same script.
BUG=v8:3195
LOG=Y
Please review this at https://codereview.chromium.org/641283003/
Base URL: https://chromium.googlesource.com/external/v8.git@bleeding_edge
Affected files (+373, -66 lines):
M src/parser.h
M src/parser.cc
M src/preparse-data.h
M src/preparse-data.cc
M src/preparse-data-format.h
M src/preparser.h
M src/preparser.cc
M src/vector.h
M test/cctest/test-parsing.cc
--
--
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.
