Revision: 10624
Author: [email protected]
Date: Tue Feb 7 06:03:13 2012
Log: Properly initialize element-transitioning array literals on ARM.
[email protected]
BUG=v8:1930
TEST=unit tests with enable_vfp3=true
Review URL: https://chromiumcodereview.appspot.com/9316131
http://code.google.com/p/v8/source/detail?r=10624
Modified:
/branches/bleeding_edge/src/arm/code-stubs-arm.cc
/branches/bleeding_edge/src/arm/macro-assembler-arm.h
=======================================
--- /branches/bleeding_edge/src/arm/code-stubs-arm.cc Fri Jan 27 08:54:22
2012
+++ /branches/bleeding_edge/src/arm/code-stubs-arm.cc Tue Feb 7 06:03:13
2012
@@ -7358,7 +7358,7 @@
// Array literal has ElementsKind of FAST_DOUBLE_ELEMENTS.
__ bind(&double_elements);
__ ldr(r5, FieldMemOperand(r1, JSObject::kElementsOffset));
- __ StoreNumberToDoubleElements(r0, r3, r1, r5, r6, r7, r9, r10,
+ __ StoreNumberToDoubleElements(r0, r3, r1, r5, r6, r7, r9, r2,
&slow_elements);
__ Ret();
}
=======================================
--- /branches/bleeding_edge/src/arm/macro-assembler-arm.h Thu Feb 2
03:22:26 2012
+++ /branches/bleeding_edge/src/arm/macro-assembler-arm.h Tue Feb 7
06:03:13 2012
@@ -803,7 +803,8 @@
// Check to see if maybe_number can be stored as a double in
// FastDoubleElements. If it can, store it at the index specified by key
in
- // the FastDoubleElements array elements, otherwise jump to fail.
+ // the FastDoubleElements array elements. Otherwise jump to fail, in
which
+ // case scratch2, scratch3 and scratch4 are unmodified.
void StoreNumberToDoubleElements(Register value_reg,
Register key_reg,
Register receiver_reg,
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
