Changeset 293087 in webkit for trunk/Source/JavaScriptCore/assembler/MacroAssemblerMIPS.h

Timestamp:

Apr 20, 2022, 12:54:47 AM (3 years ago)

Author:

Mikhail R. Gadelha

Message:

Unify calls and checks for CellTag
​https://bugs.webkit.org/show_bug.cgi?id=238025

Reviewed by Yusuke Suzuki.

This patch moves all the places where cell tags are written and checked
when branching: this will be needed when implementing the concurrency on
32 bits archs, so we can focus on changing storeCell, branchIfNotCell and
branchIfCell to implement the mutex locks/unlocks.

It also reduces the code size in JetStream2 by ~1% on average on 32
bits platforms.

• assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::store8):
(JSC::MacroAssemblerARMv7::storePair32):

• assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::storePair32):

• dfg/DFGOSRExit.cpp:

(JSC::DFG::OSRExit::compileExit):

• dfg/DFGOSRExitCompilerCommon.cpp:

(JSC::DFG::reifyInlinedCallFrames):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::checkArgumentTypes):

• dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::fillSpeculateCell):
(JSC::DFG::SpeculativeJIT::compilePeepHoleObjectStrictEquality):

• jit/AssemblyHelpers.cpp:

(JSC::AssemblyHelpers::jitAssertIsCell):

• jit/AssemblyHelpers.h:

(JSC::AssemblyHelpers::storeCell):
(JSC::AssemblyHelpers::storeTrustedValue):
(JSC::AssemblyHelpers::branchIfNotCell):
(JSC::AssemblyHelpers::branchIfCell):

• jit/CallFrameShuffler32_64.cpp:

(JSC::CallFrameShuffler::emitStore):

• jit/JSInterfaceJIT.h:

(JSC::JSInterfaceJIT::emitLoadJSCell):

File:

• trunk/Source/JavaScriptCore/assembler/MacroAssemblerMIPS.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/assembler/MacroAssemblerMIPS.h

@@ -1652 +1652 @@
     }
 
+    void storePair32(RegisterID src1, TrustedImm32 imm, Address address)
+    {
+        move(imm, addrTempRegister);
+        storePair32(src1, addrTempRegister, address);
+    }
+
+    void storePair32(TrustedImmPtr immPtr, TrustedImm32 imm32, Address address)
+    {
+        move(immPtr, addrTempRegister);
+        move(imm32, dataTempRegister);
+        storePair32(addrTempRegister, dataTempRegister, address);
+    }
+
+    void storePair32(TrustedImm32 imm1, TrustedImm32 imm2, Address address)
+    {
+        move(imm1, addrTempRegister);
+        RegisterID scratch = addrTempRegister;
+        if (imm1.m_value != imm2.m_value) {
+            scratch = dataTempRegister;
+            move(imm2, scratch);
+        }
+        storePair32(addrTempRegister, scratch, address);
+    }
+
     void storePair32(RegisterID src1, RegisterID src2, RegisterID dest)
     {
@@ -1672 +1696 @@
         store32(src1, address);
         store32(src2, address.withOffset(4));
+    }
+
+    void storePair32(TrustedImm32 imm1, TrustedImm32 imm2, BaseIndex address)
+    {
+        store32(imm1, address);
+        store32(imm2, address.withOffset(4));
+    }
+
+    void storePair32(RegisterID src1, TrustedImm32 imm, const void* address)
+    {
+        move(TrustedImmPtr(address), addrTempRegister);
+        move(imm, dataTempRegister);
+        storePair32(src1, dataTempRegister, addrTempRegister);
     }