Reviewers: Vyacheslav Egorov,

Message:
This should make the bots green again.

Description:
Fix mistakes in r11472

[email protected]

Please review this at https://chromiumcodereview.appspot.com/10274008/

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

Affected files:
  M src/ia32/stub-cache-ia32.cc
  M src/x64/stub-cache-x64.cc


Index: src/ia32/stub-cache-ia32.cc
diff --git a/src/ia32/stub-cache-ia32.cc b/src/ia32/stub-cache-ia32.cc
index 2629984c7666535bc33274b5e054a2c32ae47247..734b89b0f3a1b21523d6503b8f788e44283b89ac 100644
--- a/src/ia32/stub-cache-ia32.cc
+++ b/src/ia32/stub-cache-ia32.cc
@@ -3644,6 +3644,7 @@ void KeyedStoreStubCompiler::GenerateStoreExternalArray(
             case EXTERNAL_INT_ELEMENTS:
             case EXTERNAL_UNSIGNED_INT_ELEMENTS:
               __ mov(Operand(edi, ecx, times_2, 0), ebx);
+              break;
             default:
               UNREACHABLE();
               break;
Index: src/x64/stub-cache-x64.cc
diff --git a/src/x64/stub-cache-x64.cc b/src/x64/stub-cache-x64.cc
index e375e5b81ee60c618512a1ada3d84ae37585269c..05b767379548438018b13e4daadcfe6fef907861 100644
--- a/src/x64/stub-cache-x64.cc
+++ b/src/x64/stub-cache-x64.cc
@@ -3403,9 +3403,9 @@ void KeyedStoreStubCompiler::GenerateStoreExternalArray(
       // behavior.
       // Fast path: use machine instruction to convert to int64. If that
       // fails (out-of-range), go into the runtime.
-      __ cvttsd2siq(rdx, xmm0);
+      __ cvttsd2siq(r8, xmm0);
       __ Set(kScratchRegister, V8_UINT64_C(0x8000000000000000));
-      __ cmpq(rdx, kScratchRegister);
+      __ cmpq(r8, kScratchRegister);
       __ j(equal, &slow);

       // rdx: value (converted to an untagged integer)
@@ -3414,15 +3414,15 @@ void KeyedStoreStubCompiler::GenerateStoreExternalArray(
       switch (elements_kind) {
         case EXTERNAL_BYTE_ELEMENTS:
         case EXTERNAL_UNSIGNED_BYTE_ELEMENTS:
-          __ movb(Operand(rbx, rdi, times_1, 0), rdx);
+          __ movb(Operand(rbx, rdi, times_1, 0), r8);
           break;
         case EXTERNAL_SHORT_ELEMENTS:
         case EXTERNAL_UNSIGNED_SHORT_ELEMENTS:
-          __ movw(Operand(rbx, rdi, times_2, 0), rdx);
+          __ movw(Operand(rbx, rdi, times_2, 0), r8);
           break;
         case EXTERNAL_INT_ELEMENTS:
         case EXTERNAL_UNSIGNED_INT_ELEMENTS:
-          __ movl(Operand(rbx, rdi, times_4, 0), rdx);
+          __ movl(Operand(rbx, rdi, times_4, 0), r8);
           break;
         case EXTERNAL_PIXEL_ELEMENTS:
         case EXTERNAL_FLOAT_ELEMENTS:


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

Reply via email to