Revision: 4952
Author: [email protected]
Date: Fri Jun 25 05:31:49 2010
Log: Fix bug in CheckBothPositiveSmi macro.
Fix lint.
Review URL: http://codereview.chromium.org/2826021
http://code.google.com/p/v8/source/detail?r=4952
Modified:
/branches/bleeding_edge/src/ia32/codegen-ia32.cc
/branches/bleeding_edge/src/x64/macro-assembler-x64.cc
=======================================
--- /branches/bleeding_edge/src/ia32/codegen-ia32.cc Fri Jun 25 04:24:41
2010
+++ /branches/bleeding_edge/src/ia32/codegen-ia32.cc Fri Jun 25 05:31:49
2010
@@ -13410,7 +13410,6 @@
// Just jump to runtime to create the sub string.
__ bind(&runtime);
__ TailCallRuntime(Runtime::kSubString, 3, 1);
-
}
=======================================
--- /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Wed Jun 23
04:48:30 2010
+++ /branches/bleeding_edge/src/x64/macro-assembler-x64.cc Fri Jun 25
05:31:49 2010
@@ -652,8 +652,8 @@
if (first.is(second)) {
return CheckPositiveSmi(first);
}
- movl(kScratchRegister, first);
- orl(kScratchRegister, second);
+ movq(kScratchRegister, first);
+ or_(kScratchRegister, second);
rol(kScratchRegister, Immediate(1));
testl(kScratchRegister, Immediate(0x03));
return zero;
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
