Changeset 38064 in vbox for trunk/src/recompiler/tcg/i386/tcg-target.c

Timestamp:

Jul 19, 2011 10:08:30 AM (13 years ago)

Author:

vboxsync

Message:

tcg-target.c: Make sure the stack is aligned on darwin (VBoxREM64 only).

File:

• trunk/src/recompiler/tcg/i386/tcg-target.c (modified) (8 diffs)

trunk/src/recompiler/tcg/i386/tcg-target.c

@@ -464 +464 @@
                the 32-bit-mode absolute addressing encoding.  */
 #ifdef VBOX
-            tcg_target_long pc = (tcg_target_long)s->code_ptr 
+            tcg_target_long pc = (tcg_target_long)s->code_ptr
                                + tcg_calc_opc_len(s, opc, r, 0, 0) + 1 + 4;
 #else
@@ -993 +993 @@
 {
 #ifdef VBOX
-    tcg_target_long disp = dest - (tcg_target_long)s->code_ptr 
+    tcg_target_long disp = dest - (tcg_target_long)s->code_ptr
                          - tcg_calc_opc_len(s, call ? OPC_CALL_Jz : OPC_JMP_long, 0, 0, 0)
                          - 4;
@@ -1519 +1519 @@
     }
 
-#if !defined(VBOX) || !defined(REM_PHYS_ADDR_IN_TLB)
+# if !defined(VBOX) || !defined(REM_PHYS_ADDR_IN_TLB)
+#  if defined(VBOX) && defined(RT_OS_DARWIN) && ARCH_BITS == 32
+#   define VBOX_16_BYTE_STACK_ALIGN
+#  endif
 
     /* XXX: move that code at the end of the TB */
@@ -1531 +1534 @@
         if (opc == 3) {
             tcg_out_mov(s, TCG_TYPE_I32, TCG_REG_ECX, data_reg2);
+#  ifdef VBOX_16_BYTE_STACK_ALIGN
+            tcg_out_subi(s, TCG_REG_ESP, 12);
+#  endif
             tcg_out_pushi(s, mem_index);
             stack_adjust = 4;
@@ -1540 +1546 @@
         if (opc == 3) {
             tcg_out_mov(s, TCG_TYPE_I32, TCG_REG_EDX, args[addrlo_idx + 1]);
+#  ifdef VBOX_16_BYTE_STACK_ALIGN
+            tcg_out_pushi(s, 0);
+#  endif
             tcg_out_pushi(s, mem_index);
             tcg_out_push(s, data_reg2);
@@ -1557 +1566 @@
                 break;
             }
+#  ifdef VBOX_16_BYTE_STACK_ALIGN
+            tcg_out_subi(s, TCG_REG_ESP, 12);
+#  endif
             tcg_out_pushi(s, mem_index);
             stack_adjust = 4;
@@ -1564 +1576 @@
     tcg_out_calli(s, (tcg_target_long)qemu_st_helpers[s_bits]);
 
+#  ifdef VBOX_16_BYTE_STACK_ALIGN
+    if (stack_adjust != 0) {
+        tcg_out_addi(s, TCG_REG_ESP, RT_ALIGN(stack_adjust, 16));
+    }
+#  else
     if (stack_adjust == (TCG_TARGET_REG_BITS / 8)) {
         /* Pop and discard.  This is 2 bytes smaller than the add.  */
@@ -1570 +1587 @@
         tcg_out_addi(s, TCG_REG_ESP, stack_adjust);
     }
-
-#else  /* VBOX && REM_PHYS_ADDR_IN_TLB */
-# error Borked
+#  endif
+
+# else  /* VBOX && REM_PHYS_ADDR_IN_TLB */
+#  error Borked
     tcg_out_vbox_phys_write(s, opc, r0, data_reg, data_reg2);
-#endif /* VBOX && REM_PHYS_ADDR_IN_TLB */
+# endif /* VBOX && REM_PHYS_ADDR_IN_TLB */
 
     /* label2: */