LGTM with nits.

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

https://chromiumcodereview.appspot.com/10386045/diff/1/test/mjsunit/regress/regress-117409.js#newcode32
test/mjsunit/regress/regress-117409.js:32: // literal with a fast double
elements backing store
nit: Proper sentences with capitalization and full stop, please. (More
below.)

https://chromiumcodereview.appspot.com/10386045/diff/1/test/mjsunit/regress/regress-117409.js#newcode40
test/mjsunit/regress/regress-117409.js:40: //   backing store will be
replaces with empty fixed array
nit: replaced

https://chromiumcodereview.appspot.com/10386045/

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

Reply via email to