Revision: 4605
Author: [email protected]
Date: Thu May  6 05:57:11 2010
Log: Fixing a performance bug introduced in r4581.
Review URL: http://codereview.chromium.org/1910005
http://code.google.com/p/v8/source/detail?r=4605

Modified:
 /branches/bleeding_edge/src/arm/codegen-arm.cc
 /branches/bleeding_edge/src/ia32/codegen-ia32.cc

=======================================
--- /branches/bleeding_edge/src/arm/codegen-arm.cc      Thu May  6 04:59:10 2010
+++ /branches/bleeding_edge/src/arm/codegen-arm.cc      Thu May  6 05:57:11 2010
@@ -8522,9 +8522,9 @@
   // string length. A negative value will be greater (unsigned comparison).
   __ ldr(r0, MemOperand(sp, kPreviousIndexOffset));
   __ tst(r0, Operand(kSmiTagMask));
-  __ b(eq, &runtime);
+  __ b(ne, &runtime);
   __ cmp(r3, Operand(r0));
-  __ b(le, &runtime);
+  __ b(ls, &runtime);

   // r2: Number of capture registers
   // subject: Subject string
=======================================
--- /branches/bleeding_edge/src/ia32/codegen-ia32.cc Thu May 6 02:35:18 2010 +++ /branches/bleeding_edge/src/ia32/codegen-ia32.cc Thu May 6 05:57:11 2010
@@ -10956,7 +10956,7 @@
   // string length. A negative value will be greater (unsigned comparison).
   __ mov(eax, Operand(esp, kPreviousIndexOffset));
   __ test(eax, Immediate(kSmiTagMask));
-  __ j(zero, &runtime);
+  __ j(not_zero, &runtime);
   __ cmp(eax, Operand(ebx));
   __ j(above_equal, &runtime);

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

Reply via email to