Revision: 13465
Author:   [email protected]
Date:     Tue Jan 22 08:18:59 2013
Log: MIPS: Incorrect ARM assembly in MacroAssembler::TestJSArrayForAllocationSiteInfo

Port r13462 (74f0ddf6)

Original commit message:
Incorrect ARM assembly in MacroAssembler::TestJSArrayForAllocationSiteInfo Restored test code in allocation-site-info.js that was failing on ARM because of this bug.

BUG=
TEST=

Review URL: https://codereview.chromium.org/11896037
Patch from Akos Palfi <[email protected]>.
http://code.google.com/p/v8/source/detail?r=13465

Modified:
 /branches/bleeding_edge/src/mips/macro-assembler-mips.cc

=======================================
--- /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Mon Jan 21 05:21:53 2013 +++ /branches/bleeding_edge/src/mips/macro-assembler-mips.cc Tue Jan 22 08:18:59 2013
@@ -5454,11 +5454,12 @@
       ExternalReference::new_space_start(isolate());
   ExternalReference new_space_allocation_top =
       ExternalReference::new_space_allocation_top_address(isolate());
-  lw(scratch_reg, FieldMemOperand(receiver_reg,
-      JSArray::kSize + AllocationSiteInfo::kSize - kHeapObjectTag));
+  Addu(scratch_reg, receiver_reg,
+ Operand(JSArray::kSize + AllocationSiteInfo::kSize - kHeapObjectTag));
   Branch(&no_info_available, lt, scratch_reg, Operand(new_space_start));
-  Branch(&no_info_available, gt, scratch_reg,
-      Operand(new_space_allocation_top));
+  li(at, Operand(new_space_allocation_top));
+  lw(at, MemOperand(at));
+  Branch(&no_info_available, gt, scratch_reg, Operand(at));
   lw(scratch_reg, MemOperand(scratch_reg, -AllocationSiteInfo::kSize));
   Branch(allocation_info_present, eq, scratch_reg,
       Operand(Handle<Map>(isolate()->heap()->allocation_site_info_map())));

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

Reply via email to