Changeset 63228 in webkit for trunk/JavaScriptCore/assembler

Timestamp:

Jul 13, 2010, 11:56:18 AM (15 years ago)

Author:

[email protected]

Message:

Fix the constant encoding in data transfer instructions on ARM
​https://bugs.webkit.org/show_bug.cgi?id=42166

Reviewed by Gavin Barraclough.

The getImm function is designed to produce modified immediate constant
for data processing instructions. It should not be used to encode
any constant for data transfer. In the current situation there is no
way to use any immediate constant for data transfer. So, the moveImm
function is the desired method to pass the offset value to the data
transfer instructions.

Reported by Jacob Bramley.

• assembler/ARMAssembler.cpp:

(JSC::ARMAssembler::dataTransfer32):

• assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::call32):

Location:

trunk/JavaScriptCore/assembler

Files:

• ARMAssembler.cpp (modified) (2 diffs)
• MacroAssemblerARM.h (modified) (2 diffs)

trunk/JavaScriptCore/assembler/ARMAssembler.cpp

@@ -273 +273 @@
             dtr_u(isLoad, srcDst, ARMRegisters::S0, (offset & 0xfff) | transferFlag);
         } else {
-            ARMWord reg = getImm(offset, ARMRegisters::S0);
-            dtr_ur(isLoad, srcDst, base, reg | transferFlag);
+            moveImm(offset, ARMRegisters::S0);
+            dtr_ur(isLoad, srcDst, base, ARMRegisters::S0 | transferFlag);
         }
     } else {
@@ -284 +284 @@
             dtr_d(isLoad, srcDst, ARMRegisters::S0, (offset & 0xfff) | transferFlag);
         } else {
-            ARMWord reg = getImm(offset, ARMRegisters::S0);
-            dtr_dr(isLoad, srcDst, base, reg | transferFlag);
+            moveImm(offset, ARMRegisters::S0);
+            dtr_dr(isLoad, srcDst, base, ARMRegisters::S0 | transferFlag);
         }
     }

trunk/JavaScriptCore/assembler/MacroAssemblerARM.h

@@ -957 +957 @@
                 m_assembler.dtr_u(true, targetReg, tmpReg, offset & 0xfff);
             } else {
-                ARMWord reg = m_assembler.getImm(offset, tmpReg);
+                m_assembler.moveImm(offset, tmpReg);
                 prepareCall();
-                m_assembler.dtr_ur(true, targetReg, base, reg);
+                m_assembler.dtr_ur(true, targetReg, base, tmpReg);
             }
         } else  {
@@ -971 +971 @@
                 m_assembler.dtr_d(true, targetReg, tmpReg, offset & 0xfff);
             } else {
-                ARMWord reg = m_assembler.getImm(offset, tmpReg);
+                m_assembler.moveImm(offset, tmpReg);
                 prepareCall();
-                m_assembler.dtr_dr(true, targetReg, base, reg);
+                m_assembler.dtr_dr(true, targetReg, base, tmpReg);
             }
         }