2002-07-24 Tim Coleman <tim@timcoleman.com>
[mono.git] / mono / jit / x86.brg
index a2a3eab7d51d8951087631cdc1fa1ed714a60cfa..0d419bc1a17afd0e5804fe85318ce94bf38add7b 100644 (file)
@@ -42,6 +42,9 @@ enum {
 };
 #undef OPDEF
 
+/* alignment of activation frames */
+#define MONO_FRAME_ALIGNMENT 4
+
 void print_lmf (void);
 
 #define MBTREE_TYPE  MBTree
@@ -91,9 +94,10 @@ struct _MBTree {
                MonoClass *klass;
                MonoClassField *field;
                X86AddressInfo ainfo;
-               MonoJitCallInfo ci;
                MonoJitFieldInfo fi;
                MonoJitBranchInfo bi;
+               MonoJitCallInfo call_info;        
+               MonoJitArgumentInfo arg_info;     
        } data;
 };
 
@@ -191,6 +195,36 @@ debug_memcopy (void *dest, const void *src, size_t n);
                x86_call_code (s->code, 0);                    \
        } while (0); 
 
+#define X86_ARG_PAD(pad) do {                                               \
+       if (pad) {                                                          \
+               if (pad == 4)                                               \
+                       x86_push_reg (s->code, X86_EAX);                    \
+               else                                                        \
+                       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, pad);   \
+       }                                                                   \
+} while (0)
+
+#define X86_CALL_END    do {                                       \
+        int size = tree->data.call_info.frame_size;                \
+        if (size)                                                  \
+               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, size); \
+} while (0)
+
+#define X86_CALL_BEGIN  do {                                                     \
+        int pad = tree->data.call_info.pad;                                      \
+        X86_ARG_PAD (pad);                                                       \
+       if (tree->left->op != MB_TERM_NOP) {                                     \
+               mono_assert (lreg >= 0);                                         \
+               x86_push_reg (s->code, lreg);                                    \
+               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);              \
+       }                                                                        \
+       if (tree->data.call_info.vtype_num) {                                    \
+               int offset = VARINFO (s, tree->data.call_info.vtype_num).offset; \
+               x86_lea_membase (s->code, treg, X86_EBP, offset);                \
+               x86_push_reg (s->code, treg);                                    \
+       }                                                                        \
+} while (0)
+
 /* we use this macro to move one lreg to another - source and
    destination may overlap, but the register allocator has to
    make sure that ((d1 < d2) && (s1 < s2))
@@ -223,7 +257,7 @@ debug_memcopy (void *dest, const void *src, size_t n);
 %term LDIND_I1 LDIND_U1 LDIND_I2 LDIND_U2 LDIND_I4 LDIND_I8 LDIND_R4 LDIND_R8 LDIND_OBJ 
 %term STIND_I1 STIND_I2 STIND_I4 STIND_I8 STIND_R4 STIND_R8 STIND_OBJ
 %term ADDR_L ADDR_G ARG_I4 ARG_I8 ARG_R4 ARG_R8 ARG_OBJ CALL_I4 CALL_I8 CALL_R8 CALL_VOID
-%term BREAK SWITCH BR RET_VOID RET RET_OBJ ENDFINALLY JMP
+%term BREAK SWITCH BR RET_VOID RET RET_OBJ ENDFINALLY ENDFILTER JMP
 %term ADD ADD_OVF ADD_OVF_UN SUB SUB_OVF SUB_OVF_UN MUL MUL_OVF MUL_OVF_UN 
 %term DIV DIV_UN REM REM_UN AND OR XOR SHL SHR SHR_UN NEG NOT CKFINITE
 %term COMPARE CBRANCH BRTRUE BRFALSE CSET
@@ -240,7 +274,7 @@ debug_memcopy (void *dest, const void *src, size_t n);
 %term REMOTE_STIND_I8 REMOTE_STIND_R4 REMOTE_STIND_R8 REMOTE_STIND_OBJ
 %term SIN COS SQRT
 
-%term FUNC1 PROC3 FREE OBJADDR
+%term FUNC1 PROC2 PROC3 FREE OBJADDR VTADDR
 
 #
 # we start at stmt
@@ -346,16 +380,12 @@ addr: ADD (index, base) {
 # we pass exception in ECX to catch handler
 reg: EXCEPTION {
        int offset = VARINFO (s, tree->data.i).offset;
-       int reg = VARINFO (s, tree->data.i).reg;
 
        if (tree->reg1 != X86_ECX)
                x86_mov_reg_reg (s->code, tree->reg1, X86_ECX, 4);
        
        /* store it so that we can RETHROW it later */
