LGTM
Maybe you can add a test that triggers lazy parsing too? See
https://codereview.chromium.org/923683002/diff/20001/test/mjsunit/harmony/classes-lazy-parsing.js
https://codereview.chromium.org/816913003/diff/710001/src/ia32/code-stubs-ia32.cc
File src/ia32/code-stubs-ia32.cc (right):
https://codereview.chromium.org/816913003/diff/710001/src/ia32/code-stubs-ia32.cc#newcode1158
src/ia32/code-stubs-ia32.cc:1158: // Check if the calling frame is an
arguments adaptor frame.
Out of scope for this CL but I think this code is repeated in a few
places now. We should refactor it.
https://codereview.chromium.org/816913003/diff/710001/src/x64/code-stubs-x64.cc
File src/x64/code-stubs-x64.cc (right):
https://codereview.chromium.org/816913003/diff/710001/src/x64/code-stubs-x64.cc#newcode849
src/x64/code-stubs-x64.cc:849: //
remove empty line
https://codereview.chromium.org/816913003/
--
--
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.
