Changeset 142751 in webkit

Timestamp:

Feb 13, 2013 8:23:29 AM (11 years ago)

Author:

zherczeg@webkit.org

Message:

replaceWithJump should not decrease the offset by 1 on ARM traditional.
​https://bugs.webkit.org/show_bug.cgi?id=109689

Reviewed by Zoltan Herczeg.

• assembler/ARMAssembler.h:

(JSC::ARMAssembler::replaceWithJump):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• assembler/ARMAssembler.h (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-02-13  Zoltan Herczeg  <zherczeg@webkit.org>
+
+        replaceWithJump should not decrease the offset by 1 on ARM traditional.
+        https://bugs.webkit.org/show_bug.cgi?id=109689
+
+        Reviewed by Zoltan Herczeg.
+
+        * assembler/ARMAssembler.h:
+        (JSC::ARMAssembler::replaceWithJump):
+
 2013-02-12  Joseph Pecoraro  <pecoraro@apple.com>
 

trunk/Source/JavaScriptCore/assembler/ARMAssembler.h

@@ -905 +905 @@
         static void replaceWithJump(void* instructionStart, void* to)
         {
-            ARMWord* instruction = reinterpret_cast<ARMWord*>(instructionStart) - 1;
+            ARMWord* instruction = reinterpret_cast<ARMWord*>(instructionStart);
             intptr_t difference = reinterpret_cast<intptr_t>(to) - (reinterpret_cast<intptr_t>(instruction) + DefaultPrefetchOffset * sizeof(ARMWord));