-       if (reg < 0)
-               x86_mov_membase_reg (s->code, X86_EBP, offset, tree->reg1, 4);
-       else
-               x86_mov_reg_reg (s->code, reg, tree->reg1, 4);
+       x86_mov_membase_reg (s->code, X86_EBP, offset, tree->reg1, 4);
 }
 
 stmt: THROW (reg) {
@@ -368,14 +398,10 @@ stmt: THROW (reg) {
 }
 
 stmt: RETHROW {
-       int off = VARINFO (s, tree->data.i).offset;
-       int reg = VARINFO (s, tree->data.i).reg;
+       int offset = VARINFO (s, tree->data.i).offset;
        gpointer target;
 
-       if (reg < 0)
-               x86_push_membase (s->code, X86_EBP, off);
-       else
-               x86_push_reg (s->code, reg);
+       x86_push_membase (s->code, X86_EBP, offset);
 
        target = arch_get_throw_exception ();
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, target);
@@ -397,6 +423,16 @@ stmt: ENDFINALLY {
        x86_ret (s->code);
 }
 
+stmt: ENDFILTER (reg) {
+       /* restore ESP - which can be modified when we allocate value types
+        * in the filter */
+       x86_mov_reg_membase (s->code, X86_ESP, X86_EBP, mono_exc_esp_offset, 4);
+       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, 4);
+       if (tree->left->reg1 != X86_EAX)
+               x86_mov_reg_reg (s->code, X86_EAX, tree->left->reg1, 4);             
+       x86_ret (s->code);
+}
+
 stmt: STIND_I4 (ADDR_L, ADD (LDIND_I4 (ADDR_L), CONST_I4)) {
        int vn = tree->left->data.i;
        int treg = VARINFO (s, vn).reg;
@@ -1646,7 +1682,7 @@ reg: NEWARR (reg) {
        x86_push_imm (s->code, tree->data.p);
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_array_new_wrapper);
        x86_call_code (s->code, 0);
-       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, sizeof (gpointer) + 4);
+       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 8);
 
        x86_pop_reg (s->code, X86_EDX);
        x86_pop_reg (s->code, X86_ECX);
@@ -1668,7 +1704,7 @@ reg: NEWARR_SPEC (reg) {
        x86_push_imm (s->code, tree->data.p);
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_array_new_specific);
        x86_call_code (s->code, 0);
-       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, sizeof (gpointer) + 4);
+       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 8);
 
        x86_pop_reg (s->code, X86_EDX);
        x86_pop_reg (s->code, X86_ECX);
@@ -1689,7 +1725,7 @@ reg: NEWOBJ {
        x86_push_imm (s->code, tree->data.klass);
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_object_new_wrapper);
        x86_call_code (s->code, 0);
-       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, sizeof (gpointer));
+       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 4);
 
        x86_pop_reg (s->code, X86_EDX);
        x86_pop_reg (s->code, X86_ECX);
@@ -1709,7 +1745,7 @@ reg: NEWOBJ_SPEC {
        x86_push_imm (s->code, tree->data.p);
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_object_new_specific);
        x86_call_code (s->code, 0);
-       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, sizeof (gpointer));
+       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 4);
 
        x86_pop_reg (s->code, X86_EDX);
        x86_pop_reg (s->code, X86_ECX);
