Merged subtype and current trunk.
[cacao.git] / src / vm / jit / x86_64 / emit.c
index b56c0e33c4ff89e2a6c13b0f110e67886e63351b..9c777fb54a558b0545c5177ea11a2fdea6107ae3 100644 (file)
@@ -1,9 +1,7 @@
 /* src/vm/jit/x86_64/emit.c - x86_64 code emitter functions
 
-   Copyright (C) 1996-2005, 2006 R. Grafl, A. Krall, C. Kruegel,
-   C. Oates, R. Obermaisser, M. Platter, M. Probst, S. Ring,
-   E. Steiner, C. Thalinger, D. Thuernbeck, P. Tomsich, C. Ullrich,
-   J. Wenninger, Institut f. Computersprachen - TU Wien
+   Copyright (C) 1996-2005, 2006, 2007, 2008
+   CACAOVM - Verein zur Foerderung der freien virtuellen Maschine CACAO
 
    This file is part of CACAO.
 
    Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
    02110-1301, USA.
 
-   Contact: cacao@cacaojvm.org
-
-   Authors: Christian Thalinger
-
-   Changes:
-
-   $Id: emit.c 4943 2006-05-23 08:51:33Z twisti $
-
 */
 
+#include "config.h"
+
+#include <assert.h>
 
 #include "vm/types.h"
 
 #include "md-abi.h"
 
-#include "vm/jit/codegen-common.h"
-#include "vm/jit/emit.h"
-#include "vm/jit/jit.h"
 #include "vm/jit/x86_64/codegen.h"
-#include "vm/jit/x86_64/md-emit.h"
+#include "vm/jit/x86_64/emit.h"
 
+#include "mm/memory.h"
 
-/* code generation functions **************************************************/
+#include "threads/lock.hpp"
 
-/* emit_load_s1 ****************************************************************
+#include "vm/options.h"
 
-   Emits a possible load of the first source operand.
+#include "vm/jit/abi.h"
+#include "vm/jit/abi-asm.h"
+#include "vm/jit/asmpart.h"
+#include "vm/jit/codegen-common.hpp"
+#include "vm/jit/emit-common.hpp"
+#include "vm/jit/jit.hpp"
+#include "vm/jit/patcher-common.hpp"
+#include "vm/jit/replace.hpp"
+#include "vm/jit/trace.hpp"
+#include "vm/jit/trap.h"
+
+
+/* emit_load *******************************************************************
+
+   Emits a possible load of an operand.
 
 *******************************************************************************/
 
