LGTM

http://codereview.chromium.org/6814012/diff/1/src/compiler.cc
File src/compiler.cc (right):

http://codereview.chromium.org/6814012/diff/1/src/compiler.cc#newcode606
src/compiler.cc:606:
The preparser should be able to detect this and store it in the
preparser data with the other data that are set for lazy functions.
On the other hand, if this is sufficient, I guess it'll do.

http://codereview.chromium.org/6814012/diff/1/test/mjsunit/strict-mode-opt.js
File test/mjsunit/strict-mode-opt.js (right):

http://codereview.chromium.org/6814012/diff/1/test/mjsunit/strict-mode-opt.js#newcode29
test/mjsunit/strict-mode-opt.js:29: var MAX = 10000000;
That's a lot. Is it really necessary?
Are there any flags you could use to make optimization more frequent so
you don't need as big a loop?

http://codereview.chromium.org/6814012/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to