Revision: 9768
Author: [email protected]
Date: Tue Oct 25 02:07:09 2011
Log: CompileArrayPushCall should not use context register as a scratch.
[email protected]
Review URL: http://codereview.chromium.org/8330026
http://code.google.com/p/v8/source/detail?r=9768
Modified:
/branches/bleeding_edge/src/ia32/stub-cache-ia32.cc
/branches/bleeding_edge/src/x64/stub-cache-x64.cc
=======================================
--- /branches/bleeding_edge/src/ia32/stub-cache-ia32.cc Mon Oct 24 03:55:00
2011
+++ /branches/bleeding_edge/src/ia32/stub-cache-ia32.cc Tue Oct 25 02:07:09
2011
@@ -1511,8 +1511,8 @@
// the new element is non-Smi. For now, delegate to the builtin.
Label no_fast_elements_check;
__ JumpIfSmi(edi, &no_fast_elements_check);
- __ mov(esi, FieldOperand(edx, HeapObject::kMapOffset));
- __ CheckFastObjectElements(esi, &call_builtin, Label::kFar);
+ __ mov(ecx, FieldOperand(edx, HeapObject::kMapOffset));
+ __ CheckFastObjectElements(ecx, &call_builtin, Label::kFar);
__ bind(&no_fast_elements_check);
// We could be lucky and the elements array could be at the top of
=======================================
--- /branches/bleeding_edge/src/x64/stub-cache-x64.cc Thu Oct 20 10:08:53
2011
+++ /branches/bleeding_edge/src/x64/stub-cache-x64.cc Tue Oct 25 02:07:09
2011
@@ -1488,8 +1488,8 @@
// the new element is non-Smi. For now, delegate to the builtin.
Label no_fast_elements_check;
__ JumpIfSmi(rdi, &no_fast_elements_check);
- __ movq(rsi, FieldOperand(rdx, HeapObject::kMapOffset));
- __ CheckFastObjectElements(rsi, &call_builtin, Label::kFar);
+ __ movq(rcx, FieldOperand(rdx, HeapObject::kMapOffset));
+ __ CheckFastObjectElements(rcx, &call_builtin, Label::kFar);
__ bind(&no_fast_elements_check);
ExternalReference new_space_allocation_top =
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
