http://codereview.chromium.org/7400020/diff/4001/src/objects.cc
File src/objects.cc (right):

http://codereview.chromium.org/7400020/diff/4001/src/objects.cc#newcode7575
src/objects.cc:7575: FixedArray::OffsetOfElementAt(value);
Shouldn't this be FixedDoubleArray::OffsetOfElementAt to not have the
filler zap the middle instead of the end of the elements array?

http://codereview.chromium.org/7400020/diff/4001/test/mjsunit/unbox-double-arrays.js
File test/mjsunit/unbox-double-arrays.js (right):

http://codereview.chromium.org/7400020/diff/4001/test/mjsunit/unbox-double-arrays.js#newcode379
test/mjsunit/unbox-double-arrays.js:379:
assertEquals(expected_array_value(7), large_array3[7]);
Test the element at "length - 1"?

http://codereview.chromium.org/7400020/

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

Reply via email to