LGTM (with two nits).

https://chromiumcodereview.appspot.com/10260014/diff/1/test/mjsunit/regress/regress-125515.js
File test/mjsunit/regress/regress-125515.js (right):

https://chromiumcodereview.appspot.com/10260014/diff/1/test/mjsunit/regress/regress-125515.js#newcode28
test/mjsunit/regress/regress-125515.js:28: // Flags:
--allow-natives-syntax --expose-gc --debug-code
It seems as if --allow-natives-syntax is obsolete here.

https://chromiumcodereview.appspot.com/10260014/diff/1/test/mjsunit/regress/regress-125515.js#newcode37
test/mjsunit/regress/regress-125515.js:37: // Make sure that a ends up
old space
Missing "in".

https://chromiumcodereview.appspot.com/10260014/

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

Reply via email to