[x86/Linux] Revise VirtualMethodFixupStub (#9929)

* [x86/Linux] Revise VirtualMethodFixupStub

diff --git a/src/vm/i386/asmhelpers.S b/src/vm/i386/asmhelpers.S
index 921e09fef1223b260ed9d38db91d6dcc716575db..060bb02933b65ef5a0e1e9e03ad443756e986eef 100644 (file)
--- a/src/vm/i386/asmhelpers.S
+++ b/src/vm/i386/asmhelpers.S
@@ -821,8 +821,8 @@ NESTED_END DelayLoad_MethodCall, _TEXT
 //  to optionally patch the target of the jump so that we do not take this slow path again.
 //
 NESTED_ENTRY VirtualMethodFixupStub, _TEXT, NoHandler
-    // Pop the return address. It points right after the call instruction in the thunk.
-    pop     eax
+    // Get the return address. It points right after the call instruction in the thunk.
+    mov     eax, [esp]
     // Calculate the address of the thunk
     sub     eax, 5
 
@@ -836,9 +836,12 @@ NESTED_ENTRY VirtualMethodFixupStub, _TEXT, NoHandler
     // Set frame pointer
     PROLOG_END
 
+    sub     esp, 8
     push    eax         // address of the thunk
     push    ecx         // this ptr
+    CHECK_STACK_ALIGNMENT
     call    C_FUNC(VirtualMethodFixupWorker)
+    add     esp, 8
 
     // Restore stack pointer
     EPILOG_BEG
@@ -850,6 +853,9 @@ NESTED_ENTRY VirtualMethodFixupStub, _TEXT, NoHandler
     // Pop ebp frame
     EPILOG_END
 
+    // Pop return address
+    add   esp, 4
+
 PATCH_LABEL VirtualMethodFixupPatchLabel
     // Proceed to execute the actual method.
     jmp     eax