-s4 emit_load_s1(jitdata *jd, instruction *iptr, stackptr src, s4 tempreg)
+s4 emit_load(jitdata *jd, instruction *iptr, varinfo *src, s4 tempreg)
 {
        codegendata  *cd;
        s4            disp;
@@ -62,233 +67,583 @@ s4 emit_load_s1(jitdata *jd, instruction *iptr, stackptr src, s4 tempreg)
 
        cd = jd->cd;
 
-       if (src->flags & INMEMORY) {
+       if (IS_INMEMORY(src->flags)) {
                COUNT_SPILLS;
 
-               disp = src->regoff * 8;
-
-               if (IS_FLT_DBL_TYPE(src->type)) {
-                       if (IS_2_WORD_TYPE(src->type))
-                               M_DLD(tempreg, REG_SP, disp);
-                       else
-                               M_FLD(tempreg, REG_SP, disp);
-
-               } else {
-                       if (IS_INT_TYPE(src->type))
-                               M_ILD(tempreg, REG_SP, disp);
-                       else
-                               M_LLD(tempreg, REG_SP, disp);
+               disp = src->vv.regoff;
+
+               switch (src->type) {
+               case TYPE_INT:
+                       M_ILD(tempreg, REG_SP, disp);
+                       break;
+               case TYPE_LNG:
+               case TYPE_ADR:
+                       M_LLD(tempreg, REG_SP, disp);
+                       break;
+               case TYPE_FLT:
+                       M_FLD(tempreg, REG_SP, disp);
+                       break;
+               case TYPE_DBL:
+                       M_DLD(tempreg, REG_SP, disp);
+                       break;
+               default:
+                       vm_abort("emit_load: unknown type %d", src->type);
                }
 
                reg = tempreg;
-       } else
-               reg = src->regoff;
+       }
+       else
+               reg = src->vv.regoff;
 
        return reg;
 }
 
 
-/* emit_load_s2 ****************************************************************
-
-   Emits a possible load of the second source operand.
+/* emit_store ******************************************************************
 
+   This function generates the code to store the result of an
+   operation back into a spilled pseudo-variable.  If the
+   pseudo-variable has not been spilled in the first place, this
+   function will generate nothing.
+    
 *******************************************************************************/
 
-s4 emit_load_s2(jitdata *jd, instruction *iptr, stackptr src, s4 tempreg)
+void emit_store(jitdata *jd, instruction *iptr, varinfo *dst, s4 d)
 {
        codegendata  *cd;
        s4            disp;
-       s4            reg;
 
        /* get required compiler data */
 
        cd = jd->cd;
 
-       if (src->flags & INMEMORY) {
+       if (IS_INMEMORY(dst->flags)) {
                COUNT_SPILLS;
 
-               disp = src->regoff * 8;
+               disp = dst->vv.regoff;
 
-               if (IS_FLT_DBL_TYPE(src->type)) {
-                       if (IS_2_WORD_TYPE(src->type))
-                               M_DLD(tempreg, REG_SP, disp);
-                       else
-                               M_FLD(tempreg, REG_SP, disp);
+               switch (dst->type) {
+               case TYPE_INT:
+               case TYPE_LNG:
+               case TYPE_ADR:
+                       M_LST(d, REG_SP, disp);
+                       break;
+               case TYPE_FLT:
+                       M_FST(d, REG_SP, disp);
+                       break;
+               case TYPE_DBL:
+                       M_DST(d, REG_SP, disp);
+                       break;
+               default:
+                       vm_abort("emit_store: unknown type %d", dst->type);
+               }
+       }
+}
 
-               } else {
-                       if (IS_INT_TYPE(src->type))
-                               M_ILD(tempreg, REG_SP, disp);
-                       else
-                               M_LLD(tempreg, REG_SP, disp);
+
+/* emit_copy *******************************************************************
+
+   Generates a register/memory to register/memory copy.
+
+*******************************************************************************/
+
+void emit_copy(jitdata *jd, instruction *iptr)
+{
+       codegendata *cd;
+       varinfo     *src;
+       varinfo     *dst;
+       s4           s1, d;
+
+       /* get required compiler data */
+
+       cd = jd->cd;
+
+       /* get source and destination variables */
+
+       src = VAROP(iptr->s1);
+       dst = VAROP(iptr->dst);
+
+       if ((src->vv.regoff != dst->vv.regoff) ||
+               ((src->flags ^ dst->flags) & INMEMORY)) {
+
+               if ((src->type == TYPE_RET) || (dst->type == TYPE_RET)) {
+                       /* emit nothing, as the value won't be used anyway */
+                       return;
                }
 
-               reg = tempreg;
-       } else
-               reg = src->regoff;
+               /* If one of the variables resides in memory, we can eliminate
+                  the register move from/to the temporary register with the
+                  order of getting the destination register and the load. */
 
-       return reg;
+               if (IS_INMEMORY(src->flags)) {
+                       d  = codegen_reg_of_var(iptr->opc, dst, REG_IFTMP);
+                       s1 = emit_load(jd, iptr, src, d);
+               }
+               else {
+                       s1 = emit_load(jd, iptr, src, REG_IFTMP);
+                       d  = codegen_reg_of_var(iptr->opc, dst, s1);
+               }
+
+               if (s1 != d) {
+                       switch (src->type) {
+                       case TYPE_INT:
+                       case TYPE_LNG:
+                       case TYPE_ADR:
+                               M_MOV(s1, d);
+                               break;
+                       case TYPE_FLT:
+                       case TYPE_DBL:
+                               M_FMOV(s1, d);
+                               break;
+                       default:
+                               vm_abort("emit_copy: unknown type %d", src->type);
+                       }
+               }
+
+               emit_store(jd, iptr, dst, d);
+       }
 }
 
 
-/* emit_load_s3 ****************************************************************
+void emit_cmovxx(codegendata *cd, instruction *iptr, s4 s, s4 d)
+{
+#if 0
+       switch (iptr->flags.fields.condition) {
+       case ICMD_IFEQ:
+               M_CMOVEQ(s, d);
+               break;
+       case ICMD_IFNE:
+               M_CMOVNE(s, d);
+               break;
+       case ICMD_IFLT:
+               M_CMOVLT(s, d);
+               break;
+       case ICMD_IFGE:
+               M_CMOVGE(s, d);
+               break;
+       case ICMD_IFGT:
+               M_CMOVGT(s, d);
+               break;
+       case ICMD_IFLE:
+               M_CMOVLE(s, d);
+               break;
+       }
+#endif
+}
+
 
-   Emits a possible load of the third source operand.
+/* emit_branch *****************************************************************
+
+   Emits the code for conditional and unconditional branchs.
 
 *******************************************************************************/
 
-s4 emit_load_s3(jitdata *jd, instruction *iptr, stackptr src, s4 tempreg)
+void emit_branch(codegendata *cd, s4 disp, s4 condition, s4 reg, u4 options)
 {
-       codegendata  *cd;
-       s4            disp;
-       s4            reg;
+       s4 branchdisp;
 
-       /* get required compiler data */
+       /* NOTE: A displacement overflow cannot happen. */
 
-       cd = jd->cd;
+       /* check which branch to generate */
 
-       if (src->flags & INMEMORY) {
-               COUNT_SPILLS;
+       if (condition == BRANCH_UNCONDITIONAL) {
 
-               disp = src->regoff * 8;
+               /* calculate the different displacements */
 
-               if (IS_FLT_DBL_TYPE(src->type)) {
-                       if (IS_2_WORD_TYPE(src->type))
-                               M_DLD(tempreg, REG_SP, disp);
-                       else
-                               M_FLD(tempreg, REG_SP, disp);
+               branchdisp = disp - BRANCH_UNCONDITIONAL_SIZE;
 
-               } else {
-                       if (IS_INT_TYPE(src->type))
-                               M_ILD(tempreg, REG_SP, disp);
-                       else
-                               M_LLD(tempreg, REG_SP, disp);
+               M_JMP_IMM(branchdisp);
+       }
+       else {
+               /* calculate the different displacements */
+
+               branchdisp = disp - BRANCH_CONDITIONAL_SIZE;
+
+               switch (condition) {
+               case BRANCH_EQ:
+                       M_BEQ(branchdisp);
+                       break;
+               case BRANCH_NE:
+                       M_BNE(branchdisp);
+                       break;
+               case BRANCH_LT:
+                       M_BLT(branchdisp);
+                       break;
+               case BRANCH_GE:
+                       M_BGE(branchdisp);
+                       break;
+               case BRANCH_GT:
+                       M_BGT(branchdisp);
+                       break;
+               case BRANCH_LE:
+                       M_BLE(branchdisp);
+                       break;
+               case BRANCH_ULT:
+                       M_BULT(branchdisp);
+                       break;
+               case BRANCH_ULE:
+                       M_BULE(branchdisp);
+                       break;
+               case BRANCH_UGE:
+                       M_BUGE(branchdisp);
+                       break;
+               case BRANCH_UGT:
+                       M_BUGT(branchdisp);
+                       break;
+               default:
+                       vm_abort("emit_branch: unknown condition %d", condition);
                }
+       }
+}
 
-               reg = tempreg;
-       } else
-               reg = src->regoff;
 
-       return reg;
+/* emit_arithmetic_check *******************************************************
+
+   Emit an ArithmeticException check.
+
+*******************************************************************************/
+
+void emit_arithmetic_check(codegendata *cd, instruction *iptr, s4 reg)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+               M_TEST(reg);
+               M_BNE(8);
+               M_ALD_MEM(reg, TRAP_ArithmeticException);
+       }
 }
 
 
-/* emit_store ******************************************************************
+/* emit_arrayindexoutofbounds_check ********************************************
+
+   Emit a ArrayIndexOutOfBoundsException check.
+
+*******************************************************************************/
+
+void emit_arrayindexoutofbounds_check(codegendata *cd, instruction *iptr, s4 s1, s4 s2)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+        M_ILD(REG_ITMP3, s1, OFFSET(java_array_t, size));
+        M_ICMP(REG_ITMP3, s2);
+               M_BULT(8);
+               M_ALD_MEM(s2, TRAP_ArrayIndexOutOfBoundsException);
+       }
+}
+
+
+/* emit_arraystore_check *******************************************************
+
+   Emit an ArrayStoreException check.
+
+*******************************************************************************/
+
+void emit_arraystore_check(codegendata *cd, instruction *iptr)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+               M_TEST(REG_RESULT);
+               M_BNE(8);
+               M_ALD_MEM(REG_RESULT, TRAP_ArrayStoreException);
+       }
+}
+
+
+/* emit_classcast_check ********************************************************
+
+   Emit a ClassCastException check.
+
+*******************************************************************************/
+
+void emit_classcast_check(codegendata *cd, instruction *iptr, s4 condition, s4 reg, s4 s1)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+               switch (condition) {
+               case BRANCH_LE:
+                       M_BGT(8);
+                       break;
+               case BRANCH_GE:
+                       M_BLT(8);
+                       break;
+               case BRANCH_EQ:
+                       M_BNE(8);
+                       break;
+               case BRANCH_NE:
+                       M_BEQ(8);
+                       break;
+               case BRANCH_UGT:
+                       M_BULE(8);
+                       break;
+               default:
+                       vm_abort("emit_classcast_check: unknown condition %d", condition);
+               }
+               M_ALD_MEM(s1, TRAP_ClassCastException);
+       }
+}
+
+
+/* emit_nullpointer_check ******************************************************
+
+   Emit a NullPointerException check.
+
+*******************************************************************************/
+
+void emit_nullpointer_check(codegendata *cd, instruction *iptr, s4 reg)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+               M_TEST(reg);
+               M_BNE(8);
+               M_ALD_MEM(reg, TRAP_NullPointerException);
+       }
+}
+
+
+/* emit_exception_check ********************************************************
+
+   Emit an Exception check.
+
+*******************************************************************************/
+
+void emit_exception_check(codegendata *cd, instruction *iptr)
+{
+       if (INSTRUCTION_MUST_CHECK(iptr)) {
+               M_TEST(REG_RESULT);
+               M_BNE(8);
+               M_ALD_MEM(REG_RESULT, TRAP_CHECK_EXCEPTION);
+       }
+}
+
+
+/* emit_trap_compiler **********************************************************
+
+   Emit a trap instruction which calls the JIT compiler.
+
+*******************************************************************************/
+
+void emit_trap_compiler(codegendata *cd)
+{
+       M_ALD_MEM(REG_METHODPTR, TRAP_COMPILER);
+}
+
+
+/* emit_trap *******************************************************************
+
+   Emit a trap instruction and return the original machine code.
+
+*******************************************************************************/
+
+uint32_t emit_trap(codegendata *cd)
+{
+       uint16_t mcode;
+
+       /* Get machine code which is patched back in later. The trap is 2
+          bytes long. */
+
+       mcode = *((uint16_t *) cd->mcodeptr);
+
+       /* XXX This needs to be change to INT3 when the debugging problems
+          with gdb are resolved. */
+
+       M_UD2;
+
+       return mcode;
+}
+
+
+/* emit_verbosecall_enter ******************************************************
+
+   Generates the code for the call trace.
 
-   This function generates the code to store the result of an
-   operation back into a spilled pseudo-variable.  If the
-   pseudo-variable has not been spilled in the first place, this
-   function will generate nothing.
-    
 *******************************************************************************/
 
