Reviewers: wingo, arv (Not doing code reviews), rossberg, Dmitry Lomov (no
reviews),
Message:
On 2015/08/31 20:00:14, aperez wrote:
mailto:[email protected] changed reviewers:
+ mailto:[email protected], mailto:[email protected],
mailto:[email protected],
mailto:[email protected]
PTAL. Also, if you see my latest comment in the bug, there does not seem to
be
any performance regression when applying this change :-)
Description:
ES6: Array.prototype.slice and friends should use ToLength instead of
ToUint32
Defines a new --harmony-tolength flag, and a ToLengthFlagged() runtime
function,
that is used where ES6 requires ToLength(), but a pre-ES6 conversion existed
before. When the flag is disabled, the function uses TO_UINT32(), which is
the pre-ES6 behaviour. When the flag enabled, the ES6-compliant ToLength()
conversion is used.
Based on a patch initially from Diego Pino <[email protected]>
BUG=v8:3087
LOG=Y
Please review this at https://codereview.chromium.org/1309243003/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+270, -33 lines):
M src/array.js
M src/array-iterator.js
M src/bootstrapper.cc
M src/debug/liveedit.js
M src/flag-definitions.h
M src/harmony-array.js
M src/heap/heap.h
M src/object-observe.js
M src/runtime.js
A test/mjsunit/harmony/array-length.js
--
--
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.