@@ -1725,52 +1761,34 @@ reg: OBJADDR (reg) {
                x86_mov_reg_reg (s->code, tree->reg1, tree->left->reg1, 4);
 }
 
-stmt: FREE (reg) {
-       x86_push_reg (s->code, X86_EAX);
-       x86_push_reg (s->code, X86_ECX);
-       x86_push_reg (s->code, X86_EDX);
+reg: VTADDR (ADDR_L) {
+       int offset = VARINFO (s, tree->left->data.i).offset;
 
-       x86_push_reg (s->code, tree->left->reg1);
+       x86_lea_membase (s->code, tree->reg1, X86_EBP, offset);
+}
 
+stmt: FREE (reg) {
+       x86_push_reg (s->code, tree->left->reg1);
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, g_free);
        x86_call_code (s->code, 0);
        x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 4);
-       
-       x86_pop_reg (s->code, X86_EDX);
-       x86_pop_reg (s->code, X86_ECX);
-       x86_pop_reg (s->code, X86_EAX);
 }
 
-stmt: PROC3 (reg, CPSRC (reg, reg)) {
-       int dest_reg = tree->left->reg1;
-       int source_reg = tree->right->left->reg1;
-       int size_reg = tree->right->right->reg1;
-
-       x86_push_reg (s->code, X86_EAX);
-       x86_push_reg (s->code, X86_ECX);
-       x86_push_reg (s->code, X86_EDX);
-
-       x86_push_reg (s->code, size_reg);
-       x86_push_reg (s->code, source_reg);
-       x86_push_reg (s->code, dest_reg);
-
-       switch (tree->data.i) {
-       case MONO_MARSHAL_CONV_STR_BYVALSTR:
-               mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_string_to_byvalstr);
-               break;
-       case MONO_MARSHAL_CONV_STR_BYVALWSTR:
-               mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, mono_string_to_byvalwstr);
-               break;
-       default:
-               g_assert_not_reached ();
-       }
+stmt: PROC2 (reg, reg) {
+       x86_push_reg (s->code, tree->right->reg1);
+       x86_push_reg (s->code, tree->left->reg1);
+       mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->data.p);
+       x86_call_code (s->code, 0);
+       x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 8);
+}
 
+stmt: PROC3 (reg, CPSRC (reg, reg)) {
+       x86_push_reg (s->code, tree->right->right->reg1);
+       x86_push_reg (s->code, tree->right->left->reg1);
+       x86_push_reg (s->code, tree->left->reg1);
+       mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->data.p);
        x86_call_code (s->code, 0);
        x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, 12);