-void emit_store(jitdata *jd, instruction *iptr, stackptr dst, s4 d)
+#if !defined(NDEBUG)
+void emit_verbosecall_enter(jitdata *jd)
 {
+       methodinfo   *m;
+       codeinfo     *code;
        codegendata  *cd;
        registerdata *rd;
-       s4            disp;
-       s4            s;
-       u2            opcode;
+       methoddesc   *md;
+       s4            stackframesize;
+       s4            i, s;
 
        /* get required compiler data */
 
-       cd = jd->cd;
-       rd = jd->rd;
+       m    = jd->m;
+       code = jd->code;
+       cd   = jd->cd;
+       rd   = jd->rd;
 
-       /* do we have to generate a conditional move? */
+       md = m->parseddesc;
 
-       if ((iptr != NULL) && (iptr->opc & ICMD_CONDITION_MASK)) {
-               /* the passed register d is actually the source register */
+       /* mark trace code */
 
-               s = d;
+       M_NOP;
 
-               /* Only pass the opcode to codegen_reg_of_var to get the real
-                  destination register. */
+       /* keep 16-byte stack alignment */
 
-               opcode = iptr->opc & ICMD_OPCODE_MASK;
+       stackframesize = md->paramcount + ARG_CNT + TMP_CNT;
+       ALIGN_2(stackframesize);
 
-               /* get the real destination register */
+       M_LSUB_IMM(stackframesize * 8, REG_SP);
 
-               d = codegen_reg_of_var(rd, opcode, dst, REG_ITMP1);
+       /* save argument registers */
 
-               /* and emit the conditional move */
+       for (i = 0; i < md->paramcount; i++) {
+               if (!md->params[i].inmemory) {
+                       s = md->params[i].regoff;
 
-               emit_cmovxx(cd, iptr, s, d);
+                       switch (md->paramtypes[i].type) {
+                       case TYPE_ADR:
+                       case TYPE_INT:
+                       case TYPE_LNG:
+                               M_LST(s, REG_SP, i * 8);
+                               break;
+                       case TYPE_FLT:
+                       case TYPE_DBL:
+                               M_DST(s, REG_SP, i * 8);
+                               break;
+                       }
+               }
        }
 
-       if (dst->flags & INMEMORY) {
-               COUNT_SPILLS;
+       /* save all argument and temporary registers for leaf methods */
 
-               disp = dst->regoff * 8;
+       if (code_is_leafmethod(code)) {
+               for (i = 0; i < INT_ARG_CNT; i++)
+                       M_LST(abi_registers_integer_argument[i], REG_SP, (md->paramcount + i) * 8);
 
-               if (IS_FLT_DBL_TYPE(dst->type)) {
-                       if (IS_2_WORD_TYPE(dst->type))
-                               M_DST(d, REG_SP, disp);
-                       else
-                               M_FST(d, REG_SP, disp);
+               for (i = 0; i < FLT_ARG_CNT; i++)
+                       M_DST(abi_registers_float_argument[i], REG_SP, (md->paramcount + INT_ARG_CNT + i) * 8);
 
-               } else
-                       M_LST(d, REG_SP, disp);
+               for (i = 0; i < INT_TMP_CNT; i++)
+                       M_LST(rd->tmpintregs[i], REG_SP, (md->paramcount + ARG_CNT + i) * 8);
+
+               for (i = 0; i < FLT_TMP_CNT; i++)
+                       M_DST(rd->tmpfltregs[i], REG_SP, (md->paramcount + ARG_CNT + INT_TMP_CNT + i) * 8);
+       }
+
+       M_MOV_IMM(m, REG_A0);
+       M_MOV(REG_SP, REG_A1);
+       M_MOV(REG_SP, REG_A2);
+       M_AADD_IMM((stackframesize + cd->stackframesize + 1) * 8, REG_A2);
+       M_MOV_IMM(trace_java_call_enter, REG_ITMP1);
+       M_CALL(REG_ITMP1);
+
+       /* restore argument registers */
+
+       for (i = 0; i < md->paramcount; i++) {
+               if (!md->params[i].inmemory) {
+                       s = md->params[i].regoff;
+
+                       switch (md->paramtypes[i].type) {
+                       case TYPE_ADR:
+                       case TYPE_INT:
+                       case TYPE_LNG:
+                               M_LLD(s, REG_SP, i * 8);
+                               break;
+                       case TYPE_FLT:
+                       case TYPE_DBL:
+                               M_DLD(s, REG_SP, i * 8);
+                               break;
+                       }
+               }
+       }
+
+
+       /* restore all argument and temporary registers for leaf methods */
+
+       if (code_is_leafmethod(code)) {
+               for (i = 0; i < INT_ARG_CNT; i++)
+                       M_LLD(abi_registers_integer_argument[i], REG_SP, (md->paramcount + i) * 8);
+
+               for (i = 0; i < FLT_ARG_CNT; i++)
+                       M_DLD(abi_registers_float_argument[i], REG_SP, (md->paramcount + INT_ARG_CNT + i) * 8);
+
+               for (i = 0; i < INT_TMP_CNT; i++)
+                       M_LLD(rd->tmpintregs[i], REG_SP, (md->paramcount + ARG_CNT + i) * 8);
+
+               for (i = 0; i < FLT_TMP_CNT; i++)
+                       M_DLD(rd->tmpfltregs[i], REG_SP, (md->paramcount + ARG_CNT + INT_TMP_CNT + i) * 8);
        }
+
+       M_LADD_IMM(stackframesize * 8, REG_SP);
+
+       /* mark trace code */
+
+       M_NOP;
 }
+#endif /* !defined(NDEBUG) */
 
 
-/* emit_copy *******************************************************************
+/* emit_verbosecall_exit *******************************************************
 
-   XXX
+   Generates the code for the call trace.
 
 *******************************************************************************/
 
-void emit_copy(jitdata *jd, instruction *iptr, stackptr src, stackptr dst)
+#if !defined(NDEBUG)
+void emit_verbosecall_exit(jitdata *jd)
 {
+       methodinfo   *m;
        codegendata  *cd;
        registerdata *rd;
-       s4            s1, d;
+       methoddesc   *md;
 
        /* get required compiler data */
 
+       m  = jd->m;
        cd = jd->cd;
        rd = jd->rd;
 
-       if ((src->regoff != dst->regoff) ||
-               ((src->flags ^ dst->flags) & INMEMORY)) {
-               d = codegen_reg_of_var(rd, iptr->opc, dst, REG_IFTMP);
-               s1 = emit_load_s1(jd, iptr, src, d);
+       md = m->parseddesc;
 
-               if (s1 != d) {
-                       if (IS_FLT_DBL_TYPE(src->type))
-                               M_FMOV(s1, d);
-                       else
-                               M_MOV(s1, d);
-               }
+       /* mark trace code */
 
-               emit_store(jd, iptr, dst, d);
-       }
-}
+       M_NOP;
 
+       /* keep 16-byte stack alignment */
 
-void emit_cmovxx(codegendata *cd, instruction *iptr, s4 s, s4 d)
-{
-       switch ((iptr->opc & ICMD_CONDITION_MASK) >> 8) {
-       case ICMD_IFEQ:
-               M_CMOVEQ(s, d);
-               break;
-       case ICMD_IFNE:
-               M_CMOVNE(s, d);
-               break;
-       case ICMD_IFLT:
-               M_CMOVLT(s, d);
+       M_ASUB_IMM(2 * 8, REG_SP);
+
+       /* save return value */
+
+       switch (md->returntype.type) {
+       case TYPE_ADR:
+       case TYPE_INT:
+       case TYPE_LNG:
+               M_LST(REG_RESULT, REG_SP, 0 * 8);
                break;
-       case ICMD_IFGE:
-               M_CMOVGE(s, d);
+       case TYPE_FLT:
+       case TYPE_DBL:
+               M_DST(REG_FRESULT, REG_SP, 0 * 8);
                break;
-       case ICMD_IFGT:
-               M_CMOVGT(s, d);
+       }
+
+       M_MOV_IMM(m, REG_A0);
+       M_MOV(REG_SP, REG_A1);
+
+       M_MOV_IMM(trace_java_call_exit, REG_ITMP1);
+       M_CALL(REG_ITMP1);
+
+       /* restore return value */
+
+       switch (md->returntype.type) {
+       case TYPE_ADR:
+       case TYPE_INT:
+       case TYPE_LNG:
+               M_LLD(REG_RESULT, REG_SP, 0 * 8);
                break;
-       case ICMD_IFLE:
-               M_CMOVLE(s, d);
+       case TYPE_FLT:
+       case TYPE_DBL:
+               M_DLD(REG_FRESULT, REG_SP, 0 * 8);
                break;
        }
+
+       M_AADD_IMM(2 * 8, REG_SP);
+
+       /* mark trace code */
+
+       M_NOP;
 }
+#endif /* !defined(NDEBUG) */
 
 
 /* code generation functions **************************************************/
@@ -331,65 +686,114 @@ static void emit_membase(codegendata *cd, s4 basereg, s4 disp, s4 dreg)
 }
 
 
-void emit_ishift(codegendata *cd, s4 shift_op, stackptr src, instruction *iptr)
+static void emit_membase32(codegendata *cd, s4 basereg, s4 disp, s4 dreg)
 {
-       s4 s1 = src->prev->regoff;
-       s4 s2 = src->regoff;
-       s4 d = iptr->dst->regoff;
-       s4 d_old;
+       if ((basereg == REG_SP) || (basereg == R12)) {
+               emit_address_byte(2, dreg, REG_SP);
+               emit_address_byte(0, REG_SP, REG_SP);
+               emit_imm32(disp);
+       }
+       else {
+               emit_address_byte(2, dreg, basereg);
+               emit_imm32(disp);
+       }
+}
+
+
+static void emit_memindex(codegendata *cd, s4 reg, s4 disp, s4 basereg, s4 indexreg, s4 scale)
+{
+       if (basereg == -1) {
+               emit_address_byte(0, reg, 4);
+               emit_address_byte(scale, indexreg, 5);
+               emit_imm32(disp);
+       }
+       else if ((disp == 0) && (basereg != RBP) && (basereg != R13)) {
+               emit_address_byte(0, reg, 4);
+               emit_address_byte(scale, indexreg, basereg);
+       }
+       else if (IS_IMM8(disp)) {
+               emit_address_byte(1, reg, 4);
+               emit_address_byte(scale, indexreg, basereg);
+               emit_imm8(disp);
+       }
+       else {
+               emit_address_byte(2, reg, 4);
+               emit_address_byte(scale, indexreg, basereg);
+               emit_imm32(disp);
+       }
+}
+
+
+void emit_ishift(jitdata *jd, s4 shift_op, instruction *iptr)
+{
+       s4 s1, s2, d, d_old;
+       varinfo *v_s1,*v_s2,*v_dst;
+       codegendata *cd;
+
+       /* get required compiler data */
+
+       cd = jd->cd;
+
+       v_s1  = VAROP(iptr->s1);
+       v_s2  = VAROP(iptr->sx.s23.s2);
+       v_dst = VAROP(iptr->dst);
+
+       s1 = v_s1->vv.regoff;
+       s2 = v_s2->vv.regoff;
+       d  = v_dst->vv.regoff;
 
        M_INTMOVE(RCX, REG_ITMP1);                                    /* save RCX */
 
-       if (iptr->dst->flags & INMEMORY) {
-               if ((src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+       if (IS_INMEMORY(v_dst->flags)) {
+               if (IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        if (s1 == d) {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               emit_shiftl_membase(cd, shift_op, REG_SP, d * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               emit_shiftl_membase(cd, shift_op, REG_SP, d);
 
                        } else {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               M_ILD(REG_ITMP2, REG_SP, s1 * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               M_ILD(REG_ITMP2, REG_SP, s1);
                                emit_shiftl_reg(cd, shift_op, REG_ITMP2);
-                               M_IST(REG_ITMP2, REG_SP, d * 8);
+                               M_IST(REG_ITMP2, REG_SP, d);
                        }
 
-               } else if ((src->flags & INMEMORY) && !(src->prev->flags & INMEMORY)) {
+               } else if (IS_INMEMORY(v_s2->flags) && !IS_INMEMORY(v_s1->flags)) {
                        /* s1 may be equal to RCX */
                        if (s1 == RCX) {
                                if (s2 == d) {
-                                       M_ILD(REG_ITMP1, REG_SP, s2 * 8);
-                                       M_IST(s1, REG_SP, d * 8);
+                                       M_ILD(REG_ITMP1, REG_SP, s2);
+                                       M_IST(s1, REG_SP, d);
                                        M_INTMOVE(REG_ITMP1, RCX);
 
                                } else {
-                                       M_IST(s1, REG_SP, d * 8);
-                                       M_ILD(RCX, REG_SP, s2 * 8);
+                                       M_IST(s1, REG_SP, d);
+                                       M_ILD(RCX, REG_SP, s2);
                                }
 
                        } else {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               M_IST(s1, REG_SP, d * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               M_IST(s1, REG_SP, d);
                        }
 
-                       emit_shiftl_membase(cd, shift_op, REG_SP, d * 8);
+                       emit_shiftl_membase(cd, shift_op, REG_SP, d);
 
-               } else if (!(src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+               } else if (!IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        if (s1 == d) {
                                M_INTMOVE(s2, RCX);
-                               emit_shiftl_membase(cd, shift_op, REG_SP, d * 8);
+                               emit_shiftl_membase(cd, shift_op, REG_SP, d);
 
                        } else {
                                M_INTMOVE(s2, RCX);
-                               M_ILD(REG_ITMP2, REG_SP, s1 * 8);
+                               M_ILD(REG_ITMP2, REG_SP, s1);
                                emit_shiftl_reg(cd, shift_op, REG_ITMP2);
-                               M_IST(REG_ITMP2, REG_SP, d * 8);
+                               M_IST(REG_ITMP2, REG_SP, d);
                        }
 
                } else {
                        /* s1 may be equal to RCX */
-                       M_IST(s1, REG_SP, d * 8);
+                       M_IST(s1, REG_SP, d);
                        M_INTMOVE(s2, RCX);
-                       emit_shiftl_membase(cd, shift_op, REG_SP, d * 8);
+                       emit_shiftl_membase(cd, shift_op, REG_SP, d);
                }
 
                M_INTMOVE(REG_ITMP1, RCX);                             /* restore RCX */
@@ -400,20 +804,20 @@ void emit_ishift(codegendata *cd, s4 shift_op, stackptr src, instruction *iptr)
                        d = REG_ITMP3;
                }
                                        
-               if ((src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
-                       M_ILD(RCX, REG_SP, s2 * 8);
-                       M_ILD(d, REG_SP, s1 * 8);
+               if (IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
+                       M_ILD(RCX, REG_SP, s2);
+                       M_ILD(d, REG_SP, s1);
                        emit_shiftl_reg(cd, shift_op, d);
 
-               } else if ((src->flags & INMEMORY) && !(src->prev->flags & INMEMORY)) {
+               } else if (IS_INMEMORY(v_s2->flags) && !IS_INMEMORY(v_s1->flags)) {
                        /* s1 may be equal to RCX */
                        M_INTMOVE(s1, d);
-                       M_ILD(RCX, REG_SP, s2 * 8);
+                       M_ILD(RCX, REG_SP, s2);
                        emit_shiftl_reg(cd, shift_op, d);
 
-               } else if (!(src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+               } else if (!IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        M_INTMOVE(s2, RCX);
-                       M_ILD(d, REG_SP, s1 * 8);
+                       M_ILD(d, REG_SP, s1);
                        emit_shiftl_reg(cd, shift_op, d);
 
                } else {
@@ -447,65 +851,76 @@ void emit_ishift(codegendata *cd, s4 shift_op, stackptr src, instruction *iptr)
 }
 
 
-void emit_lshift(codegendata *cd, s4 shift_op, stackptr src, instruction *iptr)
+void emit_lshift(jitdata *jd, s4 shift_op, instruction *iptr)
 {
-       s4 s1 = src->prev->regoff;
-       s4 s2 = src->regoff;
-       s4 d = iptr->dst->regoff;
-       s4 d_old;
+       s4 s1, s2, d, d_old;
+       varinfo *v_s1,*v_s2,*v_dst;
+       codegendata *cd;
+
+       /* get required compiler data */
+
+       cd = jd->cd;
+
+       v_s1  = VAROP(iptr->s1);
+       v_s2  = VAROP(iptr->sx.s23.s2);
+       v_dst = VAROP(iptr->dst);
+
+       s1 = v_s1->vv.regoff;
+       s2 = v_s2->vv.regoff;
+       d  = v_dst->vv.regoff;
        
        M_INTMOVE(RCX, REG_ITMP1);                                    /* save RCX */
 
-       if (iptr->dst->flags & INMEMORY) {
-               if ((src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+       if (IS_INMEMORY(v_dst->flags)) {
+               if (IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        if (s1 == d) {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               emit_shift_membase(cd, shift_op, REG_SP, d * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               emit_shift_membase(cd, shift_op, REG_SP, d);
 
                        } else {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               M_LLD(REG_ITMP2, REG_SP, s1 * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               M_LLD(REG_ITMP2, REG_SP, s1);
                                emit_shift_reg(cd, shift_op, REG_ITMP2);
-                               M_LST(REG_ITMP2, REG_SP, d * 8);
+                               M_LST(REG_ITMP2, REG_SP, d);
                        }
 
-               } else if ((src->flags & INMEMORY) && !(src->prev->flags & INMEMORY)) {
+               } else if (IS_INMEMORY(v_s2->flags) && !IS_INMEMORY(v_s1->flags)) {
                        /* s1 may be equal to RCX */
                        if (s1 == RCX) {
                                if (s2 == d) {
-                                       M_ILD(REG_ITMP1, REG_SP, s2 * 8);
-                                       M_LST(s1, REG_SP, d * 8);
+                                       M_ILD(REG_ITMP1, REG_SP, s2);
+                                       M_LST(s1, REG_SP, d);
                                        M_INTMOVE(REG_ITMP1, RCX);
 
                                } else {
-                                       M_LST(s1, REG_SP, d * 8);
-                                       M_ILD(RCX, REG_SP, s2 * 8);
+                                       M_LST(s1, REG_SP, d);
+                                       M_ILD(RCX, REG_SP, s2);
                                }
 
                        } else {
-                               M_ILD(RCX, REG_SP, s2 * 8);
-                               M_LST(s1, REG_SP, d * 8);
+                               M_ILD(RCX, REG_SP, s2);
+                               M_LST(s1, REG_SP, d);
                        }
 
-                       emit_shift_membase(cd, shift_op, REG_SP, d * 8);
+                       emit_shift_membase(cd, shift_op, REG_SP, d);
 
-               } else if (!(src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+               } else if (!IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        if (s1 == d) {
                                M_INTMOVE(s2, RCX);
-                               emit_shift_membase(cd, shift_op, REG_SP, d * 8);
+                               emit_shift_membase(cd, shift_op, REG_SP, d);
 
                        } else {
                                M_INTMOVE(s2, RCX);
-                               M_LLD(REG_ITMP2, REG_SP, s1 * 8);
+                               M_LLD(REG_ITMP2, REG_SP, s1);
                                emit_shift_reg(cd, shift_op, REG_ITMP2);
-                               M_LST(REG_ITMP2, REG_SP, d * 8);
+                               M_LST(REG_ITMP2, REG_SP, d);
                        }
 
                } else {
                        /* s1 may be equal to RCX */
-                       M_LST(s1, REG_SP, d * 8);
+                       M_LST(s1, REG_SP, d);
                        M_INTMOVE(s2, RCX);
-                       emit_shift_membase(cd, shift_op, REG_SP, d * 8);
+                       emit_shift_membase(cd, shift_op, REG_SP, d);
                }
 
                M_INTMOVE(REG_ITMP1, RCX);                             /* restore RCX */
@@ -516,20 +931,20 @@ void emit_lshift(codegendata *cd, s4 shift_op, stackptr src, instruction *iptr)
                        d = REG_ITMP3;
                }
 
-               if ((src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
-                       M_ILD(RCX, REG_SP, s2 * 8);
-                       M_LLD(d, REG_SP, s1 * 8);
+               if (IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
+                       M_ILD(RCX, REG_SP, s2);
+                       M_LLD(d, REG_SP, s1);
                        emit_shift_reg(cd, shift_op, d);
 
-               } else if ((src->flags & INMEMORY) && !(src->prev->flags & INMEMORY)) {
+               } else if (IS_INMEMORY(v_s2->flags) && !IS_INMEMORY(v_s1->flags)) {
                        /* s1 may be equal to RCX */
                        M_INTMOVE(s1, d);
-                       M_ILD(RCX, REG_SP, s2 * 8);
+                       M_ILD(RCX, REG_SP, s2);
                        emit_shift_reg(cd, shift_op, d);
 
-               } else if (!(src->flags & INMEMORY) && (src->prev->flags & INMEMORY)) {
+               } else if (!IS_INMEMORY(v_s2->flags) && IS_INMEMORY(v_s1->flags)) {
                        M_INTMOVE(s2, RCX);
-                       M_LLD(d, REG_SP, s1 * 8);
+                       M_LLD(d, REG_SP, s1);
                        emit_shift_reg(cd, shift_op, d);
 
                } else {
@@ -610,7 +1025,7 @@ void emit_mov_membase_reg(codegendata *cd, s8 basereg, s8 disp, s8 reg) {
 void emit_mov_membase32_reg(codegendata *cd, s8 basereg, s8 disp, s8 reg) {
        emit_rex(1,(reg),0,(basereg));
        *(cd->mcodeptr++) = 0x8b;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
@@ -628,7 +1043,7 @@ void emit_movl_membase32_reg(codegendata *cd, s8 basereg, s8 disp, s8 reg)
 {
        emit_byte_rex((reg),0,(basereg));
        *(cd->mcodeptr++) = 0x8b;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
@@ -642,7 +1057,7 @@ void emit_mov_reg_membase(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
 void emit_mov_reg_membase32(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
        emit_rex(1,(reg),0,(basereg));
        *(cd->mcodeptr++) = 0x89;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
@@ -658,35 +1073,35 @@ void emit_movl_reg_membase(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
 void emit_movl_reg_membase32(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
        emit_byte_rex((reg),0,(basereg));
        *(cd->mcodeptr++) = 0x89;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
 void emit_mov_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg, s8 scale, s8 reg) {
        emit_rex(1,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x8b;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
 void emit_movl_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg, s8 scale, s8 reg) {
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x8b;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
 void emit_mov_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 indexreg, s8 scale) {
        emit_rex(1,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x89;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
 void emit_movl_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 indexreg, s8 scale) {
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x89;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -694,14 +1109,14 @@ void emit_movw_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 ind
        *(cd->mcodeptr++) = 0x66;
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x89;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
 void emit_movb_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 indexreg, s8 scale) {
        emit_byte_rex((reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x88;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -716,7 +1131,7 @@ void emit_mov_imm_membase(codegendata *cd, s8 imm, s8 basereg, s8 disp) {
 void emit_mov_imm_membase32(codegendata *cd, s8 imm, s8 basereg, s8 disp) {
        emit_rex(1,0,0,(basereg));
        *(cd->mcodeptr++) = 0xc7;
-       emit_membase32((basereg),(disp),0);
+       emit_membase32(cd, (basereg),(disp),0);
        emit_imm32((imm));
 }
 
@@ -734,7 +1149,7 @@ void emit_movl_imm_membase(codegendata *cd, s8 imm, s8 basereg, s8 disp) {
 void emit_movl_imm_membase32(codegendata *cd, s8 imm, s8 basereg, s8 disp) {
        emit_byte_rex(0,0,(basereg));
        *(cd->mcodeptr++) = 0xc7;
-       emit_membase32((basereg),(disp),0);
+       emit_membase32(cd, (basereg),(disp),0);
        emit_imm32((imm));
 }
 
@@ -768,6 +1183,16 @@ void emit_movslq_reg_reg(codegendata *cd, s8 reg, s8 dreg)
 }
 
 
+void emit_movzbq_reg_reg(codegendata *cd, s8 reg, s8 dreg)
+{
+       emit_rex(1,(dreg),0,(reg));
+       *(cd->mcodeptr++) = 0x0f;
+       *(cd->mcodeptr++) = 0xb6;
+       /* XXX: why do reg and dreg have to be exchanged */
+       emit_reg((dreg),(reg));
+}
+
+
 void emit_movzwq_reg_reg(codegendata *cd, s8 reg, s8 dreg)
 {
        emit_rex(1,(dreg),0,(reg));
@@ -782,7 +1207,7 @@ void emit_movswq_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg,
        emit_rex(1,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0xbf;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -790,7 +1215,7 @@ void emit_movsbq_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg,
        emit_rex(1,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0xbe;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -798,7 +1223,7 @@ void emit_movzwq_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg,
        emit_rex(1,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0xb7;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -806,7 +1231,7 @@ void emit_mov_imm_memindex(codegendata *cd, s4 imm, s4 disp, s4 basereg, s4 inde
 {
        emit_rex(1,0,(indexreg),(basereg));
        *(cd->mcodeptr++) = 0xc7;
-       emit_memindex(0,(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, 0,(disp),(basereg),(indexreg),(scale));
        emit_imm32((imm));
 }
 
@@ -815,7 +1240,7 @@ void emit_movl_imm_memindex(codegendata *cd, s4 imm, s4 disp, s4 basereg, s4 ind
 {
        emit_rex(0,0,(indexreg),(basereg));
        *(cd->mcodeptr++) = 0xc7;
-       emit_memindex(0,(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, 0,(disp),(basereg),(indexreg),(scale));
        emit_imm32((imm));
 }
 
@@ -825,7 +1250,7 @@ void emit_movw_imm_memindex(codegendata *cd, s4 imm, s4 disp, s4 basereg, s4 ind
        *(cd->mcodeptr++) = 0x66;
        emit_rex(0,0,(indexreg),(basereg));
        *(cd->mcodeptr++) = 0xc7;
-       emit_memindex(0,(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, 0,(disp),(basereg),(indexreg),(scale));
        emit_imm16((imm));
 }
 
@@ -834,11 +1259,20 @@ void emit_movb_imm_memindex(codegendata *cd, s4 imm, s4 disp, s4 basereg, s4 ind
 {
        emit_rex(0,0,(indexreg),(basereg));
        *(cd->mcodeptr++) = 0xc6;
-       emit_memindex(0,(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, 0,(disp),(basereg),(indexreg),(scale));
        emit_imm8((imm));
 }
 
 
+void emit_mov_mem_reg(codegendata *cd, s4 disp, s4 dreg)
+{
+       emit_rex(1, dreg, 0, 0);
+       *(cd->mcodeptr++) = 0x8b;
+       emit_address_byte(0, dreg, 4);
+       emit_mem(4, disp);
+}
+
+
 /*
  * alu operations
  */
@@ -905,7 +1339,8 @@ void emit_alu_imm_reg(codegendata *cd, s8 opc, s8 imm, s8 dreg) {
 }
 
 
-void emit_alu_imm32_reg(codegendata *cd, s8 opc, s8 imm, s8 dreg) {
+void emit_alu_imm32_reg(codegendata *cd, s4 opc, s4 imm, s4 dreg)
+{
        emit_rex(1,0,0,(dreg));
        *(cd->mcodeptr++) = 0x81;
        emit_reg((opc),(dreg));
@@ -913,6 +1348,15 @@ void emit_alu_imm32_reg(codegendata *cd, s8 opc, s8 imm, s8 dreg) {
 }
 
 
+void emit_alul_imm32_reg(codegendata *cd, s4 opc, s4 imm, s4 dreg)
+{
+       emit_rex(0,0,0,(dreg));
+       *(cd->mcodeptr++) = 0x81;
+       emit_reg((opc),(dreg));
+       emit_imm32((imm));
+}
+
+
 void emit_alul_imm_reg(codegendata *cd, s8 opc, s8 imm, s8 dreg) {
        if (IS_IMM8(imm)) {
                emit_rex(0,0,0,(dreg));
@@ -930,12 +1374,12 @@ void emit_alul_imm_reg(codegendata *cd, s8 opc, s8 imm, s8 dreg) {
 
 void emit_alu_imm_membase(codegendata *cd, s8 opc, s8 imm, s8 basereg, s8 disp) {
        if (IS_IMM8(imm)) {
-               emit_rex(1,(basereg),0,0);
+               emit_rex(1,0,0,(basereg));
                *(cd->mcodeptr++) = 0x83;
                emit_membase(cd, (basereg),(disp),(opc));
                emit_imm8((imm));
        } else {
-               emit_rex(1,(basereg),0,0);
+               emit_rex(1,0,0,(basereg));
                *(cd->mcodeptr++) = 0x81;
                emit_membase(cd, (basereg),(disp),(opc));
                emit_imm32((imm));
@@ -945,18 +1389,31 @@ void emit_alu_imm_membase(codegendata *cd, s8 opc, s8 imm, s8 basereg, s8 disp)
 
 void emit_alul_imm_membase(codegendata *cd, s8 opc, s8 imm, s8 basereg, s8 disp) {
        if (IS_IMM8(imm)) {
-               emit_rex(0,(basereg),0,0);
+               emit_rex(0,0,0,(basereg));
                *(cd->mcodeptr++) = 0x83;
                emit_membase(cd, (basereg),(disp),(opc));
                emit_imm8((imm));
        } else {
-               emit_rex(0,(basereg),0,0);
+               emit_rex(0,0,0,(basereg));
                *(cd->mcodeptr++) = 0x81;
                emit_membase(cd, (basereg),(disp),(opc));
                emit_imm32((imm));
        }
 }
 
+void emit_alu_memindex_reg(codegendata *cd, s8 opc, s8 disp, s8 basereg, s8 indexreg, s8 scale, s8 reg)
+{
+       emit_rex(1,(reg),(indexreg),(basereg));
+       *(cd->mcodeptr++) = (((opc)) << 3) + 3;
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
+}
+
+void emit_alul_memindex_reg(codegendata *cd, s8 opc, s8 disp, s8 basereg, s8 indexreg, s8 scale, s8 reg)
+{
+       emit_rex(0,(reg),(indexreg),(basereg));
+       *(cd->mcodeptr++) = (((opc)) << 3) + 3;
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
+}
 
 void emit_test_reg_reg(codegendata *cd, s8 reg, s8 dreg) {
        emit_rex(1,(reg),0,(dreg));
@@ -1008,6 +1465,18 @@ void emit_leal_membase_reg(codegendata *cd, s8 basereg, s8 disp, s8 reg) {
 }
 
 
+void emit_incl_reg(codegendata *cd, s8 reg)
+{
+       *(cd->mcodeptr++) = 0xff;
+       emit_reg(0,(reg));
+}
+
+void emit_incq_reg(codegendata *cd, s8 reg)
+{
+       emit_rex(1,0,0,(reg));
+       *(cd->mcodeptr++) = 0xff;
+       emit_reg(0,(reg));
+}
 
 void emit_incl_membase(codegendata *cd, s8 basereg, s8 disp)
 {
@@ -1016,6 +1485,13 @@ void emit_incl_membase(codegendata *cd, s8 basereg, s8 disp)
        emit_membase(cd, (basereg),(disp),0);
 }
 
+void emit_incq_membase(codegendata *cd, s8 basereg, s8 disp)
+{
+       emit_rex(1,0,0,(basereg));
+       *(cd->mcodeptr++) = 0xff;
+       emit_membase(cd, (basereg),(disp),0);
+}
+
 
 
 void emit_cltd(codegendata *cd) {
@@ -1152,12 +1628,6 @@ void emit_idivl_reg(codegendata *cd, s8 reg) {
 
 
 
-void emit_ret(codegendata *cd) {
-    *(cd->mcodeptr++) = 0xc3;
-}
-
-
-
 /*
  * shift ops
  */
@@ -1254,6 +1724,18 @@ void emit_jmp_imm(codegendata *cd, s8 imm) {
        emit_imm32((imm));
 }
 
+/* like emit_jmp_imm but allows 8 bit optimization */
+void emit_jmp_imm2(codegendata *cd, s8 imm) {
+       if (IS_IMM8(imm)) {
+               *(cd->mcodeptr++) = 0xeb;
+               emit_imm8((imm));
+       }
+       else {
+               *(cd->mcodeptr++) = 0xe9;
+               emit_imm32((imm));
+       }
+}
+
 
 void emit_jmp_reg(codegendata *cd, s8 reg) {
        emit_rex(0,0,0,(reg));
@@ -1275,7 +1757,8 @@ void emit_jcc(codegendata *cd, s8 opc, s8 imm) {
  */
 
 /* we need the rex byte to get all low bytes */
-void emit_setcc_reg(codegendata *cd, s8 opc, s8 reg) {
+void emit_setcc_reg(codegendata *cd, s4 opc, s4 reg)
+{
        *(cd->mcodeptr++) = (0x40 | (((reg) >> 3) & 0x01));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = (0x90 + (opc));
@@ -1284,7 +1767,8 @@ void emit_setcc_reg(codegendata *cd, s8 opc, s8 reg) {
 
 
 /* we need the rex byte to get all low bytes */
-void emit_setcc_membase(codegendata *cd, s8 opc, s8 basereg, s8 disp) {
+void emit_setcc_membase(codegendata *cd, s4 opc, s4 basereg, s4 disp)
+{
        *(cd->mcodeptr++) = (0x40 | (((basereg) >> 3) & 0x01));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = (0x90 + (opc));
@@ -1292,7 +1776,7 @@ void emit_setcc_membase(codegendata *cd, s8 opc, s8 basereg, s8 disp) {
 }
 
 
-void emit_cmovcc_reg_reg(codegendata *cd, s8 opc, s8 reg, s8 dreg)
+void emit_cmovcc_reg_reg(codegendata *cd, s4 opc, s4 reg, s4 dreg)
 {
        emit_rex(1,(dreg),0,(reg));
        *(cd->mcodeptr++) = 0x0f;
@@ -1301,7 +1785,7 @@ void emit_cmovcc_reg_reg(codegendata *cd, s8 opc, s8 reg, s8 dreg)
 }
 
 
-void emit_cmovccl_reg_reg(codegendata *cd, s8 opc, s8 reg, s8 dreg)
+void emit_cmovccl_reg_reg(codegendata *cd, s4 opc, s4 reg, s4 dreg)
 {
        emit_rex(0,(dreg),0,(reg));
        *(cd->mcodeptr++) = 0x0f;
@@ -1310,7 +1794,6 @@ void emit_cmovccl_reg_reg(codegendata *cd, s8 opc, s8 reg, s8 dreg)
 }
 
 
-
 void emit_neg_reg(codegendata *cd, s8 reg)
 {
        emit_rex(1,0,0,(reg));
@@ -1352,23 +1835,20 @@ void emit_xchg_reg_reg(codegendata *cd, s8 reg, s8 dreg) {
 }
 
 
-void emit_nop(codegendata *cd) {
-    *(cd->mcodeptr++) = 0x90;
-}
-
-
 
 /*
  * call instructions
  */
-void emit_call_reg(codegendata *cd, s8 reg) {
-       emit_rex(1,0,0,(reg));
+void emit_call_reg(codegendata *cd, s8 reg)
+{
+       emit_rex(0,0,0,(reg));
        *(cd->mcodeptr++) = 0xff;
        emit_reg(2,(reg));
 }
 
 
-void emit_call_imm(codegendata *cd, s8 imm) {
+void emit_call_imm(codegendata *cd, s8 imm)
+{
        *(cd->mcodeptr++) = 0xe8;
        emit_imm32((imm));
 }
@@ -1543,7 +2023,7 @@ void emit_movd_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 ind
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x7e;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -1570,7 +2050,7 @@ void emit_movd_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg, s
        emit_rex(0,(dreg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x6e;
-       emit_memindex((dreg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (dreg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -1635,7 +2115,7 @@ void emit_movss_reg_membase32(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
        emit_byte_rex((reg),0,(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x11;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
@@ -1655,7 +2135,7 @@ void emit_movsd_reg_membase32(codegendata *cd, s8 reg, s8 basereg, s8 disp) {
        emit_byte_rex((reg),0,(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x11;
-       emit_membase32((basereg),(disp),(reg));
+       emit_membase32(cd, (basereg),(disp),(reg));
 }
 
 
@@ -1675,7 +2155,7 @@ void emit_movss_membase32_reg(codegendata *cd, s8 basereg, s8 disp, s8 dreg) {
        emit_byte_rex((dreg),0,(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x10;
-       emit_membase32((basereg),(disp),(dreg));
+       emit_membase32(cd, (basereg),(disp),(dreg));
 }
 
 
@@ -1713,7 +2193,7 @@ void emit_movsd_membase32_reg(codegendata *cd, s8 basereg, s8 disp, s8 dreg) {
        emit_byte_rex((dreg),0,(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x10;
-       emit_membase32((basereg),(disp),(dreg));
+       emit_membase32(cd, (basereg),(disp),(dreg));
 }
 
 
@@ -1742,7 +2222,7 @@ void emit_movss_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 in
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x11;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -1751,7 +2231,7 @@ void emit_movsd_reg_memindex(codegendata *cd, s8 reg, s8 disp, s8 basereg, s8 in
        emit_rex(0,(reg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x11;
-       emit_memindex((reg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (reg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -1760,7 +2240,7 @@ void emit_movss_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg,
        emit_rex(0,(dreg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x10;
-       emit_memindex((dreg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (dreg),(disp),(basereg),(indexreg),(scale));
 }
 
 
@@ -1769,7 +2249,7 @@ void emit_movsd_memindex_reg(codegendata *cd, s8 disp, s8 basereg, s8 indexreg,
        emit_rex(0,(dreg),(indexreg),(basereg));
        *(cd->mcodeptr++) = 0x0f;
        *(cd->mcodeptr++) = 0x10;
-       emit_memindex((dreg),(disp),(basereg),(indexreg),(scale));
+       emit_memindex(cd, (dreg),(disp),(basereg),(indexreg),(scale));
 }