lgtm

LGTM with nits


http://codereview.chromium.org/8383002/diff/1/src/x64/codegen-x64.cc
File src/x64/codegen-x64.cc (right):

http://codereview.chromium.org/8383002/diff/1/src/x64/codegen-x64.cc#newcode216
src/x64/codegen-x64.cc:216: // length does not have to be set since we
still use the exact same field
Or even simpler:
NaNs.  The Array's length remains unchanged.

http://codereview.chromium.org/8383002/

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

Reply via email to