Changeset 290882 in webkit

Timestamp:

Mar 7, 2022, 3:47:23 AM (5 years ago)

Author:

commit-queue@webkit.org

Message:

[JSC] Add missing CPU(RISCV64) guards in CTI thunk prologue, epilogue generators
​https://bugs.webkit.org/show_bug.cgi?id=237517

Unreviewed, adding missing CPU(RISCV64) guards to the CTI thunk prologue
and epilogue generators in CCallHelpers, reusing the ARM variants of the
pair push/pop calls.

Patch by Zan Dobersek <​zdobersek@igalia.com> on 2022-03-07

• jit/CCallHelpers.cpp:

(JSC::CCallHelpers::emitCTIThunkPrologue):
(JSC::CCallHelpers::emitCTIThunkEpilogue):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• jit/CCallHelpers.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2022-03-07  Zan Dobersek  <zdobersek@igalia.com>
+
+        [JSC] Add missing CPU(RISCV64) guards in CTI thunk prologue, epilogue generators
+        https://bugs.webkit.org/show_bug.cgi?id=237517
+
+        Unreviewed, adding missing CPU(RISCV64) guards to the CTI thunk prologue
+        and epilogue generators in CCallHelpers, reusing the ARM variants of the
+        pair push/pop calls.
+
+        * jit/CCallHelpers.cpp:
+        (JSC::CCallHelpers::emitCTIThunkPrologue):
+        (JSC::CCallHelpers::emitCTIThunkEpilogue):
+
 2022-03-05  Mark Lam  <mark.lam@apple.com>
 

trunk/Source/JavaScriptCore/jit/CCallHelpers.cpp

@@ -105 +105 @@
 #if CPU(X86_64)
     push(X86Registers::ebp); // return address pushed by the call instruction
-#elif CPU(ARM64) || CPU(ARM_THUMB2)
+#elif CPU(ARM64) || CPU(ARM_THUMB2) || CPU(RISCV64)
     pushPair(framePointerRegister, linkRegister);
 #elif CPU(MIPS)
@@ -125 +125 @@
 #if CPU(X86_64)
     pop(X86Registers::ebp); // Return address left on stack
-#elif CPU(ARM64) || CPU(ARM_THUMB2)
+#elif CPU(ARM64) || CPU(ARM_THUMB2) || CPU(RISCV64)
     popPair(framePointerRegister, linkRegister);
 #elif CPU(MIPS)