lgtm with nits.
https://chromiumcodereview.appspot.com/10905232/diff/2002/test/mjsunit/array-bounds-check-removal.js File test/mjsunit/array-bounds-check-removal.js (right): https://chromiumcodereview.appspot.com/10905232/diff/2002/test/mjsunit/array-bounds-check-removal.js#newcode34 test/mjsunit/array-bounds-check-removal.js:34: var v = [0, 1, 2, 3, 4, 5, 6, 7,8, 9]; nit: The indentation seems to be off. There's no space before the 8. https://chromiumcodereview.appspot.com/10905232/diff/2002/test/mjsunit/array-bounds-check-removal.js#newcode48 test/mjsunit/array-bounds-check-removal.js:48: return r; nit: We can just drop the else branch. https://chromiumcodereview.appspot.com/10905232/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
