Reviewers: Vyacheslav Egorov,

Message:
This seems to be an accident. Compare with macro-assembler-ia32.cc:110 where it
returns if the test results in an equal.

Description:
Fixing a bug in arm as pointed out in issue 1759.

BUG=v8:1759


Please review this at http://codereview.chromium.org/8229012/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files:
  M src/arm/macro-assembler-arm.cc


Index: src/arm/macro-assembler-arm.cc
diff --git a/src/arm/macro-assembler-arm.cc b/src/arm/macro-assembler-arm.cc
index b274b2fe42aea26ad74784bab6a50931839ee45c..918f9ebe060d245df5a6e6026ce5363afb98c497 100644
--- a/src/arm/macro-assembler-arm.cc
+++ b/src/arm/macro-assembler-arm.cc
@@ -551,7 +551,7 @@ void MacroAssembler::RememberedSetHelper(Register object, // For debug tests.
     b(eq, &done);
   } else {
     ASSERT(and_then == kReturnAtEnd);
-    Ret(ne);
+    Ret(eq);
   }
   push(lr);
   StoreBufferOverflowStub store_buffer_overflow =


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

Reply via email to