Reviewers: Toon Verwaest,
Message:
I believe this patch addresses Toon's concerns from Rafael's original path,
save
the note about naming (which I haven't fixed only because I'm not sure what
to
do with SmartSlice/SimpleSlice).
Description:
Remove SmartMove, bringing Array methods further into spec compliance
This is one step towards a single codepath for each method in array.js.
This patch is based on rafaelw's https://codereview.chromium.org/349073002
BUG=v8:2615
LOG=Y
Please review this at https://codereview.chromium.org/455933002/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+22, -176 lines):
M src/array.js
M test/mjsunit/array-functions-prototype-misc.js
M test/mjsunit/array-natives-elements.js
M test/mjsunit/array-shift2.js
M test/mjsunit/array-unshift.js
M test/mjsunit/bugs/bug-2615.js
A + test/mjsunit/regress/regress-2615.js
M tools/generate-runtime-tests.py
--
--
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.
