more stuff storm needs
[ia32rtools.git] / tools / translate.c
index e2b6d20..7493761 100644 (file)
@@ -32,15 +32,19 @@ enum op_flags {
   OPF_RMD    = (1 << 0), /* removed or optimized out */
   OPF_DATA   = (1 << 1), /* data processing - writes to dst opr */
   OPF_FLAGS  = (1 << 2), /* sets flags */
-  OPF_JMP    = (1 << 3), /* branches, ret and call */
-  OPF_CC     = (1 << 4), /* uses flags */
-  OPF_TAIL   = (1 << 5), /* ret or tail call */
-  OPF_RSAVE  = (1 << 6), /* push/pop is local reg save/load */
-  OPF_REP    = (1 << 7), /* prefixed by rep */
-  OPF_REPZ   = (1 << 8), /* rep is repe/repz */
-  OPF_REPNZ  = (1 << 9), /* rep is repne/repnz */
-  OPF_FARG   = (1 << 10), /* push collected as func arg (no reuse) */
-  OPF_EBP_S  = (1 << 11), /* ebp used as scratch here, not BP */
+  OPF_JMP    = (1 << 3), /* branch, call */
+  OPF_CJMP   = (1 << 4), /* cond. branch (cc or jecxz) */
+  OPF_CC     = (1 << 5), /* uses flags */
+  OPF_TAIL   = (1 << 6), /* ret or tail call */
+  OPF_RSAVE  = (1 << 7), /* push/pop is local reg save/load */
+  OPF_REP    = (1 << 8), /* prefixed by rep */
+  OPF_REPZ   = (1 << 9), /* rep is repe/repz */
+  OPF_REPNZ  = (1 << 10), /* rep is repne/repnz */
+  OPF_FARG   = (1 << 11), /* push collected as func arg (no reuse) */
+  OPF_EBP_S  = (1 << 12), /* ebp used as scratch here, not BP */
+  OPF_DF     = (1 << 13), /* DF flag set */
+  OPF_ATAIL  = (1 << 14), /* tail call with reused arg frame */
+  OPF_32BIT  = (1 << 15), /* 32bit division */
 };
 
 enum op_op {
@@ -48,15 +52,20 @@ enum op_op {
        OP_NOP,
        OP_PUSH,
        OP_POP,
+       OP_LEAVE,
        OP_MOV,
        OP_LEA,
        OP_MOVZX,
        OP_MOVSX,
+       OP_XCHG,
        OP_NOT,
        OP_CDQ,
        OP_STOS,
        OP_MOVS,
        OP_CMPS,
+       OP_SCAS,
+       OP_STD,
+       OP_CLD,
        OP_RET,
        OP_ADD,
        OP_SUB,
@@ -68,6 +77,8 @@ enum op_op {
        OP_SAR,
        OP_ROL,
        OP_ROR,
+       OP_RCL,
+       OP_RCR,
        OP_ADC,
        OP_SBB,
        OP_INC,
@@ -81,6 +92,7 @@ enum op_op {
        OP_CMP,
        OP_CALL,
        OP_JMP,
+       OP_JECXZ,
        OP_JO,
        OP_JNO,
        OP_JC,
@@ -150,6 +162,7 @@ struct parsed_op {
 // datap:
 // OP_CALL - parser proto hint (str), ptr to struct parsed_proto
 // (OPF_CC) - point to one of (OPF_FLAGS) that affects cc op
+// OP_POP - point to OP_PUSH in push/pop pair
 
 struct parsed_equ {
   char name[64];
@@ -398,7 +411,7 @@ static const char *parse_stack_el(const char *name, char *extra_reg)
     // must be a number after esp+, already converted to 0x..
     s = name + 4;
     if (!('0' <= *s && *s <= '9')) {
-                 aerr("%s nan?\n", __func__);
+      aerr("%s nan?\n", __func__);
       return NULL;
     }
     if (s[0] == '0' && s[1] == 'x')
@@ -446,8 +459,8 @@ static int guess_lmod_from_c_type(enum opr_lenmod *lmod,
 {
   static const char *dword_types[] = {
     "int", "_DWORD", "UINT_PTR", "DWORD",
-    "WPARAM", "LPARAM", "UINT",
-    "HIMC",
+    "WPARAM", "LPARAM", "UINT", "__int32",
+    "LONG", "HIMC",
   };
   static const char *word_types[] = {
     "uint16_t", "int16_t", "_WORD",
@@ -456,7 +469,7 @@ static int guess_lmod_from_c_type(enum opr_lenmod *lmod,
   static const char *byte_types[] = {
     "uint8_t", "int8_t", "char",
     "unsigned __int8", "__int8", "BYTE", "_BYTE",
-    "_UNKNOWN",
+    "CHAR", "_UNKNOWN",
   };
   const char *n;
   int i;
@@ -701,6 +714,8 @@ static const struct {
   { "repnz",  OPF_REP|OPF_REPNZ },
 };
 
+#define OPF_CJMP_CC (OPF_JMP|OPF_CJMP|OPF_CC)
+
 static const struct {
   const char *name;
   enum op_op op;
@@ -711,10 +726,12 @@ static const struct {
   { "nop",  OP_NOP,    0, 0, 0 },
   { "push", OP_PUSH,   1, 1, 0 },
   { "pop",  OP_POP,    1, 1, OPF_DATA },
+  { "leave",OP_LEAVE,  0, 0, OPF_DATA },
   { "mov" , OP_MOV,    2, 2, OPF_DATA },
   { "lea",  OP_LEA,    2, 2, OPF_DATA },
   { "movzx",OP_MOVZX,  2, 2, OPF_DATA },
   { "movsx",OP_MOVSX,  2, 2, OPF_DATA },
+  { "xchg", OP_XCHG,   2, 2, OPF_DATA },
   { "not",  OP_NOT,    1, 1, OPF_DATA },
   { "cdq",  OP_CDQ,    0, 0, OPF_DATA },
   { "stosb",OP_STOS,   0, 0, OPF_DATA },
@@ -726,6 +743,11 @@ static const struct {
   { "cmpsb",OP_CMPS,   0, 0, OPF_DATA|OPF_FLAGS },
   { "cmpsw",OP_CMPS,   0, 0, OPF_DATA|OPF_FLAGS },
   { "cmpsd",OP_CMPS,   0, 0, OPF_DATA|OPF_FLAGS },
+  { "scasb",OP_SCAS,   0, 0, OPF_DATA|OPF_FLAGS },
+  { "scasw",OP_SCAS,   0, 0, OPF_DATA|OPF_FLAGS },
+  { "scasd",OP_SCAS,   0, 0, OPF_DATA|OPF_FLAGS },
+  { "std",  OP_STD,    0, 0, OPF_DATA }, // special flag
+  { "cld",  OP_CLD,    0, 0, OPF_DATA },
   { "add",  OP_ADD,    2, 2, OPF_DATA|OPF_FLAGS },
   { "sub",  OP_SUB,    2, 2, OPF_DATA|OPF_FLAGS },
   { "and",  OP_AND,    2, 2, OPF_DATA|OPF_FLAGS },
@@ -737,6 +759,8 @@ static const struct {
   { "sar",  OP_SAR,    2, 2, OPF_DATA|OPF_FLAGS },
   { "rol",  OP_ROL,    2, 2, OPF_DATA|OPF_FLAGS },
   { "ror",  OP_ROR,    2, 2, OPF_DATA|OPF_FLAGS },
+  { "rcl",  OP_RCL,    2, 2, OPF_DATA|OPF_FLAGS|OPF_CC },
+  { "rcr",  OP_RCR,    2, 2, OPF_DATA|OPF_FLAGS|OPF_CC },
   { "adc",  OP_ADC,    2, 2, OPF_DATA|OPF_FLAGS|OPF_CC },
   { "sbb",  OP_SBB,    2, 2, OPF_DATA|OPF_FLAGS|OPF_CC },
   { "inc",  OP_INC,    1, 1, OPF_DATA|OPF_FLAGS },
@@ -751,35 +775,36 @@ static const struct {
   { "retn", OP_RET,    0, 1, OPF_TAIL },
   { "call", OP_CALL,   1, 1, OPF_JMP|OPF_DATA|OPF_FLAGS },
   { "jmp",  OP_JMP,    1, 1, OPF_JMP },
-  { "jo",   OP_JO,     1, 1, OPF_JMP|OPF_CC }, // 70 OF=1
-  { "jno",  OP_JNO,    1, 1, OPF_JMP|OPF_CC }, // 71 OF=0
-  { "jc",   OP_JC,     1, 1, OPF_JMP|OPF_CC }, // 72 CF=1
-  { "jb",   OP_JC,     1, 1, OPF_JMP|OPF_CC }, // 72
-  { "jnc",  OP_JNC,    1, 1, OPF_JMP|OPF_CC }, // 73 CF=0
-  { "jnb",  OP_JNC,    1, 1, OPF_JMP|OPF_CC }, // 73
-  { "jae",  OP_JNC,    1, 1, OPF_JMP|OPF_CC }, // 73
-  { "jz",   OP_JZ,     1, 1, OPF_JMP|OPF_CC }, // 74 ZF=1
-  { "je",   OP_JZ,     1, 1, OPF_JMP|OPF_CC }, // 74
-  { "jnz",  OP_JNZ,    1, 1, OPF_JMP|OPF_CC }, // 75 ZF=0
-  { "jne",  OP_JNZ,    1, 1, OPF_JMP|OPF_CC }, // 75
-  { "jbe",  OP_JBE,    1, 1, OPF_JMP|OPF_CC }, // 76 CF=1 || ZF=1
-  { "jna",  OP_JBE,    1, 1, OPF_JMP|OPF_CC }, // 76
-  { "ja",   OP_JA,     1, 1, OPF_JMP|OPF_CC }, // 77 CF=0 && ZF=0
-  { "jnbe", OP_JA,     1, 1, OPF_JMP|OPF_CC }, // 77
-  { "js",   OP_JS,     1, 1, OPF_JMP|OPF_CC }, // 78 SF=1
-  { "jns",  OP_JNS,    1, 1, OPF_JMP|OPF_CC }, // 79 SF=0
-  { "jp",   OP_JP,     1, 1, OPF_JMP|OPF_CC }, // 7a PF=1
-  { "jpe",  OP_JP,     1, 1, OPF_JMP|OPF_CC }, // 7a
-  { "jnp",  OP_JNP,    1, 1, OPF_JMP|OPF_CC }, // 7b PF=0
-  { "jpo",  OP_JNP,    1, 1, OPF_JMP|OPF_CC }, // 7b
-  { "jl",   OP_JL,     1, 1, OPF_JMP|OPF_CC }, // 7c SF!=OF
-  { "jnge", OP_JL,     1, 1, OPF_JMP|OPF_CC }, // 7c
-  { "jge",  OP_JGE,    1, 1, OPF_JMP|OPF_CC }, // 7d SF=OF
-  { "jnl",  OP_JGE,    1, 1, OPF_JMP|OPF_CC }, // 7d
-  { "jle",  OP_JLE,    1, 1, OPF_JMP|OPF_CC }, // 7e ZF=1 || SF!=OF
-  { "jng",  OP_JLE,    1, 1, OPF_JMP|OPF_CC }, // 7e
-  { "jg",   OP_JG,     1, 1, OPF_JMP|OPF_CC }, // 7f ZF=0 && SF=OF
-  { "jnle", OP_JG,     1, 1, OPF_JMP|OPF_CC }, // 7f
+  { "jecxz",OP_JECXZ,  1, 1, OPF_JMP|OPF_CJMP },
+  { "jo",   OP_JO,     1, 1, OPF_CJMP_CC }, // 70 OF=1
+  { "jno",  OP_JNO,    1, 1, OPF_CJMP_CC }, // 71 OF=0
+  { "jc",   OP_JC,     1, 1, OPF_CJMP_CC }, // 72 CF=1
+  { "jb",   OP_JC,     1, 1, OPF_CJMP_CC }, // 72
+  { "jnc",  OP_JNC,    1, 1, OPF_CJMP_CC }, // 73 CF=0
+  { "jnb",  OP_JNC,    1, 1, OPF_CJMP_CC }, // 73
+  { "jae",  OP_JNC,    1, 1, OPF_CJMP_CC }, // 73
+  { "jz",   OP_JZ,     1, 1, OPF_CJMP_CC }, // 74 ZF=1
+  { "je",   OP_JZ,     1, 1, OPF_CJMP_CC }, // 74
+  { "jnz",  OP_JNZ,    1, 1, OPF_CJMP_CC }, // 75 ZF=0
+  { "jne",  OP_JNZ,    1, 1, OPF_CJMP_CC }, // 75
+  { "jbe",  OP_JBE,    1, 1, OPF_CJMP_CC }, // 76 CF=1 || ZF=1
+  { "jna",  OP_JBE,    1, 1, OPF_CJMP_CC }, // 76
+  { "ja",   OP_JA,     1, 1, OPF_CJMP_CC }, // 77 CF=0 && ZF=0
+  { "jnbe", OP_JA,     1, 1, OPF_CJMP_CC }, // 77
+  { "js",   OP_JS,     1, 1, OPF_CJMP_CC }, // 78 SF=1
+  { "jns",  OP_JNS,    1, 1, OPF_CJMP_CC }, // 79 SF=0
+  { "jp",   OP_JP,     1, 1, OPF_CJMP_CC }, // 7a PF=1
+  { "jpe",  OP_JP,     1, 1, OPF_CJMP_CC }, // 7a
+  { "jnp",  OP_JNP,    1, 1, OPF_CJMP_CC }, // 7b PF=0
+  { "jpo",  OP_JNP,    1, 1, OPF_CJMP_CC }, // 7b
+  { "jl",   OP_JL,     1, 1, OPF_CJMP_CC }, // 7c SF!=OF
+  { "jnge", OP_JL,     1, 1, OPF_CJMP_CC }, // 7c
+  { "jge",  OP_JGE,    1, 1, OPF_CJMP_CC }, // 7d SF=OF
+  { "jnl",  OP_JGE,    1, 1, OPF_CJMP_CC }, // 7d
+  { "jle",  OP_JLE,    1, 1, OPF_CJMP_CC }, // 7e ZF=1 || SF!=OF
+  { "jng",  OP_JLE,    1, 1, OPF_CJMP_CC }, // 7e
+  { "jg",   OP_JG,     1, 1, OPF_CJMP_CC }, // 7f ZF=0 && SF=OF
+  { "jnle", OP_JG,     1, 1, OPF_CJMP_CC }, // 7f
   { "seto",   OP_JO,   1, 1, OPF_DATA|OPF_CC },
   { "setno",  OP_JNO,  1, 1, OPF_DATA|OPF_CC },
   { "setc",   OP_JC,   1, 1, OPF_DATA|OPF_CC },
@@ -883,13 +908,14 @@ static void parse_op(struct parsed_op *op, char words[16][256], int wordc)
     break;
 
   case OP_STOS:
+  case OP_SCAS:
     if (op->operand_cnt != 0)
       break;
-    if      (IS(words[op_w], "stosb"))
+    if      (words[op_w][4] == 'b')
       lmod = OPLM_BYTE;
-    else if (IS(words[op_w], "stosw"))
+    else if (words[op_w][4] == 'w')
       lmod = OPLM_WORD;
-    else if (IS(words[op_w], "stosd"))
+    else if (words[op_w][4] == 'd')
       lmod = OPLM_DWORD;
     op->operand_cnt = 3;
     setup_reg_opr(&op->operand[0], xDI, lmod, &op->regmask_src);
@@ -915,6 +941,19 @@ static void parse_op(struct parsed_op *op, char words[16][256], int wordc)
     op->regmask_dst = op->regmask_src;
     break;
 
+  case OP_XCHG:
+    op->regmask_src |= op->regmask_dst;
+    op->regmask_dst |= op->regmask_src;
+    break;
+
+  case OP_JECXZ:
+    op->operand_cnt = 1;
+    op->regmask_src = 1 << xCX;
+    op->operand[0].type = OPT_REG;
+    op->operand[0].reg = xCX;
+    op->operand[0].lmod = OPLM_DWORD;
+    break;
+
   case OP_IMUL:
     if (op->operand_cnt != 1)
       break;
@@ -1389,8 +1428,10 @@ static void stack_frame_access(struct parsed_op *po,
       ferr(po, "bp_arg arg%d/w offset %d and type '%s' is too small\n",
         i + 1, offset, g_func_pp->arg[i].type.name);
     }
-    if (popr->is_ptr && popr->lmod != OPLM_DWORD)
-      ferr(po, "bp_arg arg%d: non-dword ptr access\n", i + 1);
+    // can't check this because msvc likes to reuse
+    // arg space for scratch..
+    //if (popr->is_ptr && popr->lmod != OPLM_DWORD)
+    //  ferr(po, "bp_arg arg%d: non-dword ptr access\n", i + 1);
   }
   else
   {
@@ -1450,9 +1491,13 @@ static void stack_frame_access(struct parsed_op *po,
 static void check_func_pp(struct parsed_op *po,
   const struct parsed_proto *pp, const char *pfx)
 {
+  char buf[256];
+
   if (pp->argc_reg != 0) {
-    if (!g_allow_regfunc && !pp->is_fastcall)
-      ferr(po, "%s: reg arg in arg-call unhandled yet\n", pfx);
+    if (/*!g_allow_regfunc &&*/ !pp->is_fastcall) {
+      pp_print(buf, sizeof(buf), pp);
+      ferr(po, "%s: unexpected reg arg in icall: %s\n", pfx, buf);
+    }
     if (pp->argc_stack > 0 && pp->argc_reg != 2)
       ferr(po, "%s: %d reg arg(s) with %d stack arg(s)\n",
         pfx, pp->argc_reg, pp->argc_stack);
@@ -1472,7 +1517,7 @@ static void check_label_read_ref(struct parsed_op *po, const char *name)
 }
 
 static char *out_src_opr(char *buf, size_t buf_size,
-       struct parsed_op *po, struct parsed_opr *popr, const char *cast,
+  struct parsed_op *po, struct parsed_opr *popr, const char *cast,
   int is_lea)
 {
   char tmp1[256], tmp2[256];
@@ -1694,6 +1739,8 @@ static enum parsed_flag_op split_cond(struct parsed_op *po,
     *is_inv = 1;
     return PFO_LE;
 
+  case OP_RCL:
+  case OP_RCR:
   case OP_ADC:
   case OP_SBB:
     return PFO_C;
@@ -1909,7 +1956,7 @@ static void op_set_clear_flag(struct parsed_op *po,
 
 // last op in stream - unconditional branch or ret
 #define LAST_OP(_i) ((ops[_i].flags & OPF_TAIL) \
-  || (ops[_i].flags & (OPF_JMP|OPF_CC)) == OPF_JMP)
+  || (ops[_i].flags & (OPF_JMP|OPF_CJMP)) == OPF_JMP)
 
 static int scan_for_pop(int i, int opcnt, const char *reg,
   int magic, int depth, int *maxdepth, int do_flags)
@@ -1956,7 +2003,7 @@ static int scan_for_pop(int i, int opcnt, const char *reg,
         return -1;
       }
 
-      if (po->flags & OPF_CC) {
+      if (po->flags & OPF_CJMP) {
         ret |= scan_for_pop(po->bt_i, opcnt, reg, magic,
                  depth, maxdepth, do_flags);
         if (ret < 0)
@@ -2032,6 +2079,52 @@ static int scan_for_pop_ret(int i, int opcnt, const char *reg,
   return found ? 0 : -1;
 }
 
+static void scan_propagate_df(int i, int opcnt)
+{
+  struct parsed_op *po = &ops[i];
+  int j;
+
+  for (; i < opcnt; i++) {
+    po = &ops[i];
+    if (po->flags & OPF_DF)
+      return; // already resolved
+    po->flags |= OPF_DF;
+
+    if (po->op == OP_CALL)
+      ferr(po, "call with DF set?\n");
+
+    if (po->flags & OPF_JMP) {
+      if (po->btj != NULL) {
+        // jumptable
+        for (j = 0; j < po->btj->count; j++)
+          scan_propagate_df(po->btj->d[j].bt_i, opcnt);
+        return;
+      }
+
+      if (po->bt_i < 0) {
+        ferr(po, "dead branch\n");
+        return;
+      }
+
+      if (po->flags & OPF_CJMP)
+        scan_propagate_df(po->bt_i, opcnt);
+      else
+        i = po->bt_i - 1;
+      continue;
+    }
+
+    if (po->flags & OPF_TAIL)
+      break;
+
+    if (po->op == OP_CLD) {
+      po->flags |= OPF_RMD;
+      return;
+    }
+  }
+
+  ferr(po, "missing DF clear?\n");
+}
+
 // is operand 'opr' modified by parsed_op 'po'?
 static int is_opr_modified(const struct parsed_opr *opr,
   const struct parsed_op *po)
@@ -2162,7 +2255,7 @@ static int scan_for_flag_set(int i, int magic, int *branched,
       return 0;
     }
 
-    if ((ops[i].flags & OPF_JMP) && !(ops[i].flags & OPF_CC))
+    if ((ops[i].flags & (OPF_JMP|OPF_CJMP)) == OPF_JMP)
       return -1;
   }
 
@@ -2222,18 +2315,20 @@ static int scan_for_reg_clear(int i, int reg)
 }
 
 // scan for positive, constant esp adjust
-static int scan_for_esp_adjust(int i, int opcnt, int *adj)
+static int scan_for_esp_adjust(int i, int opcnt, int *adj,
+  int *multipath)
 {
   const struct parsed_proto *pp;
   struct parsed_op *po;
   int first_pop = -1;
-  *adj = 0;
+
+  *adj = *multipath = 0;
 
   for (; i < opcnt; i++) {
     po = &ops[i];
 
     if (g_labels[i][0] != 0)
-      break;
+      *multipath = 1;
 
     if (po->op == OP_ADD && po->operand[0].reg == xSP) {
       if (po->operand[1].type != OPT_CONST)
@@ -2243,17 +2338,24 @@ static int scan_for_esp_adjust(int i, int opcnt, int *adj)
         ferr(&ops[i], "unaligned esp adjust: %x\n", *adj);
       return i;
     }
-    else if (po->op == OP_PUSH) {
-      if (first_pop == -1)
-        first_pop = -2; // none
+    else if (po->op == OP_PUSH && !(po->flags & OPF_RMD)) {
+      //if (first_pop == -1)
+      //  first_pop = -2; // none
       *adj -= lmod_bytes(po, po->operand[0].lmod);
     }
-    else if (po->op == OP_POP) {
-      if (first_pop == -1)
+    else if (po->op == OP_POP && !(po->flags & OPF_RMD)) {
+      // seems like msvc only uses 'pop ecx' for stack realignment..
+      if (po->operand[0].type != OPT_REG || po->operand[0].reg != xCX)
+        break;
+      if (first_pop == -1 && *adj >= 0)
         first_pop = i;
       *adj += lmod_bytes(po, po->operand[0].lmod);
     }
     else if (po->flags & (OPF_JMP|OPF_TAIL)) {
+      if (po->op == OP_JMP && po->btj == NULL) {
+        i = po->bt_i - 1;
+        continue;
+      }
       if (po->op != OP_CALL)
         break;
       if (po->operand[0].type != OPT_LABEL)
@@ -2264,10 +2366,7 @@ static int scan_for_esp_adjust(int i, int opcnt, int *adj)
     }
   }
 
-  if (*adj == 4 && first_pop >= 0 && ops[first_pop].op == OP_POP
-    && ops[first_pop].operand[0].type == OPT_REG
-    && ops[first_pop].operand[0].reg == xCX)
-  {
+  if (first_pop >= 0) {
     // probably 'pop ecx' was used..
     return first_pop;
   }
@@ -2299,7 +2398,7 @@ static void scan_fwd_set_flags(int i, int opcnt, int magic, int flags)
       }
 
       scan_fwd_set_flags(po->bt_i, opcnt, magic, flags);
-      if (!(po->flags & OPF_CC))
+      if (!(po->flags & OPF_CJMP))
         return;
     }
     if (po->flags & OPF_TAIL)
@@ -2505,7 +2604,7 @@ static int collect_call_args_r(struct parsed_op *po, int i,
       if (lr->next != NULL)
         need_op_saving = 1;
       for (; lr->next; lr = lr->next) {
-        if ((ops[lr->i].flags & (OPF_JMP|OPF_CC)) != OPF_JMP)
+        if ((ops[lr->i].flags & (OPF_JMP|OPF_CJMP)) != OPF_JMP)
           may_reuse = 1;
         ret = collect_call_args_r(po, lr->i, pp, regmask, save_arg_vars,
                 arg, magic, need_op_saving, may_reuse);
@@ -2513,7 +2612,7 @@ static int collect_call_args_r(struct parsed_op *po, int i,
           return ret;
       }
 
-      if ((ops[lr->i].flags & (OPF_JMP|OPF_CC)) != OPF_JMP)
+      if ((ops[lr->i].flags & (OPF_JMP|OPF_CJMP)) != OPF_JMP)
         may_reuse = 1;
       if (j > 0 && LAST_OP(j - 1)) {
         // follow last branch in reverse
@@ -2554,7 +2653,7 @@ static int collect_call_args_r(struct parsed_op *po, int i,
 
       ferr(po, "arg collect %d/%d hit pop\n", arg, pp->argc);
     }
-    else if ((ops[j].flags & (OPF_JMP|OPF_CC)) == (OPF_JMP|OPF_CC))
+    else if (ops[j].flags & OPF_CJMP)
     {
       if (pp->is_unresolved)
         break;
@@ -2711,8 +2810,9 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
   const char *tmpname;
   enum parsed_flag_op pfo;
   int save_arg_vars = 0;
-  int cmp_result_vars = 0;
-  int need_mul_var = 0;
+  int cond_vars = 0;
+  int need_tmp_var = 0;
+  int need_tmp64 = 0;
   int had_decl = 0;
   int label_pending = 0;
   int regmask_save = 0;
@@ -2739,7 +2839,7 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
 
   fprintf(fout, "%s ", g_func_pp->ret_type.name);
   output_pp_attrs(fout, g_func_pp, g_ida_func_attr & IDAFA_NORETURN);
-  fprintf(fout, "%s(", funcn);
+  fprintf(fout, "%s(", g_func_pp->name);
 
   for (i = 0; i < g_func_pp->argc; i++) {
     if (i > 0)
@@ -2825,28 +2925,44 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
       for (; i < opcnt; i++)
         if (ops[i].op == OP_RET)
           break;
-      if (i == opcnt && (ops[i - 1].flags & OPF_JMP) && found)
-        break;
+      j = i - 1;
+      if (i == opcnt && (ops[j].flags & OPF_JMP)) {
+        if (found) {
+          if (ops[j].op == OP_JMP && !ops[j].operand[0].had_ds)
+            // probably local branch back..
+            break;
+          if (ops[j].op == OP_CALL)
+            // probably noreturn call..
+            break;
+        }
+        j--;
+      }
 
-      if (ops[i - 1].op == OP_POP && IS(opr_name(&ops[i - 1], 0), "ebp"))
-        ops[i - 1].flags |= OPF_RMD;
+      if ((ops[j].op == OP_POP && IS(opr_name(&ops[j], 0), "ebp"))
+          || ops[j].op == OP_LEAVE)
+      {
+        ops[j].flags |= OPF_RMD;
+      }
       else if (!(g_ida_func_attr & IDAFA_NORETURN))
-        ferr(&ops[i - 1], "'pop ebp' expected\n");
+        ferr(&ops[j], "'pop ebp' expected\n");
 
       if (g_stack_fsz != 0) {
-        if (ops[i - 2].op == OP_MOV
-            && IS(opr_name(&ops[i - 2], 0), "esp")
-            && IS(opr_name(&ops[i - 2], 1), "ebp"))
+        if (ops[j - 1].op == OP_MOV
+            && IS(opr_name(&ops[j - 1], 0), "esp")
+            && IS(opr_name(&ops[j - 1], 1), "ebp"))
         {
-          ops[i - 2].flags |= OPF_RMD;
+          ops[j - 1].flags |= OPF_RMD;
+        }
+        else if (ops[j].op != OP_LEAVE
+          && !(g_ida_func_attr & IDAFA_NORETURN))
+        {
+          ferr(&ops[j - 1], "esp restore expected\n");
         }
-        else if (!(g_ida_func_attr & IDAFA_NORETURN))
-          ferr(&ops[i - 2], "esp restore expected\n");
 
-        if (ecx_push && ops[i - 3].op == OP_POP
-          && IS(opr_name(&ops[i - 3], 0), "ecx"))
+        if (ecx_push && ops[j - 2].op == OP_POP
+          && IS(opr_name(&ops[j - 2], 0), "ecx"))
         {
-          ferr(&ops[i - 3], "unexpected ecx pop\n");
+          ferr(&ops[j - 2], "unexpected ecx pop\n");
         }
       }
 
@@ -2993,6 +3109,8 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
 tailcall:
     po->op = OP_CALL;
     po->flags |= OPF_TAIL;
+    if (i > 0 && ops[i - 1].op == OP_POP)
+      po->flags |= OPF_ATAIL;
     i--; // reprocess
   }
 
@@ -3027,7 +3145,7 @@ tailcall:
           pp = calloc(1, sizeof(*pp));
           my_assert_not(pp, NULL);
           pp->is_fptr = 1;
-          ret = scan_for_esp_adjust(i + 1, opcnt, &j);
+          ret = scan_for_esp_adjust(i + 1, opcnt, &j, &l);
           if (ret < 0) {
             if (!g_allow_regfunc)
               ferr(po, "non-__cdecl indirect call unhandled yet\n");
@@ -3048,7 +3166,7 @@ tailcall:
       // look for and make use of esp adjust
       ret = -1;
       if (!pp->is_stdcall && pp->argc_stack > 0)
-        ret = scan_for_esp_adjust(i + 1, opcnt, &j);
+        ret = scan_for_esp_adjust(i + 1, opcnt, &j, &l);
       if (ret >= 0) {
         if (pp->is_vararg) {
           if (j / 4 < pp->argc_stack)
@@ -3069,27 +3187,39 @@ tailcall:
             tmpname, pp->argc_stack * 4, j);
 
         ops[ret].flags |= OPF_RMD;
-        // a bit of a hack, but deals with use of
-        // single adj for multiple calls
-        ops[ret].operand[1].val -= j;
+        if (ops[ret].op == OP_POP && j > 4) {
+          // deal with multi-pop stack adjust
+          j = pp->argc_stack;
+          while (ops[ret].op == OP_POP && j > 0 && ret < opcnt) {
+            ops[ret].flags |= OPF_RMD;
+            j--;
+            ret++;
+          }
+        }
+        else if (!l) {
+          // a bit of a hack, but deals with use of
+          // single adj for multiple calls
+          ops[ret].operand[1].val -= j;
+        }
       }
       else if (pp->is_vararg)
         ferr(po, "missing esp_adjust for vararg func '%s'\n",
           pp->name);
 
-      if (!pp->is_unresolved) {
+      if (!pp->is_unresolved && !(po->flags & OPF_ATAIL)) {
         // since we know the args, collect them
         collect_call_args(po, i, pp, &regmask, &save_arg_vars,
           i + opcnt * 2);
       }
 
       if (strstr(pp->ret_type.name, "int64"))
-        need_mul_var = 1;
+        need_tmp64 = 1;
     }
   }
 
   // pass4:
   // - find POPs for PUSHes, rm both
+  // - scan for STD/CLD, propagate DF
   // - scan for all used registers
   // - find flag set ops for their users
   // - do unreselved calls
@@ -3108,40 +3238,63 @@ tailcall:
         regmask_save |= 1 << reg;
     }
 
-    if (po->op == OP_PUSH
-        && po->argnum == 0 && !(po->flags & OPF_RSAVE)
-        && po->operand[0].type == OPT_REG)
+    if (po->op == OP_PUSH && po->argnum == 0
+      && !(po->flags & OPF_RSAVE))
     {
-      reg = po->operand[0].reg;
-      if (reg < 0)
-        ferr(po, "reg not set for push?\n");
-
-      depth = 0;
-      ret = scan_for_pop(i + 1, opcnt,
-              po->operand[0].name, i + opcnt * 3, 0, &depth, 0);
-      if (ret == 1) {
-        if (depth > 1)
-          ferr(po, "too much depth: %d\n", depth);
-
-        po->flags |= OPF_RMD;
-        scan_for_pop(i + 1, opcnt, po->operand[0].name,
-          i + opcnt * 4, 0, &depth, 1);
-        continue;
+      if (po->operand[0].type == OPT_REG)
+      {
+        reg = po->operand[0].reg;
+        if (reg < 0)
+          ferr(po, "reg not set for push?\n");
+
+        depth = 0;
+        ret = scan_for_pop(i + 1, opcnt,
+                po->operand[0].name, i + opcnt * 3, 0, &depth, 0);
+        if (ret == 1) {
+          if (depth > 1)
+            ferr(po, "too much depth: %d\n", depth);
+
+          po->flags |= OPF_RMD;
+          scan_for_pop(i + 1, opcnt, po->operand[0].name,
+            i + opcnt * 4, 0, &depth, 1);
+          continue;
+        }
+        ret = scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, 0);
+        if (ret == 0) {
+          arg = OPF_RMD;
+          if (regmask & (1 << reg)) {
+            if (regmask_save & (1 << reg))
+              ferr(po, "%s already saved?\n", po->operand[0].name);
+            arg = OPF_RSAVE;
+          }
+          po->flags |= arg;
+          scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, arg);
+          continue;
+        }
       }
-      ret = scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, 0);
-      if (ret == 0) {
-        arg = OPF_RMD;
-        if (regmask & (1 << reg)) {
-          if (regmask_save & (1 << reg))
-            ferr(po, "%s already saved?\n", po->operand[0].name);
-          arg = OPF_RSAVE;
+      else if (po->operand[0].type == OPT_CONST) {
+        for (j = i + 1; j < opcnt; j++) {
+          if ((ops[j].flags & (OPF_JMP|OPF_TAIL|OPF_RSAVE))
+            || ops[j].op == OP_PUSH || g_labels[i][0] != 0)
+          {
+            break;
+          }
+
+          if (!(ops[j].flags & OPF_RMD) && ops[j].op == OP_POP)
+          {
+            po->flags |= OPF_RMD;
+            ops[j].datap = po;
+            break;
+          }
         }
-        po->flags |= arg;
-        scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, arg);
-        continue;
       }
     }
 
+    if (po->op == OP_STD) {
+      po->flags |= OPF_DF | OPF_RMD;
+      scan_propagate_df(i + 1, opcnt);
+    }
+
     regmask_now = po->regmask_src | po->regmask_dst;
     if (regmask_now & (1 << xBP)) {
       if (g_bp_frame && !(po->flags & OPF_EBP_S)) {
@@ -3174,13 +3327,14 @@ tailcall:
 
         // to get nicer code, we try to delay test and cmp;
         // if we can't because of operand modification, or if we
-        // have math op, or branch, make it calculate flags explicitly
-        if (tmp_op->op == OP_TEST || tmp_op->op == OP_CMP) {
+        // have arith op, or branch, make it calculate flags explicitly
+        if (tmp_op->op == OP_TEST || tmp_op->op == OP_CMP)
+        {
           if (branched || scan_for_mod(tmp_op, setters[j] + 1, i, 0) >= 0)
             pfomask = 1 << pfo;
         }
-        else if (tmp_op->op == OP_CMPS) {
-          pfomask = 1 << PFO_Z;
+        else if (tmp_op->op == OP_CMPS || tmp_op->op == OP_SCAS) {
+          pfomask = 1 << pfo;
         }
         else {
           // see if we'll be able to handle based on op result
@@ -3189,22 +3343,29 @@ tailcall:
               || branched
               || scan_for_mod_opr0(tmp_op, setters[j] + 1, i) >= 0)
             pfomask = 1 << pfo;
+
+          if (tmp_op->op == OP_ADD && pfo == PFO_C)
+            need_tmp64 = 1;
         }
         if (pfomask) {
           tmp_op->pfomask |= pfomask;
-          cmp_result_vars |= pfomask;
+          cond_vars |= pfomask;
         }
         // note: may overwrite, currently not a problem
         po->datap = tmp_op;
       }
 
-      if (po->op == OP_ADC || po->op == OP_SBB)
-        cmp_result_vars |= 1 << PFO_C;
+      if (po->op == OP_RCL || po->op == OP_RCR
+       || po->op == OP_ADC || po->op == OP_SBB)
+        cond_vars |= 1 << PFO_C;
+    }
+    else if (po->op == OP_CMPS || po->op == OP_SCAS) {
+      cond_vars |= 1 << PFO_Z;
     }
     else if (po->op == OP_MUL
       || (po->op == OP_IMUL && po->operand_cnt == 1))
     {
-      need_mul_var = 1;
+      need_tmp64 = 1;
     }
     else if (po->op == OP_CALL) {
       pp = po->datap;
@@ -3213,9 +3374,22 @@ tailcall:
 
       if (pp->is_unresolved) {
         int regmask_stack = 0;
-        collect_call_args(po, i, pp, &regmask_stack, &save_arg_vars,
+        collect_call_args(po, i, pp, &regmask, &save_arg_vars,
           i + opcnt * 2);
 
+        // this is pretty rough guess:
+        // see ecx and edx were pushed (and not their saved versions)
+        for (arg = 0; arg < pp->argc; arg++) {
+          if (pp->arg[arg].reg != NULL)
+            continue;
+
+          tmp_op = pp->arg[arg].datap;
+          if (tmp_op == NULL)
+            ferr(po, "parsed_op missing for arg%d\n", arg);
+          if (tmp_op->argnum == 0 && tmp_op->operand[0].type == OPT_REG)
+            regmask_stack |= 1 << tmp_op->operand[0].reg;
+        }
+
         if (!((regmask_stack & (1 << xCX))
           && (regmask_stack & (1 << xDX))))
         {
@@ -3235,7 +3409,6 @@ tailcall:
             regmask |= 1 << xDX;
           }
         }
-        regmask |= regmask_stack;
 
         // note: __cdecl doesn't fall into is_unresolved category
         if (pp->argc_stack > 0)
@@ -3289,6 +3462,21 @@ tailcall:
     }
     else if (po->op == OP_RET && !IS(g_func_pp->ret_type.name, "void"))
       regmask |= 1 << xAX;
+    else if (po->op == OP_DIV || po->op == OP_IDIV) {
+      // 32bit division is common, look for it
+      if (po->op == OP_DIV)
+        ret = scan_for_reg_clear(i, xDX);
+      else
+        ret = scan_for_cdq_edx(i);
+      if (ret >= 0)
+        po->flags |= OPF_32BIT;
+      else
+        need_tmp64 = 1;
+    }
+
+    if (po->op == OP_RCL || po->op == OP_RCR || po->op == OP_XCHG) {
+      need_tmp_var = 1;
+    }
   }
 
   // pass4:
@@ -3389,17 +3577,22 @@ tailcall:
     }
   }
 
-  if (cmp_result_vars) {
+  if (cond_vars) {
     for (i = 0; i < 8; i++) {
-      if (cmp_result_vars & (1 << i)) {
+      if (cond_vars & (1 << i)) {
         fprintf(fout, "  u32 cond_%s;\n", parsed_flag_op_names[i]);
         had_decl = 1;
       }
     }
   }
 
-  if (need_mul_var) {
-    fprintf(fout, "  u64 mul_tmp;\n");
+  if (need_tmp_var) {
+    fprintf(fout, "  u32 tmp;\n");
+    had_decl = 1;
+  }
+
+  if (need_tmp64) {
+    fprintf(fout, "  u64 tmp64;\n");
     had_decl = 1;
   }
 
@@ -3475,7 +3668,9 @@ tailcall:
       if (po->flags & OPF_JMP) {
         fprintf(fout, "  if %s\n", buf1);
       }
-      else if (po->op == OP_ADC || po->op == OP_SBB) {
+      else if (po->op == OP_RCL || po->op == OP_RCR
+               || po->op == OP_ADC || po->op == OP_SBB)
+      {
         if (is_delayed)
           fprintf(fout, "  cond_%s = %s;\n",
             parsed_flag_op_names[pfo], buf1);
@@ -3491,6 +3686,27 @@ tailcall:
 
     pfomask = po->pfomask;
 
+    if (po->flags & (OPF_REPZ|OPF_REPNZ)) {
+      // we need initial flags for ecx=0 case..
+      if (i > 0 && ops[i - 1].op == OP_XOR
+        && IS(ops[i - 1].operand[0].name,
+              ops[i - 1].operand[1].name))
+      {
+        fprintf(fout, "  cond_z = ");
+        if (pfomask & (1 << PFO_C))
+          fprintf(fout, "cond_c = ");
+        fprintf(fout, "0;\n");
+      }
+      else if (last_arith_dst != NULL) {
+        out_src_opr_u32(buf3, sizeof(buf3), po, last_arith_dst);
+        out_test_for_cc(buf1, sizeof(buf1), po, PFO_Z, 0,
+          last_arith_dst->lmod, buf3);
+        fprintf(fout, "  cond_z = %s;\n", buf1);
+      }
+      else
+        ferr(po, "missing initial ZF\n");
+    }
+
     switch (po->op)
     {
       case OP_MOV:
@@ -3536,6 +3752,19 @@ tailcall:
               buf3, 0));
         break;
 
+      case OP_XCHG:
+        assert_operand_cnt(2);
+        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        fprintf(fout, "  tmp = %s;",
+          out_src_opr(buf1, sizeof(buf1), po, &po->operand[0], "", 0));
+        fprintf(fout, " %s = %s;",
+          out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
+          out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], "", 0));
+        fprintf(fout, " %s = tmp;",
+          out_dst_opr(buf1, sizeof(buf1), po, &po->operand[1]));
+        snprintf(g_comment, sizeof(g_comment), "xchg");
+        break;
+
       case OP_NOT:
         assert_operand_cnt(1);
         out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
@@ -3551,65 +3780,73 @@ tailcall:
         break;
 
       case OP_STOS:
-        // assumes DF=0
         assert_operand_cnt(3);
         if (po->flags & OPF_REP) {
-          fprintf(fout, "  for (; ecx != 0; ecx--, edi += %d)\n",
+          fprintf(fout, "  for (; ecx != 0; ecx--, edi %c= %d)\n",
+            (po->flags & OPF_DF) ? '-' : '+',
             lmod_bytes(po, po->operand[0].lmod));
           fprintf(fout, "    %sedi = eax;",
             lmod_cast_u_ptr(po, po->operand[0].lmod));
           strcpy(g_comment, "rep stos");
         }
         else {
-          fprintf(fout, "    %sedi = eax; edi += %d;",
+          fprintf(fout, "    %sedi = eax; edi %c= %d;",
             lmod_cast_u_ptr(po, po->operand[0].lmod),
+            (po->flags & OPF_DF) ? '-' : '+',
             lmod_bytes(po, po->operand[0].lmod));
           strcpy(g_comment, "stos");
         }
         break;
 
       case OP_MOVS:
-        // assumes DF=0
         assert_operand_cnt(3);
         j = lmod_bytes(po, po->operand[0].lmod);
         strcpy(buf1, lmod_cast_u_ptr(po, po->operand[0].lmod));
+        l = (po->flags & OPF_DF) ? '-' : '+';
         if (po->flags & OPF_REP) {
           fprintf(fout,
-            "  for (; ecx != 0; ecx--, edi += %d, esi += %d)\n",
-            j, j);
+            "  for (; ecx != 0; ecx--, edi %c= %d, esi %c= %d)\n",
+            l, j, l, j);
           fprintf(fout,
             "    %sedi = %sesi;", buf1, buf1);
           strcpy(g_comment, "rep movs");
         }
         else {
-          fprintf(fout, "    %sedi = %sesi; edi += %d; esi += %d;",
-            buf1, buf1, j, j);
+          fprintf(fout, "    %sedi = %sesi; edi %c= %d; esi %c= %d;",
+            buf1, buf1, l, j, l, j);
           strcpy(g_comment, "movs");
         }
         break;
 
       case OP_CMPS:
-        // assumes DF=0
         // repe ~ repeat while ZF=1
         assert_operand_cnt(3);
         j = lmod_bytes(po, po->operand[0].lmod);
         strcpy(buf1, lmod_cast_u_ptr(po, po->operand[0].lmod));
+        l = (po->flags & OPF_DF) ? '-' : '+';
         if (po->flags & OPF_REP) {
           fprintf(fout,
-            "  for (; ecx != 0; ecx--, edi += %d, esi += %d)\n",
-            j, j);
+            "  for (; ecx != 0; ecx--, edi %c= %d, esi %c= %d) {\n",
+            l, j, l, j);
+          if (pfomask & (1 << PFO_C)) {
+            // ugh..
+            fprintf(fout,
+            "    cond_c = %sedi < %sesi;\n", buf1, buf1);
+            pfomask &= ~(1 << PFO_C);
+          }
           fprintf(fout,
             "    if ((cond_z = (%sedi == %sesi)) %s 0)\n",
               buf1, buf1, (po->flags & OPF_REPZ) ? "==" : "!=");
           fprintf(fout,
-            "      break;");
+            "      break;\n"
+            "  }");
           snprintf(g_comment, sizeof(g_comment), "rep%s cmps",
             (po->flags & OPF_REPZ) ? "e" : "ne");
         }
         else {
           fprintf(fout,
-            "    cond_z = (%sedi = %sesi); edi += %d; esi += %d;",
-            buf1, buf1, j, j);
+            "    cond_z = (%sedi = %sesi); edi %c= %d; esi %c= %d;",
+            buf1, buf1, l, j, l, j);
           strcpy(g_comment, "cmps");
         }
         pfomask &= ~(1 << PFO_Z);
@@ -3617,9 +3854,37 @@ tailcall:
         delayed_flag_op = NULL;
         break;
 
+      case OP_SCAS:
+        // only does ZF (for now)
+        // repe ~ repeat while ZF=1
+        assert_operand_cnt(3);
+        j = lmod_bytes(po, po->operand[0].lmod);
+        l = (po->flags & OPF_DF) ? '-' : '+';
+        if (po->flags & OPF_REP) {
+          fprintf(fout,
+            "  for (; ecx != 0; ecx--, edi %c= %d)\n", l, j);
+          fprintf(fout,
+            "    if ((cond_z = (%seax == %sedi)) %s 0)\n",
+              lmod_cast_u(po, po->operand[0].lmod),
+              lmod_cast_u_ptr(po, po->operand[0].lmod),
+              (po->flags & OPF_REPZ) ? "==" : "!=");
+          fprintf(fout,
+            "      break;");
+          snprintf(g_comment, sizeof(g_comment), "rep%s scas",
+            (po->flags & OPF_REPZ) ? "e" : "ne");
+        }
+        else {
+          fprintf(fout, "    cond_z = (%seax = %sedi); edi %c= %d;",
+              lmod_cast_u(po, po->operand[0].lmod),
+              lmod_cast_u_ptr(po, po->operand[0].lmod), l, j);
+          strcpy(g_comment, "scas");
+        }
+        pfomask &= ~(1 << PFO_Z);
+        last_arith_dst = NULL;
+        delayed_flag_op = NULL;
+        break;
+
       // arithmetic w/flags
-      case OP_ADD:
-      case OP_SUB:
       case OP_AND:
       case OP_OR:
         propagate_lmod(po, &po->operand[0], &po->operand[1]);
@@ -3649,8 +3914,8 @@ tailcall:
                 j = l - j;
               else
                 j -= 1;
-              fprintf(fout, "  cond_c = (%s & 0x%02x) ? 1 : 0;\n",
-                buf1, 1 << j);
+              fprintf(fout, "  cond_c = (%s >> %d) & 1;\n",
+                buf1, j);
             }
             else
               ferr(po, "zero shift?\n");
@@ -3697,11 +3962,51 @@ tailcall:
         delayed_flag_op = NULL;
         break;
 
+      case OP_RCL:
+      case OP_RCR:
+        assert_operand_cnt(2);
+        out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+        l = lmod_bytes(po, po->operand[0].lmod) * 8;
+        if (po->operand[1].type == OPT_CONST) {
+          j = po->operand[1].val % l;
+          if (j == 0)
+            ferr(po, "zero rotate\n");
+          fprintf(fout, "  tmp = (%s >> %d) & 1;\n",
+            buf1, (po->op == OP_RCL) ? (l - j) : (j - 1));
+          if (po->op == OP_RCL) {
+            fprintf(fout,
+              "  %s = (%s << %d) | (cond_c << %d)",
+              buf1, buf1, j, j - 1);
+            if (j != 1)
+              fprintf(fout, " | (%s >> %d)", buf1, l + 1 - j);
+          }
+          else {
+            fprintf(fout,
+              "  %s = (%s >> %d) | (cond_c << %d)",
+              buf1, buf1, j, l - j);
+            if (j != 1)
+              fprintf(fout, " | (%s << %d)", buf1, l + 1 - j);
+          }
+          fprintf(fout, ";\n");
+          fprintf(fout, "  cond_c = tmp;");
+        }
+        else
+          ferr(po, "TODO\n");
+        strcpy(g_comment, (po->op == OP_RCL) ? "rcl" : "rcr");
+        output_std_flags(fout, po, &pfomask, buf1);
+        last_arith_dst = &po->operand[0];
+        delayed_flag_op = NULL;
+        break;
+
       case OP_XOR:
         assert_operand_cnt(2);
         propagate_lmod(po, &po->operand[0], &po->operand[1]);
         if (IS(opr_name(po, 0), opr_name(po, 1))) {
           // special case for XOR
+          if (pfomask & (1 << PFO_BE)) { // weird, but it happens..
+            fprintf(fout, "  cond_be = 1;\n");
+            pfomask &= ~(1 << PFO_BE);
+          }
           fprintf(fout, "  %s = 0;",
             out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]));
           last_arith_dst = &po->operand[0];
@@ -3710,6 +4015,36 @@ tailcall:
         }
         goto dualop_arith;
 
+      case OP_ADD:
+        assert_operand_cnt(2);
+        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        if (pfomask & (1 << PFO_C)) {
+          fprintf(fout, "  tmp64 = (u64)%s + %s;\n",
+            out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]),
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
+          fprintf(fout, "  cond_c = tmp64 >> 32;\n");
+          fprintf(fout, "  %s = (u32)tmp64;",
+            out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]));
+          strcat(g_comment, "add64");
+          pfomask &= ~(1 << PFO_C);
+          output_std_flags(fout, po, &pfomask, buf1);
+          last_arith_dst = &po->operand[0];
+          delayed_flag_op = NULL;
+          break;
+        }
+        goto dualop_arith;
+
+      case OP_SUB:
+        assert_operand_cnt(2);
+        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        if (pfomask & (1 << PFO_C)) {
+          fprintf(fout, "  cond_c = %s < %s;\n",
+            out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]),
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
+          pfomask &= ~(1 << PFO_C);
+        }
+        goto dualop_arith;
+
       case OP_ADC:
       case OP_SBB:
         assert_operand_cnt(2);
@@ -3772,10 +4107,10 @@ tailcall:
       case OP_MUL:
         assert_operand_cnt(1);
         strcpy(buf1, po->op == OP_IMUL ? "(s64)(s32)" : "(u64)");
-        fprintf(fout, "  mul_tmp = %seax * %s%s;\n", buf1, buf1,
+        fprintf(fout, "  tmp64 = %seax * %s%s;\n", buf1, buf1,
           out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[0]));
-        fprintf(fout, "  edx = mul_tmp >> 32;\n");
-        fprintf(fout, "  eax = mul_tmp;");
+        fprintf(fout, "  edx = tmp64 >> 32;\n");
+        fprintf(fout, "  eax = tmp64;");
         last_arith_dst = NULL;
         delayed_flag_op = NULL;
         break;
@@ -3786,20 +4121,32 @@ tailcall:
         if (po->operand[0].lmod != OPLM_DWORD)
           ferr(po, "unhandled lmod %d\n", po->operand[0].lmod);
 
-        // 32bit division is common, look for it
-        if (po->op == OP_DIV)
-          ret = scan_for_reg_clear(i, xDX);
-        else
-          ret = scan_for_cdq_edx(i);
-        if (ret >= 0) {
-          out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]);
-          strcpy(buf2, lmod_cast(po, po->operand[0].lmod,
-            po->op == OP_IDIV));
-          fprintf(fout, "  edx = %seax %% %s%s;\n", buf2, buf2, buf1);
-          fprintf(fout, "  eax = %seax / %s%s;", buf2, buf2, buf1);
+        out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]);
+        strcpy(buf2, lmod_cast(po, po->operand[0].lmod,
+          po->op == OP_IDIV));
+        switch (po->operand[0].lmod) {
+        case OPLM_DWORD:
+          if (po->flags & OPF_32BIT)
+            snprintf(buf3, sizeof(buf3), "%seax", buf2);
+          else {
+            fprintf(fout, "  tmp64 = ((u64)edx << 32) | eax;\n");
+            snprintf(buf3, sizeof(buf3), "%stmp64",
+              (po->op == OP_IDIV) ? "(s64)" : "");
+          }
+          if (po->operand[0].type == OPT_REG
+            && po->operand[0].reg == xDX)
+          {
+            fprintf(fout, "  eax = %s / %s%s;", buf3, buf2, buf1);
+            fprintf(fout, "  edx = %s %% %s%s;\n", buf3, buf2, buf1);
+          }
+          else {
+            fprintf(fout, "  edx = %s %% %s%s;\n", buf3, buf2, buf1);
+            fprintf(fout, "  eax = %s / %s%s;", buf3, buf2, buf1);
+          }
+          break;
+        default:
+          ferr(po, "unhandled division type\n");
         }
-        else
-          ferr(po, "TODO 64bit divident\n");
         last_arith_dst = NULL;
         delayed_flag_op = NULL;
         break;
@@ -3830,6 +4177,12 @@ tailcall:
         // else SETcc - should already be handled
         break;
 
+      case OP_JECXZ:
+        fprintf(fout, "  if (ecx == 0)\n");
+        fprintf(fout, "    goto %s;", po->operand[0].name);
+        strcat(g_comment, "jecxz");
+        break;
+
       case OP_JMP:
         assert_operand_cnt(1);
         last_arith_dst = NULL;
@@ -3864,7 +4217,7 @@ tailcall:
         if (strstr(pp->ret_type.name, "int64")) {
           if (po->flags & OPF_TAIL)
             ferr(po, "int64 and tail?\n");
-          fprintf(fout, "mul_tmp = ");
+          fprintf(fout, "tmp64 = ");
         }
         else if (!IS(pp->ret_type.name, "void")) {
           if (po->flags & OPF_TAIL) {
@@ -3886,38 +4239,68 @@ tailcall:
         fprintf(fout, "%s%s(", pp->name,
           pp->has_structarg ? "_sa" : "");
 
-        for (arg = 0; arg < pp->argc; arg++) {
-          if (arg > 0)
-            fprintf(fout, ", ");
+        if (po->flags & OPF_ATAIL) {
+          if (pp->argc_stack != g_func_pp->argc_stack
+            || (pp->argc_stack > 0
+                && pp->is_stdcall != g_func_pp->is_stdcall))
+            ferr(po, "incompatible tailcall\n");
 
-          cast[0] = 0;
-          if (pp->arg[arg].type.is_ptr)
-            snprintf(cast, sizeof(cast), "(%s)", pp->arg[arg].type.name);
+          for (arg = j = 0; arg < pp->argc; arg++) {
+            if (arg > 0)
+              fprintf(fout, ", ");
 
-          if (pp->arg[arg].reg != NULL) {
-            fprintf(fout, "%s%s", cast, pp->arg[arg].reg);
-            continue;
-          }
+            cast[0] = 0;
+            if (pp->arg[arg].type.is_ptr)
+              snprintf(cast, sizeof(cast), "(%s)",
+                pp->arg[arg].type.name);
 
-          // stack arg
-          tmp_op = pp->arg[arg].datap;
-          if (tmp_op == NULL)
-            ferr(po, "parsed_op missing for arg%d\n", arg);
-          if (tmp_op->argnum != 0) {
-            fprintf(fout, "%ss_a%d", cast, tmp_op->argnum);
+            if (pp->arg[arg].reg != NULL) {
+              fprintf(fout, "%s%s", cast, pp->arg[arg].reg);
+              continue;
+            }
+            // stack arg
+            for (; j < g_func_pp->argc; j++)
+              if (g_func_pp->arg[j].reg == NULL)
+                break;
+            fprintf(fout, "%sa%d", cast, j + 1);
+            j++;
           }
-          else {
-            fprintf(fout, "%s",
-              out_src_opr(buf1, sizeof(buf1),
-                tmp_op, &tmp_op->operand[0], cast, 0));
+        }
+        else {
+          for (arg = 0; arg < pp->argc; arg++) {
+            if (arg > 0)
+              fprintf(fout, ", ");
+
+            cast[0] = 0;
+            if (pp->arg[arg].type.is_ptr)
+              snprintf(cast, sizeof(cast), "(%s)",
+                pp->arg[arg].type.name);
+
+            if (pp->arg[arg].reg != NULL) {
+              fprintf(fout, "%s%s", cast, pp->arg[arg].reg);
+              continue;
+            }
+
+            // stack arg
+            tmp_op = pp->arg[arg].datap;
+            if (tmp_op == NULL)
+              ferr(po, "parsed_op missing for arg%d\n", arg);
+            if (tmp_op->argnum != 0) {
+              fprintf(fout, "%ss_a%d", cast, tmp_op->argnum);
+            }
+            else {
+              fprintf(fout, "%s",
+                out_src_opr(buf1, sizeof(buf1),
+                  tmp_op, &tmp_op->operand[0], cast, 0));
+            }
           }
         }
         fprintf(fout, ");");
 
         if (strstr(pp->ret_type.name, "int64")) {
           fprintf(fout, "\n");
-          fprintf(fout, "  edx = mul_tmp >> 32;\n");
-          fprintf(fout, "  eax = mul_tmp;");
+          fprintf(fout, "  edx = tmp64 >> 32;\n");
+          fprintf(fout, "  eax = tmp64;");
         }
 
         if (pp->is_unresolved) {
@@ -3950,6 +4333,8 @@ tailcall:
         }
         if (pp->is_noreturn)
           strcat(g_comment, " noreturn");
+        if ((po->flags & OPF_ATAIL) && pp->argc_stack > 0)
+          strcat(g_comment, " argframe");
         delayed_flag_op = NULL;
         last_arith_dst = NULL;
         break;
@@ -3966,6 +4351,8 @@ tailcall:
           fprintf(fout, "  return (%s)eax;",
             g_func_pp->ret_type.name);
         }
+        else if (IS(g_func_pp->ret_type.name, "__int64"))
+          fprintf(fout, "  return ((u64)edx << 32) | eax;");
         else
           fprintf(fout, "  return eax;");
 
@@ -3996,6 +4383,16 @@ tailcall:
           fprintf(fout, "  %s = s_%s;", buf1, buf1);
           break;
         }
+        else if (po->datap != NULL) {
+          // push/pop pair
+          tmp_op = po->datap;
+          out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+          fprintf(fout, "  %s = %s;", buf1,
+            out_src_opr(buf2, sizeof(buf2),
+              tmp_op, &tmp_op->operand[0],
+              po->operand[0].is_ptr ? "(void *)" : "", 0));
+          break;
+        }
         ferr(po, "stray pop encountered\n");
         break;
 
@@ -4022,10 +4419,16 @@ tailcall:
       fprintf(fout, "\n");
 
     // some sanity checking
-    if ((po->flags & OPF_REP) && po->op != OP_STOS
-        && po->op != OP_MOVS && po->op != OP_CMPS)
-      ferr(po, "unexpected rep\n");
-    if ((po->flags & (OPF_REPZ|OPF_REPNZ)) && po->op != OP_CMPS)
+    if (po->flags & OPF_REP) {
+      if (po->op != OP_STOS && po->op != OP_MOVS
+          && po->op != OP_CMPS && po->op != OP_SCAS)
+        ferr(po, "unexpected rep\n");
+      if (!(po->flags & (OPF_REPZ|OPF_REPNZ))
+          && (po->op == OP_CMPS || po->op == OP_SCAS))
+        ferr(po, "cmps/scas with plain rep\n");
+    }
+    if ((po->flags & (OPF_REPZ|OPF_REPNZ))
+        && po->op != OP_CMPS && po->op != OP_SCAS)
       ferr(po, "unexpected repz/repnz\n");
 
     if (pfomask != 0)