LGTM with nits.
http://codereview.chromium.org/7886032/diff/1/src/execution.cc File src/execution.cc (right): http://codereview.chromium.org/7886032/diff/1/src/execution.cc#newcode165 src/execution.cc:165: // For somehwat unclear reasons, 'global' can be the JSBuiltinsObject nit: s/hwat/what/ Or just remove the "For...reasons" part completely. http://codereview.chromium.org/7886032/diff/1/test/mjsunit/regress/regress-bind-receiver.js File test/mjsunit/regress/regress-bind-receiver.js (right): http://codereview.chromium.org/7886032/diff/1/test/mjsunit/regress/regress-bind-receiver.js#newcode28 test/mjsunit/regress/regress-bind-receiver.js:28: // Check that modifying an array while sorting it does not segfault. See I think this comment does not apply to this test. http://codereview.chromium.org/7886032/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