-
-       x86_pop_reg (s->code, X86_EDX);
-       x86_pop_reg (s->code, X86_ECX);
-       x86_pop_reg (s->code, X86_EAX);
 }
 
 reg: FUNC1 (reg) {
@@ -1796,9 +1814,9 @@ reg: FUNC1 (reg) {
 
 reg: LOCALLOC (reg) {
        int offset = 0;
-       /* size must be aligned to 4 bytes */
-       x86_alu_reg_imm (s->code, X86_ADD, tree->left->reg1, 3);
-       x86_alu_reg_imm (s->code, X86_AND, tree->left->reg1, ~3);
+       /* size must be aligned to MONO_FRAME_ALIGNMENT bytes */
+       x86_alu_reg_imm (s->code, X86_ADD, tree->left->reg1, MONO_FRAME_ALIGNMENT - 1);
+       x86_alu_reg_imm (s->code, X86_AND, tree->left->reg1, ~(MONO_FRAME_ALIGNMENT - 1));
 
        /* allocate space on stack */
        x86_alu_reg_reg (s->code, X86_SUB, X86_ESP, tree->left->reg1);
@@ -1990,7 +2008,8 @@ reg: ISINST (reg) {
 stmt: INITOBJ (reg) {
        int i, j;
 
-       i = tree->data.i;
+       if (!(i = tree->data.i))
+               return;
 
        if (i == 1 || i == 2 || i == 4) {
                x86_mov_membase_imm (s->code, tree->left->reg1, 0, 0, i);
@@ -2619,9 +2638,11 @@ stmt: RET_VOID {
        } 
 }
 
-
 stmt: ARG_I4 (LDIND_I4 (addr)) {
        MBTree *at = tree->left->left;
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
 
        switch (at->data.ainfo.amode) {
 
@@ -2646,6 +2667,9 @@ stmt: ARG_I4 (LDIND_I4 (addr)) {
 
 stmt: ARG_I4 (LDIND_I4 (ADDR_L)) {
        int treg = VARINFO (s, tree->left->left->data.i).reg;
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_reg (s->code, treg);
 } cost {
        MBCOND ((VARINFO (data, tree->left->left->data.i).reg >= 0));
@@ -2653,15 +2677,23 @@ stmt: ARG_I4 (LDIND_I4 (ADDR_L)) {
 }
 
 stmt: ARG_I4 (reg) {
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_reg (s->code, tree->left->reg1);
-       PRINT_REG ("ARG_I4",  tree->left->reg1);
 }
 
 stmt: ARG_I4 (ADDR_G) {
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_imm (s->code, tree->left->data.p);
 }
 
 stmt: ARG_I4 (CONST_I4) "MB_USE_OPT1(0)" {
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_imm (s->code, tree->left->data.i);
 }
 
@@ -2720,24 +2752,11 @@ reg: CALL_I4 (this, reg) {
        if (lreg == treg || rreg == treg) 
                mono_assert_not_reached ();
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_call_reg (s->code, rreg);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-       
-       PRINT_REG ("CALL_I4", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
 }
@@ -2749,25 +2768,12 @@ reg: CALL_I4 (this, ADDR_G) {
        if (lreg == treg) 
                treg = X86_EDX;
        
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->right->data.p);
        x86_call_code (s->code, 0);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-
-       PRINT_REG ("CALL_I4", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
 }
@@ -2802,16 +2808,7 @@ reg: CALL_I4 (this, INTF_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_mov_reg_membase (s->code, lreg, lreg, 
@@ -2819,10 +2816,7 @@ reg: CALL_I4 (this, INTF_ADDR) {
        x86_mov_reg_membase (s->code, lreg, lreg, tree->right->data.m->klass->interface_id << 2, 4);
        x86_call_virtual (s->code, lreg, tree->right->data.m->slot << 2);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-
-       PRINT_REG ("CALL_I4(INTERFACE)", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
 }
@@ -2877,25 +2871,13 @@ reg: CALL_I4 (this, VFUNC_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);        
        x86_call_virtual (s->code, lreg, 
                G_STRUCT_OFFSET (MonoVTable, vtable) + (tree->right->data.m->slot << 2));
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-
-       PRINT_REG ("CALL_I4(VIRTUAL)", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
 }
@@ -2907,48 +2889,31 @@ stmt: CALL_VOID (this, ADDR_G) {
        if (lreg == treg) 
                treg = X86_EDX;
        
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->right->data.p);
        x86_call_code (s->code, 0);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 stmt: CALL_VOID (this, reg) {
-       int lreg = tree->left->reg1;
        int treg = X86_EAX;
+       int lreg = tree->left->reg1;
+       int rreg = tree->right->reg1;
 
-       if (lreg == treg) 
+       if (lreg == treg || rreg == treg
                treg = X86_EDX;
+       if (lreg == treg || rreg == treg) 
+               treg = X86_ECX;
+       if (lreg == treg || rreg == treg) 
+               mono_assert_not_reached ();
        
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_call_reg (s->code, tree->right->reg1);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 stmt: CALL_VOID (this, INTF_ADDR) {
@@ -2958,16 +2923,7 @@ stmt: CALL_VOID (this, INTF_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_mov_reg_membase (s->code, lreg, lreg, 
@@ -2975,8 +2931,7 @@ stmt: CALL_VOID (this, INTF_ADDR) {
        x86_mov_reg_membase (s->code, lreg, lreg, tree->right->data.m->klass->interface_id << 2, 4);
        x86_call_virtual (s->code, lreg, tree->right->data.m->slot << 2);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 stmt: CALL_VOID (this, VFUNC_ADDR) {
@@ -2986,23 +2941,13 @@ stmt: CALL_VOID (this, VFUNC_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_call_virtual (s->code, lreg, 
                G_STRUCT_OFFSET (MonoVTable, vtable) + (tree->right->data.m->slot << 2));
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 stmt: SWITCH (reg) {
@@ -3671,24 +3616,11 @@ lreg: CALL_I8 (this, reg) {
        if (lreg == treg || rreg == treg) 
                mono_assert_not_reached ();
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_call_reg (s->code, rreg);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-       
-       PRINT_REG ("CALL_I8", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
        mono_assert (tree->reg2 == X86_EDX);
@@ -3701,23 +3633,12 @@ lreg: CALL_I8 (this, ADDR_G) {
        if (lreg == treg) 
                treg = X86_EDX;
        
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->right->data.p);
        x86_call_code (s->code, 0);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
        mono_assert (tree->reg2 == X86_EDX);
@@ -3730,26 +3651,13 @@ lreg: CALL_I8 (this, VFUNC_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_call_virtual (s->code, lreg, 
                G_STRUCT_OFFSET (MonoVTable, vtable) + (tree->right->data.m->slot << 2));
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-
-       PRINT_REG ("CALL0_I8(VIRTUAL)", tree->reg1);
-       PRINT_REG ("CALL1_I8(VIRTUAL)", tree->reg2);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
        mono_assert (tree->reg2 == X86_EDX);
@@ -3762,16 +3670,7 @@ lreg: CALL_I8 (this, INTF_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_mov_reg_membase (s->code, lreg, lreg, 
@@ -3779,10 +3678,7 @@ lreg: CALL_I8 (this, INTF_ADDR) {
        x86_mov_reg_membase (s->code, lreg, lreg, tree->right->data.m->klass->interface_id << 2, 4);
        x86_call_virtual (s->code, lreg, tree->right->data.m->slot << 2);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
-
-       PRINT_REG ("CALL_I8(INTERFACE)", tree->reg1);
+       X86_CALL_END;
 
        mono_assert (tree->reg1 == X86_EAX);
        mono_assert (tree->reg2 == X86_EDX);
@@ -3811,6 +3707,9 @@ stmt: RET (lreg) {
 
 
 stmt: ARG_I8 (lreg) {
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_reg (s->code, tree->left->reg2);
        x86_push_reg (s->code, tree->left->reg1);
 }
@@ -4241,7 +4140,7 @@ freg: CONV_R_UN (lreg) {
        x86_fst80_membase (s->code, X86_ESP, 0);
 
        /* test if lreg is negative */
-       x86_test_reg_reg (s->code, tree->left->reg1, tree->left->reg1);
+       x86_test_reg_reg (s->code, tree->left->reg2, tree->left->reg2);
        br [0] = s->code; x86_branch8 (s->code, X86_CC_GEZ, 0, TRUE);
 
        /* add correction constant mn */
@@ -4249,7 +4148,7 @@ freg: CONV_R_UN (lreg) {
        x86_fld80_membase (s->code, X86_ESP, 0);
        x86_fp_op_reg (s->code, X86_FADD, 1, TRUE);
        x86_fst80_membase (s->code, X86_ESP, 0);
-
+       //x86_breakpoint (s->code);
        x86_patch (br [0], s->code);
 
        x86_fld80_membase (s->code, X86_ESP, 0);
@@ -4525,12 +4424,16 @@ stmt: REMOTE_STIND_R8 (reg, freg) {
 }
 
 stmt: ARG_R4 (freg) {
-       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, 4);
+       int pad = tree->data.arg_info.pad;
+
+       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, 4 + pad);
        x86_fst_membase (s->code, X86_ESP, 0, FALSE, TRUE);
 }
 
 stmt: ARG_R8 (freg) {
-       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, 8);
+       int pad = tree->data.arg_info.pad;
+
+       x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, 8 + pad);
        x86_fst_membase (s->code, X86_ESP, 0, TRUE, TRUE);
 }
 
@@ -4606,22 +4509,11 @@ freg: CALL_R8 (this, reg) {
        if (lreg == treg || rreg == treg) 
                mono_assert_not_reached ();
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_call_reg (s->code, rreg);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);   
+       X86_CALL_END;
 }
 
 freg: CALL_R8 (this, ADDR_G) {
@@ -4631,23 +4523,12 @@ freg: CALL_R8 (this, ADDR_G) {
        if (lreg == treg) 
                treg = X86_EDX;
        
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-               x86_alu_membase_imm (s->code, X86_CMP, lreg, 0, 0);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, tree->right->data.p);
        x86_call_code (s->code, 0);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 freg: CALL_R8 (this, INTF_ADDR) {
@@ -4657,16 +4538,7 @@ freg: CALL_R8 (this, INTF_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_mov_reg_membase (s->code, lreg, lreg, 
@@ -4674,8 +4546,7 @@ freg: CALL_R8 (this, INTF_ADDR) {
        x86_mov_reg_membase (s->code, lreg, lreg, tree->right->data.m->klass->interface_id << 2, 4);
        x86_call_virtual (s->code, lreg, tree->right->data.m->slot << 2);
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 freg: CALL_R8 (this, VFUNC_ADDR) {
@@ -4685,23 +4556,13 @@ freg: CALL_R8 (this, VFUNC_ADDR) {
        if (lreg == treg) 
                treg = X86_EDX;
 
-       if (tree->left->op != MB_TERM_NOP) {
-               mono_assert (lreg >= 0);
-               x86_push_reg (s->code, lreg);
-       }
-
-       if (tree->data.ci.vtype_num) {
-               int offset = VARINFO (s, tree->data.ci.vtype_num).offset;
-               x86_lea_membase (s->code, treg, X86_EBP, offset);
-               x86_push_reg (s->code, treg);
-       }
+       X86_CALL_BEGIN;
 
        x86_mov_reg_membase (s->code, lreg, lreg, 0, 4);
        x86_call_virtual (s->code, lreg, 
                G_STRUCT_OFFSET (MonoVTable, vtable) + (tree->right->data.m->slot << 2));
 
-       if (tree->data.ci.args_size)
-               x86_alu_reg_imm (s->code, X86_ADD, X86_ESP, tree->data.ci.args_size);
+       X86_CALL_END;
 }
 
 stmt: RET (freg) {
@@ -4788,16 +4649,21 @@ stmt: REMOTE_STIND_OBJ (reg, reg) {
 }
 
 stmt: ARG_OBJ (CONST_I4) {
+       int pad = tree->data.arg_info.pad;
+
+       X86_ARG_PAD (pad);
        x86_push_imm (s->code, tree->left->data.i);     
 }
 
 stmt: ARG_OBJ (reg) {
-       int size = tree->data.i;
+       int size = tree->data.arg_info.size;
+       int pad = tree->data.arg_info.pad;
        int sa;
        
-       mono_assert (size > 0);
-       sa = size + 3;
-       sa &= ~3;
+       if (!size) 
+               return;
+
+       sa = size + pad;
 
        /* reserve space for the argument */
        x86_alu_reg_imm (s->code, X86_SUB, X86_ESP, sa);
@@ -4827,7 +4693,6 @@ stmt: RET_OBJ (reg) {
        x86_push_reg (s->code, tree->left->reg1);
        x86_push_membase (s->code, X86_EBP, 8);
 
-
        mono_add_jump_info (s, s->code, MONO_JUMP_INFO_ABS, MEMCOPY);
        x86_call_code (s->code, 0);