http://codereview.chromium.org/1628003/diff/1/4 File src/arm/ic-arm.cc (right):
http://codereview.chromium.org/1628003/diff/1/4#newcode728 src/arm/ic-arm.cc:728: __ push(r1); // receiver Use stm here? http://codereview.chromium.org/1628003/diff/1/2 File test/mjsunit/string-index.js (right): http://codereview.chromium.org/1628003/diff/1/2#newcode170 test/mjsunit/string-index.js:170: // Test for keyed ic. Please also cover the case when the receiver becomes smi. http://codereview.chromium.org/1628003/diff/1/2#newcode178 test/mjsunit/string-index.js:178: assertEquals(new Number(foo[1]) + 1, result); Why "+ 1"? http://codereview.chromium.org/1628003/diff/1/2#newcode188 test/mjsunit/string-index.js:188: assertEquals(new Number(foo[key]) + 1, result); Same here. http://codereview.chromium.org/1628003 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev To unsubscribe, reply using "remove me" as the subject.
