Reviewers: rossberg,

Message:
(rossberg, this is for you)

(ulan, mstarzinger, dcarney, jochen fyi)

Here's a new version of the "delay internalizing strings". I'd like another
round of comments...

cctest, mjsunit, webkit, mozilla and test262 pass. I'm planning to add more
tests for function name inferring.

Speed-wise it's a bit slower than baseline (5-7% on my non-lazy test data, both
on mobile and on desktop). That small regression shouldn't regress any
benchmarks.

Description:
Parser: Delay internalizing strings and values.

This is needed so that we can run Parser on a non-main thread (independent
of the Isolate and the V8 heap).

BUG=

Please review this at https://codereview.chromium.org/314603004/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+1426, -621 lines):
  M include/v8.h
  M src/ast.h
  M src/ast.cc
  A src/ast-string-table.h
  A src/ast-string-table.cc
  M src/compiler.h
  M src/compiler.cc
  M src/func-name-inferrer.h
  M src/func-name-inferrer.cc
  M src/heap.h
  M src/interface.h
  M src/interface.cc
  M src/list.h
  M src/objects.cc
  M src/parser.h
  M src/parser.cc
  M src/preparser.h
  M src/prettyprinter.h
  M src/prettyprinter.cc
  M src/rewriter.cc
  M src/scanner.h
  M src/scanner.cc
  M src/scopeinfo.cc
  M src/scopes.h
  M src/scopes.cc
  M src/utils.h
  M src/utils.cc
  M src/variables.h
  M src/variables.cc
  M test/cctest/test-ast.cc
  M test/cctest/test-parsing.cc
  M tools/gyp/v8.gyp
  M tools/parser-shell.cc


--
--
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.

Reply via email to