header pre-parsing
[ia32rtools.git] / tools / translate.c
index da1c8f6..7cbf8b7 100644 (file)
@@ -1,3 +1,4 @@
+#define _GNU_SOURCE
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -7,29 +8,41 @@
 
 #define ARRAY_SIZE(x) (sizeof(x) / sizeof(x[0]))
 #define IS(w, y) !strcmp(w, y)
+#define IS_START(w, y) !strncmp(w, y, strlen(y))
 
 #include "protoparse.h"
 
-const char *asmfn;
+static const char *asmfn;
 static int asmln;
+static FILE *g_fhdr;
 
+#define anote(fmt, ...) \
+       printf("%s:%d: note: " fmt, asmfn, asmln, ##__VA_ARGS__)
 #define awarn(fmt, ...) \
-       printf("warning:%s:%d: " fmt, asmfn, asmln, ##__VA_ARGS__)
+       printf("%s:%d: warning: " fmt, asmfn, asmln, ##__VA_ARGS__)
 #define aerr(fmt, ...) do { \
-       printf("error:%s:%d: " fmt, asmfn, asmln, ##__VA_ARGS__); \
+       printf("%s:%d: error: " fmt, asmfn, asmln, ##__VA_ARGS__); \
+  fcloseall(); \
        exit(1); \
 } while (0)
 
 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_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) */
 };
 
 enum op_op {
        OP_INVAL,
+       OP_NOP,
        OP_PUSH,
        OP_POP,
        OP_MOV,
@@ -37,6 +50,10 @@ enum op_op {
        OP_MOVZX,
        OP_MOVSX,
        OP_NOT,
+       OP_CDQ,
+       OP_STOS,
+       OP_MOVS,
+       OP_CMPS,
        OP_RET,
        OP_ADD,
        OP_SUB,
@@ -46,12 +63,17 @@ enum op_op {
        OP_SHL,
        OP_SHR,
        OP_SAR,
+       OP_ROL,
+       OP_ROR,
        OP_ADC,
        OP_SBB,
        OP_INC,
        OP_DEC,
+       OP_NEG,
        OP_MUL,
        OP_IMUL,
+       OP_DIV,
+       OP_IDIV,
        OP_TEST,
        OP_CMP,
        OP_CALL,
@@ -75,14 +97,15 @@ enum op_op {
 };
 
 enum opr_type {
-       OPT_UNSPEC,
-       OPT_REG,
-       OPT_REGMEM,
-       OPT_LABEL,
+  OPT_UNSPEC,
+  OPT_REG,
+  OPT_REGMEM,
+  OPT_LABEL,
   OPT_OFFSET,
-       OPT_CONST,
+  OPT_CONST,
 };
 
+// must be sorted (larger len must be further in enum)
 enum opr_lenmod {
        OPLM_UNSPEC,
        OPLM_BYTE,
@@ -95,6 +118,10 @@ enum opr_lenmod {
 struct parsed_opr {
   enum opr_type type;
   enum opr_lenmod lmod;
+  unsigned int is_ptr:1;   // pointer in C
+  unsigned int is_array:1; // array in C
+  unsigned int size_mismatch:1; // type override differs from C
+  unsigned int size_lt:1;  // type override is larger than C
   int reg;
   unsigned int val;
   char name[256];
@@ -107,14 +134,17 @@ struct parsed_op {
   int operand_cnt;
   int regmask_src;        // all referensed regs
   int regmask_dst;
-  int pfomask;            // parsed_flag_op that can't be delayed
+  int pfomask;            // flagop: parsed_flag_op that can't be delayed
+  int argnum;             // push: altered before call arg #
+  int cc_scratch;         // scratch storage during analysis
+  int bt_i;               // branch target for branches
+  struct parsed_data *btj;// branch targets for jumptables
   void *datap;
 };
 
 // datap:
-// OP_PUSH - arg number if arg is altered before call
 // OP_CALL - ptr to parsed_proto
-// (OPF_CC) - point to corresponding (OPF_FLAGS)
+// (OPF_CC) - point to one of (OPF_FLAGS) that affects cc op
 
 struct parsed_equ {
   char name[64];
@@ -122,22 +152,60 @@ struct parsed_equ {
   int offset;
 };
 
-#define MAX_OPS 1024
+struct parsed_data {
+  char label[256];
+  enum opr_type type;
+  enum opr_lenmod lmod;
+  int count;
+  int count_alloc;
+  struct {
+    union {
+      char *label;
+      unsigned int val;
+    } u;
+    int bt_i;
+  } *d;
+};
+
+struct label_ref {
+  int i;
+  struct label_ref *next;
+};
+
+enum ida_func_attr {
+  IDAFA_BP_FRAME = (1 << 0),
+  IDAFA_LIB_FUNC = (1 << 1),
+  IDAFA_STATIC   = (1 << 2),
+  IDAFA_NORETURN = (1 << 3),
+  IDAFA_THUNK    = (1 << 4),
+  IDAFA_FPD      = (1 << 5),
+};
+
+#define MAX_OPS 4096
 
 static struct parsed_op ops[MAX_OPS];
 static struct parsed_equ *g_eqs;
 static int g_eqcnt;
 static char g_labels[MAX_OPS][32];
-static struct parsed_proto g_func_pp;
+static struct label_ref g_label_refs[MAX_OPS];
+static const struct parsed_proto *g_func_pp;
+static struct parsed_data *g_func_pd;
+static int g_func_pd_cnt;
 static char g_func[256];
 static char g_comment[256];
 static int g_bp_frame;
-static int g_bp_stack;
+static int g_sp_frame;
+static int g_stack_fsz;
+static int g_ida_func_attr;
 #define ferr(op_, fmt, ...) do { \
-  printf("error:%s:#%ld: '%s': " fmt, g_func, (op_) - ops, \
+  printf("error:%s:#%zd: '%s': " fmt, g_func, (op_) - ops, \
     dump_op(op_), ##__VA_ARGS__); \
+  fcloseall(); \
   exit(1); \
 } while (0)
+#define fnote(op_, fmt, ...) \
+  printf("error:%s:#%zd: '%s': " fmt, g_func, (op_) - ops, \
+    dump_op(op_), ##__VA_ARGS__)
 
 #define MAX_REGS 8
 
@@ -175,64 +243,49 @@ static int char_array_i(const char *array[], size_t len, const char *s)
   return -1;
 }
 
-static int parse_reg(int *reg_out, enum opr_lenmod *reg_lmod,
-  int *regmask, char *s)
+static void printf_number(char *buf, size_t buf_size,
+  unsigned long number)
 {
-  char w[16];
-  int reg = xUNSPEC;
-  int c = 0;
+  // output in C-friendly form
+  snprintf(buf, buf_size, number < 10 ? "%lu" : "0x%02lx", number);
+}
 
-  while (*s != 0) {
-    while (my_isblank(*s) || my_issep(*s))
-      s++;
-    s = next_idt(w, sizeof(w), s);
-    if (w[0] == 0)
-      break;
-    c++;
-    reg = char_array_i(regs_r32, ARRAY_SIZE(regs_r32), w);
-    if (reg >= 0) {
-      *reg_lmod = OPLM_DWORD;
-      *regmask |= 1 << reg;
-      continue;
-    }
-    reg = char_array_i(regs_r16, ARRAY_SIZE(regs_r16), w);
-    if (reg >= 0) {
-      *reg_lmod = OPLM_WORD;
-      *regmask |= 1 << reg;
-      continue;
-    }
-    reg = char_array_i(regs_r8h, ARRAY_SIZE(regs_r8h), w);
-    if (reg >= 0) {
-      *reg_lmod = OPLM_BYTE;
-      *regmask |= 1 << reg;
-      continue;
-    }
-    reg = char_array_i(regs_r8l, ARRAY_SIZE(regs_r8l), w);
-    if (reg >= 0) {
-      *reg_lmod = OPLM_BYTE;
-      *regmask |= 1 << reg;
-      continue;
-    }
+static int parse_reg(enum opr_lenmod *reg_lmod, const char *s)
+{
+  int reg;
 
-    return -1;
+  reg = char_array_i(regs_r32, ARRAY_SIZE(regs_r32), s);
+  if (reg >= 0) {
+    *reg_lmod = OPLM_DWORD;
+    return reg;
   }
-
-  if (c == 1) {
-    *reg_out = reg;
-    return 0;
+  reg = char_array_i(regs_r16, ARRAY_SIZE(regs_r16), s);
+  if (reg >= 0) {
+    *reg_lmod = OPLM_WORD;
+    return reg;
+  }
+  reg = char_array_i(regs_r8h, ARRAY_SIZE(regs_r8h), s);
+  if (reg >= 0) {
+    *reg_lmod = OPLM_BYTE;
+    return reg;
+  }
+  reg = char_array_i(regs_r8l, ARRAY_SIZE(regs_r8l), s);
+  if (reg >= 0) {
+    *reg_lmod = OPLM_BYTE;
+    return reg;
   }
 
   return -1;
 }
 
-static long parse_number(const char *number)
+static unsigned long parse_number(const char *number)
 {
   int len = strlen(number);
   const char *p = number;
   char *endp = NULL;
+  unsigned long ret;
   int neg = 0;
   int bad;
-  long ret;
 
   if (*p == '-') {
     neg = 1;
@@ -241,11 +294,11 @@ static long parse_number(const char *number)
   if (len > 1 && *p == '0')
     p++;
   if (number[len - 1] == 'h') {
-    ret = strtol(p, &endp, 16);
+    ret = strtoul(p, &endp, 16);
     bad = (*endp != 'h');
   }
   else {
-    ret = strtol(p, &endp, 10);
+    ret = strtoul(p, &endp, 10);
     bad = (*endp != 0);
   }
   if (bad)
@@ -255,6 +308,145 @@ static long parse_number(const char *number)
   return ret;
 }
 
+static int parse_indmode(char *name, int *regmask, int need_c_cvt)
+{
+  enum opr_lenmod lmod;
+  char cvtbuf[256];
+  char *d = cvtbuf;
+  char *s = name;
+  char w[64];
+  long number;
+  int reg;
+  int c = 0;
+
+  *d = 0;
+
+  while (*s != 0) {
+    d += strlen(d);
+    while (my_isblank(*s))
+      s++;
+    for (; my_issep(*s); d++, s++)
+      *d = *s;
+    while (my_isblank(*s))
+      s++;
+    *d = 0;
+
+    // skip 'ds:' prefix
+    if (IS_START(s, "ds:"))
+      s += 3;
+
+    s = next_idt(w, sizeof(w), s);
+    if (w[0] == 0)
+      break;
+    c++;
+
+    reg = parse_reg(&lmod, w);
+    if (reg >= 0) {
+      *regmask |= 1 << reg;
+      goto pass;
+    }
+
+    if ('0' <= w[0] && w[0] <= '9') {
+      number = parse_number(w);
+      printf_number(d, sizeof(cvtbuf) - (d - cvtbuf), number);
+      continue;
+    }
+
+    // probably some label/identifier - pass
+
+pass:
+    snprintf(d, sizeof(cvtbuf) - (d - cvtbuf), "%s", w);
+  }
+
+  if (need_c_cvt)
+    strcpy(name, cvtbuf);
+
+  return c;
+}
+
+static int is_reg_in_str(const char *s)
+{
+  int i;
+
+  if (strlen(s) < 3 || (s[3] && !my_issep(s[3]) && !my_isblank(s[3])))
+    return 0;
+
+  for (i = 0; i < ARRAY_SIZE(regs_r32); i++)
+    if (!strncmp(s, regs_r32[i], 3))
+      return 1;
+
+  return 0;
+}
+
+static const char *parse_stack_el(const char *name, char *extra_reg)
+{
+  const char *p, *p2, *s;
+  char *endp = NULL;
+  char buf[32];
+  long val;
+  int len;
+
+  p = name;
+  if (IS_START(p + 3, "+ebp+") && is_reg_in_str(p)) {
+    p += 4;
+    if (extra_reg != NULL) {
+      strncpy(extra_reg, name, 3);
+      extra_reg[4] = 0;
+    }
+  }
+
+  if (IS_START(p, "ebp+")) {
+    p += 4;
+
+    p2 = strchr(p, '+');
+    if (p2 != NULL && is_reg_in_str(p)) {
+      if (extra_reg != NULL) {
+        strncpy(extra_reg, p, p2 - p);
+        extra_reg[p2 - p] = 0;
+      }
+      p = p2 + 1;
+    }
+
+    if (!('0' <= *p && *p <= '9'))
+      return p;
+
+    return NULL;
+  }
+
+  if (!IS_START(name, "esp+"))
+    return NULL;
+
+  p = strchr(name + 4, '+');
+  if (p) {
+    // must be a number after esp+, already converted to 0x..
+    s = name + 4;
+    if (!('0' <= *s && *s <= '9')) {
+                 aerr("%s nan?\n", __func__);
+      return NULL;
+    }
+    if (s[0] == '0' && s[1] == 'x')
+      s += 2;
+    len = p - s;
+    if (len < sizeof(buf) - 1) {
+      strncpy(buf, s, len);
+      buf[len] = 0;
+      val = strtol(buf, &endp, 16);
+      if (val == 0 || *endp != 0) {
+        aerr("%s num parse fail for '%s'\n", __func__, buf);
+        return NULL;
+      }
+    }
+    p++;
+  }
+  else
+    p = name + 4;
+
+  if ('0' <= *p && *p <= '9')
+    return NULL;
+
+  return p;
+}
+
 static int guess_lmod_from_name(struct parsed_opr *opr)
 {
   if (!strncmp(opr->name, "dword_", 6)) {
@@ -272,78 +464,186 @@ static int guess_lmod_from_name(struct parsed_opr *opr)
   return 0;
 }
 
+static int guess_lmod_from_c_type(enum opr_lenmod *lmod,
+  const struct parsed_type *c_type)
+{
+  static const char *dword_types[] = {
+    "int", "_DWORD", "DWORD", "HANDLE", "HWND", "HMODULE",
+    "WPARAM", "UINT",
+  };
+  static const char *word_types[] = {
+    "uint16_t", "int16_t",
+    "unsigned __int16", "__int16",
+  };
+  static const char *byte_types[] = {
+    "uint8_t", "int8_t", "char",
+    "unsigned __int8", "__int8", "BYTE",
+  };
+  const char *n;
+  int i;
+
+  if (c_type->is_ptr) {
+    *lmod = OPLM_DWORD;
+    return 1;
+  }
+
+  n = skip_type_mod(c_type->name);
+
+  for (i = 0; i < ARRAY_SIZE(dword_types); i++) {
+    if (IS(n, dword_types[i])) {
+      *lmod = OPLM_DWORD;
+      return 1;
+    }
+  }
+
+  for (i = 0; i < ARRAY_SIZE(word_types); i++) {
+    if (IS(n, word_types[i])) {
+      *lmod = OPLM_WORD;
+      return 1;
+    }
+  }
+
+  for (i = 0; i < ARRAY_SIZE(byte_types); i++) {
+    if (IS(n, byte_types[i])) {
+      *lmod = OPLM_BYTE;
+      return 1;
+    }
+  }
+
+  return 0;
+}
+
+static enum opr_type lmod_from_directive(const char *d)
+{
+  if (IS(d, "dd"))
+    return OPLM_DWORD;
+  else if (IS(d, "dw"))
+    return OPLM_WORD;
+  else if (IS(d, "db"))
+    return OPLM_BYTE;
+
+  aerr("unhandled directive: '%s'\n", d);
+  return OPLM_UNSPEC;
+}
+
+static void setup_reg_opr(struct parsed_opr *opr, int reg, enum opr_lenmod lmod,
+  int *regmask)
+{
+  opr->type = OPT_REG;
+  opr->reg = reg;
+  opr->lmod = lmod;
+  *regmask |= 1 << reg;
+}
+
+static struct parsed_equ *equ_find(struct parsed_op *po, const char *name,
+  int *extra_offs);
+
 static int parse_operand(struct parsed_opr *opr,
   int *regmask, int *regmask_indirect,
-       char words[16][256], int wordc, int w, unsigned int op_flags)
+  char words[16][256], int wordc, int w, unsigned int op_flags)
 {
+  const struct parsed_proto *pp;
   enum opr_lenmod tmplmod;
-  int tmpreg;
+  unsigned long number;
   int ret, len;
+  int wordc_in;
+  char *tmp;
   int i;
 
-       if (w >= wordc)
-               aerr("parse_operand w %d, wordc %d\n", w, wordc);
+  if (w >= wordc)
+    aerr("parse_operand w %d, wordc %d\n", w, wordc);
 
-       opr->reg = xUNSPEC;
+  opr->reg = xUNSPEC;
 
-       for (i = w; i < wordc; i++) {
-               len = strlen(words[i]);
-               if (words[i][len - 1] == ',') {
-                       words[i][len - 1] = 0;
-                       wordc = i + 1;
-                       break;
-               }
-       }
+  for (i = w; i < wordc; i++) {
+    len = strlen(words[i]);
+    if (words[i][len - 1] == ',') {
+      words[i][len - 1] = 0;
+      wordc = i + 1;
+      break;
+    }
+  }
 
-       if (op_flags & OPF_JMP) {
-               const char *label;
-
-               if (wordc - w == 3 && IS(words[w + 1], "ptr"))
-                       label = words[w + 2];
-               else if (wordc - w == 2 && IS(words[w], "short"))
-                       label = words[w + 1];
-               else if (wordc - w == 1)
-                       label = words[w];
-               else
-                       aerr("jump parse error");
-
-               opr->type = OPT_LABEL;
-               strcpy(opr->name, label);
-               return wordc;
-       }
+  wordc_in = wordc - w;
 
-       if (wordc - w >= 3) {
-               if (IS(words[w + 1], "ptr")) {
-                       if (IS(words[w], "dword"))
-                               opr->lmod = OPLM_DWORD;
-                       else if (IS(words[w], "word"))
-                               opr->lmod = OPLM_WORD;
-                       else if (IS(words[w], "byte"))
-                               opr->lmod = OPLM_BYTE;
-                       else
-                               aerr("type parsing failed\n");
-                       w += 2;
-               }
-       }
+  if ((op_flags & OPF_JMP) && wordc_in > 0
+      && !('0' <= words[w][0] && words[w][0] <= '9'))
+  {
+    const char *label = NULL;
+
+    if (wordc_in == 3 && !strncmp(words[w], "near", 4)
+     && IS(words[w + 1], "ptr"))
+      label = words[w + 2];
+    else if (wordc_in == 2 && IS(words[w], "short"))
+      label = words[w + 1];
+    else if (wordc_in == 1
+          && strchr(words[w], '[') == NULL
+          && parse_reg(&tmplmod, words[w]) < 0)
+      label = words[w];
+
+    if (label != NULL) {
+      opr->type = OPT_LABEL;
+      if (IS_START(label, "ds:"))
+        label += 3;
+      strcpy(opr->name, label);
+      return wordc;
+    }
+  }
 
-       if (wordc - w == 2 && IS(words[w], "offset")) {
-               opr->type = OPT_OFFSET;
-               strcpy(opr->name, words[w + 1]);
-               return wordc;
-       }
+  if (wordc_in >= 3) {
+    if (IS(words[w + 1], "ptr")) {
+      if (IS(words[w], "dword"))
+        opr->lmod = OPLM_DWORD;
+      else if (IS(words[w], "word"))
+        opr->lmod = OPLM_WORD;
+      else if (IS(words[w], "byte"))
+        opr->lmod = OPLM_BYTE;
+      else
+        aerr("type parsing failed\n");
+      w += 2;
+      wordc_in = wordc - w;
+    }
+  }
+
+  if (wordc_in == 2) {
+    if (IS(words[w], "offset")) {
+      opr->type = OPT_OFFSET;
+      strcpy(opr->name, words[w + 1]);
+      return wordc;
+    }
+    if (IS(words[w], "(offset")) {
+      char *p = strchr(words[w + 1], ')');
+      if (p == NULL)
+        aerr("parse of bracketed offset failed\n");
+      *p = 0;
+      opr->type = OPT_OFFSET;
+      strcpy(opr->name, words[w + 1]);
+      return wordc;
+    }
+  }
 
-  if (wordc - w != 1)
+  if (wordc_in != 1)
     aerr("parse_operand 1 word expected\n");
 
-  strcpy(opr->name, words[w]);
+  tmp = words[w];
+  if (IS_START(tmp, "ds:"))
+    tmp += 3;
+  strcpy(opr->name, tmp);
 
   if (words[w][0] == '[') {
     opr->type = OPT_REGMEM;
     ret = sscanf(words[w], "[%[^]]]", opr->name);
     if (ret != 1)
       aerr("[] parse failure\n");
-    // only need the regmask
-    parse_reg(&tmpreg, &tmplmod, regmask_indirect, opr->name);
+
+    parse_indmode(opr->name, regmask_indirect, 1);
+    if (opr->lmod == OPLM_UNSPEC && parse_stack_el(opr->name, NULL)) {
+      // might be an equ
+      struct parsed_equ *eq =
+        equ_find(NULL, parse_stack_el(opr->name, NULL), &i);
+      if (eq)
+        opr->lmod = eq->lmod;
+    }
     return wordc;
   }
   else if (strchr(words[w], '[')) {
@@ -351,36 +651,67 @@ static int parse_operand(struct parsed_opr *opr,
     opr->type = OPT_REGMEM;
     if (opr->lmod == OPLM_UNSPEC)
       guess_lmod_from_name(opr);
-    parse_reg(&tmpreg, &tmplmod, regmask_indirect,
-      strchr(words[w], '['));
+    parse_indmode(strchr(words[w], '['), regmask_indirect, 0);
     return wordc;
   }
   else if (('0' <= words[w][0] && words[w][0] <= '9')
     || words[w][0] == '-')
   {
+    number = parse_number(words[w]);
     opr->type = OPT_CONST;
-    opr->val = (unsigned int)parse_number(words[w]);
+    opr->val = number;
+    printf_number(opr->name, sizeof(opr->name), number);
     return wordc;
   }
 
-  ret = parse_reg(&opr->reg, &tmplmod, regmask, opr->name);
-  if (ret == 0) {
-    opr->type = OPT_REG;
-    opr->lmod = tmplmod;
+  ret = parse_reg(&tmplmod, opr->name);
+  if (ret >= 0) {
+    setup_reg_opr(opr, ret, tmplmod, regmask);
     return wordc;
   }
 
   // most likely var in data segment
   opr->type = OPT_LABEL;
+
+  pp = proto_parse(g_fhdr, opr->name);
+  if (pp != NULL) {
+    if (pp->is_fptr) {
+      opr->lmod = OPLM_DWORD;
+      opr->is_ptr = 1;
+    }
+    else {
+      if (!guess_lmod_from_c_type(&tmplmod, &pp->type))
+        anote("unhandled C type '%s' for '%s'\n",
+          pp->type.name, opr->name);
+      
+      if (opr->lmod == OPLM_UNSPEC)
+        opr->lmod = tmplmod;
+      else if (opr->lmod != tmplmod) {
+        opr->size_mismatch = 1;
+        if (tmplmod < opr->lmod)
+          opr->size_lt = 1;
+      }
+    }
+    opr->is_ptr = pp->type.is_ptr;
+    opr->is_array = pp->type.is_array;
+  }
+
   if (opr->lmod == OPLM_UNSPEC)
     guess_lmod_from_name(opr);
-  if (opr->lmod != OPLM_UNSPEC)
-    return wordc;
-
-  // TODO: scan data seg to determine type?
   return wordc;
 }
 
+static const struct {
+  const char *name;
+  unsigned int flags;
+} pref_table[] = {
+  { "rep",    OPF_REP },
+  { "repe",   OPF_REP|OPF_REPZ },
+  { "repz",   OPF_REP|OPF_REPZ },
+  { "repne",  OPF_REP|OPF_REPNZ },
+  { "repnz",  OPF_REP|OPF_REPNZ },
+};
+
 static const struct {
   const char *name;
   enum op_op op;
@@ -388,6 +719,7 @@ static const struct {
   unsigned int maxopr;
   unsigned int flags;
 } op_table[] = {
+  { "nop",  OP_NOP,    0, 0, 0 },
   { "push", OP_PUSH,   1, 1, 0 },
   { "pop",  OP_POP,    1, 1, OPF_DATA },
   { "mov" , OP_MOV,    2, 2, OPF_DATA },
@@ -395,6 +727,16 @@ static const struct {
   { "movzx",OP_MOVZX,  2, 2, OPF_DATA },
   { "movsx",OP_MOVSX,  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 },
+  { "stosw",OP_STOS,   0, 0, OPF_DATA },
+  { "stosd",OP_STOS,   0, 0, OPF_DATA },
+  { "movsb",OP_MOVS,   0, 0, OPF_DATA },
+  { "movsw",OP_MOVS,   0, 0, OPF_DATA },
+  { "movsd",OP_MOVS,   0, 0, OPF_DATA },
+  { "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 },
   { "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 },
@@ -404,22 +746,28 @@ static const struct {
   { "shr",  OP_SHR,    2, 2, OPF_DATA|OPF_FLAGS },
   { "sal",  OP_SHL,    2, 2, OPF_DATA|OPF_FLAGS },
   { "sar",  OP_SAR,    2, 2, OPF_DATA|OPF_FLAGS },
-//  { "adc",  OP_ADC,    2, 2, OPF_DATA|OPF_FLAGS|OPF_CC },
+  { "rol",  OP_ROL,    2, 2, OPF_DATA|OPF_FLAGS },
+  { "ror",  OP_ROR,    2, 2, OPF_DATA|OPF_FLAGS },
+  { "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 },
   { "dec",  OP_DEC,    1, 1, OPF_DATA|OPF_FLAGS },
-//  { "mul",  OP_MUL,    1, 1, OPF_DATA|OPF_FLAGS },
+  { "neg",  OP_NEG,    1, 1, OPF_DATA|OPF_FLAGS },
+  { "mul",  OP_MUL,    1, 1, OPF_DATA|OPF_FLAGS },
   { "imul", OP_IMUL,   1, 3, OPF_DATA|OPF_FLAGS },
+  { "div",  OP_DIV,    1, 1, OPF_DATA|OPF_FLAGS },
+  { "idiv", OP_IDIV,   1, 1, OPF_DATA|OPF_FLAGS },
   { "test", OP_TEST,   2, 2, OPF_FLAGS },
   { "cmp",  OP_CMP,    2, 2, OPF_FLAGS },
-  { "retn", OP_RET,    0, 1, OPF_JMP },
-  { "call", OP_CALL,   1, 1, OPF_JMP },
+  { "retn", OP_RET,    0, 1, OPF_JMP|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
@@ -443,26 +791,72 @@ static const struct {
   { "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
+  { "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 },
+  { "setb",   OP_JC,   1, 1, OPF_DATA|OPF_CC },
+  { "setnc",  OP_JNC,  1, 1, OPF_DATA|OPF_CC },
+  { "setae",  OP_JNC,  1, 1, OPF_DATA|OPF_CC },
+  { "setz",   OP_JZ,   1, 1, OPF_DATA|OPF_CC },
+  { "sete",   OP_JZ,   1, 1, OPF_DATA|OPF_CC },
+  { "setnz",  OP_JNZ,  1, 1, OPF_DATA|OPF_CC },
+  { "setne",  OP_JNZ,  1, 1, OPF_DATA|OPF_CC },
+  { "setbe",  OP_JBE,  1, 1, OPF_DATA|OPF_CC },
+  { "setna",  OP_JBE,  1, 1, OPF_DATA|OPF_CC },
+  { "seta",   OP_JA,   1, 1, OPF_DATA|OPF_CC },
+  { "setnbe", OP_JA,   1, 1, OPF_DATA|OPF_CC },
+  { "sets",   OP_JS,   1, 1, OPF_DATA|OPF_CC },
+  { "setns",  OP_JNS,  1, 1, OPF_DATA|OPF_CC },
+  { "setp",   OP_JP,   1, 1, OPF_DATA|OPF_CC },
+  { "setpe",  OP_JP,   1, 1, OPF_DATA|OPF_CC },
+  { "setnp",  OP_JNP,  1, 1, OPF_DATA|OPF_CC },
+  { "setpo",  OP_JNP,  1, 1, OPF_DATA|OPF_CC },
+  { "setl",   OP_JL,   1, 1, OPF_DATA|OPF_CC },
+  { "setnge", OP_JL,   1, 1, OPF_DATA|OPF_CC },
+  { "setge",  OP_JGE,  1, 1, OPF_DATA|OPF_CC },
+  { "setnl",  OP_JGE,  1, 1, OPF_DATA|OPF_CC },
+  { "setle",  OP_JLE,  1, 1, OPF_DATA|OPF_CC },
+  { "setng",  OP_JLE,  1, 1, OPF_DATA|OPF_CC },
+  { "setg",   OP_JG,   1, 1, OPF_DATA|OPF_CC },
+  { "setnle", OP_JG,   1, 1, OPF_DATA|OPF_CC },
 };
 
 static void parse_op(struct parsed_op *op, char words[16][256], int wordc)
 {
+  enum opr_lenmod lmod = OPLM_UNSPEC;
+  int prefix_flags = 0;
   int regmask_ind;
   int regmask;
+  int op_w = 0;
   int opr = 0;
-  int w = 1;
+  int w = 0;
   int i;
 
+  for (i = 0; i < ARRAY_SIZE(pref_table); i++) {
+    if (IS(words[w], pref_table[i].name)) {
+      prefix_flags = pref_table[i].flags;
+      break;
+    }
+  }
+
+  if (prefix_flags) {
+    if (wordc <= 1)
+      aerr("lone prefix: '%s'\n", words[0]);
+    w++;
+  }
+
+  op_w = w;
   for (i = 0; i < ARRAY_SIZE(op_table); i++) {
-    if (IS(words[0], op_table[i].name))
+    if (IS(words[w], op_table[i].name))
       break;
   }
 
   if (i == ARRAY_SIZE(op_table))
     aerr("unhandled op: '%s'\n", words[0]);
+  w++;
 
   op->op = op_table[i].op;
-  op->flags = op_table[i].flags;
+  op->flags = op_table[i].flags | prefix_flags;
   op->regmask_src = op->regmask_dst = 0;
 
   for (opr = 0; opr < op_table[i].minopr; opr++) {
@@ -482,11 +876,119 @@ static void parse_op(struct parsed_op *op, char words[16][256], int wordc)
       words, wordc, w, op->flags);
   }
 
-  op->operand_cnt = opr;
-
   if (w < wordc)
     aerr("parse_op %s incomplete: %d/%d\n",
       words[0], w, wordc);
+
+  // special cases
+  op->operand_cnt = opr;
+  if (!strncmp(op_table[i].name, "set", 3))
+    op->operand[0].lmod = OPLM_BYTE;
+
+  // ops with implicit argumets
+  switch (op->op) {
+  case OP_CDQ:
+    op->operand_cnt = 2;
+    setup_reg_opr(&op->operand[0], xDX, OPLM_DWORD, &op->regmask_dst);
+    setup_reg_opr(&op->operand[1], xAX, OPLM_DWORD, &op->regmask_src);
+    break;
+
+  case OP_STOS:
+    if (op->operand_cnt != 0)
+      break;
+    if      (IS(words[op_w], "stosb"))
+      lmod = OPLM_BYTE;
+    else if (IS(words[op_w], "stosw"))
+      lmod = OPLM_WORD;
+    else if (IS(words[op_w], "stosd"))
+      lmod = OPLM_DWORD;
+    op->operand_cnt = 3;
+    setup_reg_opr(&op->operand[0], xDI, lmod, &op->regmask_src);
+    setup_reg_opr(&op->operand[1], xCX, OPLM_DWORD, &op->regmask_src);
+    op->regmask_dst = op->regmask_src;
+    setup_reg_opr(&op->operand[2], xAX, OPLM_DWORD, &op->regmask_src);
+    break;
+
+  case OP_MOVS:
+  case OP_CMPS:
+    if (op->operand_cnt != 0)
+      break;
+    if      (words[op_w][4] == 'b')
+      lmod = OPLM_BYTE;
+    else if (words[op_w][4] == 'w')
+      lmod = OPLM_WORD;
+    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);
+    setup_reg_opr(&op->operand[1], xSI, OPLM_DWORD, &op->regmask_src);
+    setup_reg_opr(&op->operand[2], xCX, OPLM_DWORD, &op->regmask_src);
+    op->regmask_dst = op->regmask_src;
+    break;
+
+  case OP_IMUL:
+    if (op->operand_cnt != 1)
+      break;
+    // fallthrough
+  case OP_MUL:
+    // singleop mul
+    op->regmask_dst = (1 << xDX) | (1 << xAX);
+    op->regmask_src |= (1 << xAX);
+    if (op->operand[0].lmod == OPLM_UNSPEC)
+      op->operand[0].lmod = OPLM_DWORD;
+    break;
+
+  case OP_DIV:
+  case OP_IDIV:
+    // we could set up operands for edx:eax, but there is no real need to
+    // (see is_opr_modified())
+    regmask = (1 << xDX) | (1 << xAX);
+    op->regmask_dst = regmask;
+    op->regmask_src |= regmask;
+    if (op->operand[0].lmod == OPLM_UNSPEC)
+      op->operand[0].lmod = OPLM_DWORD;
+    break;
+
+  case OP_SHL:
+  case OP_SHR:
+  case OP_SAR:
+  case OP_ROL:
+  case OP_ROR:
+    if (op->operand[1].lmod == OPLM_UNSPEC)
+      op->operand[1].lmod = OPLM_BYTE;
+    break;
+
+  case OP_PUSH:
+    if (op->operand[0].lmod == OPLM_UNSPEC
+        && (op->operand[0].type == OPT_CONST
+         || op->operand[0].type == OPT_OFFSET
+         || op->operand[0].type == OPT_LABEL))
+      op->operand[0].lmod = OPLM_DWORD;
+    break;
+
+  // alignment
+  case OP_MOV:
+    if (op->operand[0].type == OPT_REG && op->operand[1].type == OPT_REG
+     && op->operand[0].reg == xDI && op->operand[1].reg == xDI)
+    {
+      op->flags |= OPF_RMD;
+    }
+    break;
+
+  case OP_LEA:
+    if (op->operand[0].type == OPT_REG
+     && op->operand[1].type == OPT_REGMEM)
+    {
+      char buf[16];
+      snprintf(buf, sizeof(buf), "%s+0", op->operand[0].name);
+      if (IS(buf, op->operand[1].name))
+        op->flags |= OPF_RMD;
+    }
+    break;
+
+  default:
+    break;
+  }
 }
 
 static const char *op_name(enum op_op op)
@@ -507,6 +1009,9 @@ static const char *dump_op(struct parsed_op *po)
   char *p = out;
   int i;
 
+  if (po == NULL)
+    return "???";
+
   snprintf(out, sizeof(out), "%s", op_name(po->op));
   for (i = 0; i < po->operand_cnt; i++) {
     p += strlen(p);
@@ -520,110 +1025,349 @@ static const char *dump_op(struct parsed_op *po)
   return out;
 }
 
-static const char *opr_name(struct parsed_op *po, int opr_num)
+static const char *lmod_type_u(struct parsed_op *po,
+  enum opr_lenmod lmod)
 {
-  if (opr_num >= po->operand_cnt)
-    ferr(po, "opr OOR: %d/%d\n", opr_num, po->operand_cnt);
-  return po->operand[opr_num].name;
+  switch (lmod) {
+  case OPLM_DWORD:
+    return "u32";
+  case OPLM_WORD:
+    return "u16";
+  case OPLM_BYTE:
+    return "u8";
+  default:
+    ferr(po, "invalid lmod: %d\n", lmod);
+    return "(_invalid_)";
+  }
 }
 
-static unsigned int opr_const(struct parsed_op *po, int opr_num)
+static const char *lmod_cast_u(struct parsed_op *po,
+  enum opr_lenmod lmod)
 {
-  if (opr_num >= po->operand_cnt)
-    ferr(po, "opr OOR: %d/%d\n", opr_num, po->operand_cnt);
-  if (po->operand[opr_num].type != OPT_CONST)
-    ferr(po, "opr %d: const expected\n", opr_num);
-  return po->operand[opr_num].val;
+  switch (lmod) {
+  case OPLM_DWORD:
+    return "";
+  case OPLM_WORD:
+    return "(u16)";
+  case OPLM_BYTE:
+    return "(u8)";
+  default:
+    ferr(po, "invalid lmod: %d\n", lmod);
+    return "(_invalid_)";
+  }
 }
 
-static const char *opr_reg_p(struct parsed_op *po, struct parsed_opr *popr)
+static const char *lmod_cast_u_ptr(struct parsed_op *po,
+  enum opr_lenmod lmod)
 {
-  if ((unsigned int)popr->reg >= MAX_REGS)
-    ferr(po, "invalid reg: %d\n", popr->reg);
-  return regs_r32[popr->reg];
+  switch (lmod) {
+  case OPLM_DWORD:
+    return "*(u32 *)";
+  case OPLM_WORD:
+    return "*(u16 *)";
+  case OPLM_BYTE:
+    return "*(u8 *)";
+  default:
+    ferr(po, "invalid lmod: %d\n", lmod);
+    return "(_invalid_)";
+  }
 }
 
-static struct parsed_equ *equ_find(struct parsed_op *po, const char *name)
+static const char *lmod_cast_s(struct parsed_op *po,
+  enum opr_lenmod lmod)
 {
-  int i;
-
-  for (i = 0; i < g_eqcnt; i++)
-    if (IS(g_eqs[i].name, name))
+  switch (lmod) {
+  case OPLM_DWORD:
+    return "(s32)";
+  case OPLM_WORD:
+    return "(s16)";
+  case OPLM_BYTE:
+    return "(s8)";
+  default:
+    ferr(po, "%s: invalid lmod: %d\n", __func__, lmod);
+    return "(_invalid_)";
+  }
+}
+
+static const char *lmod_cast(struct parsed_op *po,
+  enum opr_lenmod lmod, int is_signed)
+{
+  return is_signed ?
+    lmod_cast_s(po, lmod) :
+    lmod_cast_u(po, lmod);
+}
+
+static int lmod_bytes(struct parsed_op *po, enum opr_lenmod lmod)
+{
+  switch (lmod) {
+  case OPLM_DWORD:
+    return 4;
+  case OPLM_WORD:
+    return 2;
+  case OPLM_BYTE:
+    return 1;
+  default:
+    ferr(po, "%s: invalid lmod: %d\n", __func__, lmod);
+    return 0;
+  }
+}
+
+static const char *opr_name(struct parsed_op *po, int opr_num)
+{
+  if (opr_num >= po->operand_cnt)
+    ferr(po, "opr OOR: %d/%d\n", opr_num, po->operand_cnt);
+  return po->operand[opr_num].name;
+}
+
+static unsigned int opr_const(struct parsed_op *po, int opr_num)
+{
+  if (opr_num >= po->operand_cnt)
+    ferr(po, "opr OOR: %d/%d\n", opr_num, po->operand_cnt);
+  if (po->operand[opr_num].type != OPT_CONST)
+    ferr(po, "opr %d: const expected\n", opr_num);
+  return po->operand[opr_num].val;
+}
+
+static const char *opr_reg_p(struct parsed_op *po, struct parsed_opr *popr)
+{
+  if ((unsigned int)popr->reg >= MAX_REGS)
+    ferr(po, "invalid reg: %d\n", popr->reg);
+  return regs_r32[popr->reg];
+}
+
+static struct parsed_equ *equ_find(struct parsed_op *po, const char *name,
+  int *extra_offs)
+{
+  const char *p;
+  char *endp;
+  int namelen;
+  int i;
+
+  *extra_offs = 0;
+  namelen = strlen(name);
+
+  p = strchr(name, '+');
+  if (p != NULL) {
+    namelen = p - name;
+    if (namelen <= 0)
+      ferr(po, "equ parse failed for '%s'\n", name);
+
+    if (IS_START(p, "0x"))
+      p += 2;
+    *extra_offs = strtol(p, &endp, 16);
+    if (*endp != 0)
+      ferr(po, "equ parse failed for '%s'\n", name);
+  }
+
+  for (i = 0; i < g_eqcnt; i++)
+    if (strncmp(g_eqs[i].name, name, namelen) == 0
+     && g_eqs[i].name[namelen] == 0)
       break;
-  if (i >= g_eqcnt)
-    ferr(po, "unresolved equ name: '%s'\n", name);
+  if (i >= g_eqcnt) {
+    if (po != NULL)
+      ferr(po, "unresolved equ name: '%s'\n", name);
+    return NULL;
+  }
 
   return &g_eqs[i];
 }
 
-static void bg_frame_access(struct parsed_op *po, enum opr_lenmod lmod,
-  char *buf, size_t buf_size, const char *bp_arg,
-  int is_src, int is_lea)
+static void stack_frame_access(struct parsed_op *po,
+  struct parsed_opr *popr, char *buf, size_t buf_size,
+  const char *name, const char *cast, int is_src, int is_lea)
 {
+  enum opr_lenmod tmp_lmod = OPLM_UNSPEC;
   const char *prefix = "";
+  const char *bp_arg = NULL;
+  char ofs_reg[16] = { 0, };
   struct parsed_equ *eq;
+  const char *p;
+  char *endp = NULL;
   int i, arg_i, arg_s;
+  int stack_ra = 0;
+  int offset = 0;
   int sf_ofs;
+  int lim;
+
+  if (!IS_START(name, "ebp-")) {
+    bp_arg = parse_stack_el(name, ofs_reg);
+    snprintf(g_comment, sizeof(g_comment), "%s", bp_arg);
+    eq = equ_find(po, bp_arg, &offset);
+    if (eq == NULL)
+      ferr(po, "detected but missing eq\n");
+    offset += eq->offset;
+  }
+  else {
+    p = name + 4;
+    if (IS_START(p, "0x"))
+      p += 2;
+    offset = -strtoul(p, &endp, 16);
+    if (*endp != 0)
+      ferr(po, "ebp- parse of '%s' failed\n", name);
+  }
 
-  snprintf(g_comment, sizeof(g_comment), "%s", bp_arg);
+  if (!strncmp(name, "ebp", 3))
+    stack_ra = 4;
 
-  eq = equ_find(po, bp_arg);
+  if (stack_ra <= offset && offset < stack_ra + 4)
+    ferr(po, "reference to ra? %d %d\n", offset, stack_ra);
 
-  if (eq->offset >= 0) {
-    arg_i = eq->offset / 4 - 2;
-    if (arg_i < 0 || arg_i >= g_func_pp.argc_stack)
-      ferr(po, "offset %d doesn't map to any arg\n", eq->offset);
+  if (offset > stack_ra)
+  {
+    arg_i = (offset - stack_ra - 4) / 4;
+    if (arg_i < 0 || arg_i >= g_func_pp->argc_stack)
+    {
+      if (g_func_pp->is_vararg
+          && arg_i == g_func_pp->argc_stack && is_lea)
+      {
+        // should be va_list
+        if (cast[0] == 0)
+          cast = "(u32)";
+        snprintf(buf, buf_size, "%sap", cast);
+        return;
+      }
+      ferr(po, "offset %d (%s,%d) doesn't map to any arg\n",
+        offset, bp_arg, arg_i);
+    }
+    if (ofs_reg[0] != 0)
+      ferr(po, "offset reg on arg access?\n");
 
-    for (i = arg_s = 0; i < g_func_pp.argc; i++) {
-      if (g_func_pp.arg[i].reg != NULL)
+    for (i = arg_s = 0; i < g_func_pp->argc; i++) {
+      if (g_func_pp->arg[i].reg != NULL)
         continue;
       if (arg_s == arg_i)
         break;
       arg_s++;
     }
-    if (i == g_func_pp.argc)
+    if (i == g_func_pp->argc)
       ferr(po, "arg %d not in prototype?\n", arg_i);
-    if (is_lea)
-      ferr(po, "lea to arg?\n");
 
-    snprintf(buf, buf_size, "%sa%d", is_src ? "(u32)" : "", i + 1);
+    popr->is_ptr = g_func_pp->arg[i].type.is_ptr;
+
+    switch (popr->lmod)
+    {
+    case OPLM_BYTE:
+      if (is_lea)
+        ferr(po, "lea/byte to arg?\n");
+      if (is_src && (offset & 3) == 0)
+        snprintf(buf, buf_size, "(u8)a%d", i + 1);
+      else
+        snprintf(buf, buf_size, "BYTE%d(a%d)", offset & 3, i + 1);
+      break;
+
+    case OPLM_WORD:
+      if (is_lea)
+        ferr(po, "lea/word to arg?\n");
+      if (offset & 1)
+        ferr(po, "unaligned arg access\n");
+      if (is_src && (offset & 2) == 0)
+        snprintf(buf, buf_size, "(u16)a%d", i + 1);
+      else
+        snprintf(buf, buf_size, "%sWORD(a%d)",
+          (offset & 2) ? "HI" : "LO", i + 1);
+      break;
+
+    case OPLM_DWORD:
+      if (cast[0])
+        prefix = cast;
+      else if (is_src)
+        prefix = "(u32)";
+      if (offset & 3) {
+        snprintf(g_comment, sizeof(g_comment), "%s unaligned", bp_arg);
+        if (is_lea)
+          snprintf(buf, buf_size, "(u32)&a%d + %d",
+            i + 1, offset & 3);
+        else
+          snprintf(buf, buf_size, "%s(a%d >> %d)",
+            prefix, i + 1, (offset & 3) * 8);
+      }
+      else {
+        snprintf(buf, buf_size, "%s%sa%d",
+          prefix, is_lea ? "&" : "", i + 1);
+      }
+      break;
+
+    default:
+      ferr(po, "bp_arg bad lmod: %d\n", popr->lmod);
+    }
+
+    // common problem
+    guess_lmod_from_c_type(&tmp_lmod, &g_func_pp->arg[i].type);
+    if ((offset & 3) && tmp_lmod != OPLM_DWORD)
+      ferr(po, "bp_arg arg/w offset %d and type '%s'\n",
+        offset, g_func_pp->arg[i].type.name);
   }
-  else {
-    if (g_bp_stack == 0)
-      ferr(po, "bp_stack access after it was not detected\n");
+  else
+  {
+    if (g_stack_fsz == 0)
+      ferr(po, "stack var access without stackframe\n");
 
-    sf_ofs = g_bp_stack + eq->offset;
-    if (sf_ofs < 0)
-      ferr(po, "bp_stack offset %d/%d\n", eq->offset, g_bp_stack);
+    sf_ofs = g_stack_fsz + offset;
+    lim = (ofs_reg[0] != 0) ? -4 : 0;
+    if (offset > 0 || sf_ofs < lim)
+      ferr(po, "bp_stack offset %d/%d\n", offset, g_stack_fsz);
 
     if (is_lea)
-      prefix = "&";
+      prefix = "(u32)&";
+    else
+      prefix = cast;
 
-    switch (lmod)
+    switch (popr->lmod)
     {
     case OPLM_BYTE:
-      snprintf(buf, buf_size, "%ssf.b[%d]", prefix, sf_ofs);
+      snprintf(buf, buf_size, "%ssf.b[%d%s%s]",
+        prefix, sf_ofs, ofs_reg[0] ? "+" : "", ofs_reg);
       break;
+
     case OPLM_WORD:
+      if ((sf_ofs & 1) || ofs_reg[0] != 0) {
+        // known unaligned or possibly unaligned
+        strcat(g_comment, " unaligned");
+        if (prefix[0] == 0)
+          prefix = "*(u16 *)&";
+        snprintf(buf, buf_size, "%ssf.b[%d%s%s]",
+          prefix, sf_ofs, ofs_reg[0] ? "+" : "", ofs_reg);
+        break;
+      }
       snprintf(buf, buf_size, "%ssf.w[%d]", prefix, sf_ofs / 2);
       break;
+
     case OPLM_DWORD:
+      if ((sf_ofs & 3) || ofs_reg[0] != 0) {
+        // known unaligned or possibly unaligned
+        strcat(g_comment, " unaligned");
+        if (prefix[0] == 0)
+          prefix = "*(u32 *)&";
+        snprintf(buf, buf_size, "%ssf.b[%d%s%s]",
+          prefix, sf_ofs, ofs_reg[0] ? "+" : "", ofs_reg);
+        break;
+      }
       snprintf(buf, buf_size, "%ssf.d[%d]", prefix, sf_ofs / 4);
       break;
+
     default:
-      ferr(po, "bp_stack bad lmod: %d\n", lmod);
+      ferr(po, "bp_stack bad lmod: %d\n", popr->lmod);
     }
   }
 }
 
+static void check_label_read_ref(struct parsed_op *po, const char *name)
+{
+  if (IS_START(name, "sub_"))
+    ferr(po, "func reference?\n");
+}
+
 static char *out_src_opr(char *buf, size_t buf_size,
-       struct parsed_op *po, struct parsed_opr *popr, int is_lea)
+       struct parsed_op *po, struct parsed_opr *popr, const char *cast,
+  int is_lea)
 {
-  const char *cast = "";
   char tmp1[256], tmp2[256];
   char expr[256];
   int ret;
 
+  if (cast == NULL)
+    cast = "";
+
   switch (popr->type) {
   case OPT_REG:
     if (is_lea)
@@ -631,13 +1375,16 @@ static char *out_src_opr(char *buf, size_t buf_size,
 
     switch (popr->lmod) {
     case OPLM_DWORD:
-      snprintf(buf, buf_size, "%s", opr_reg_p(po, popr));
+      snprintf(buf, buf_size, "%s%s", cast, opr_reg_p(po, popr));
       break;
     case OPLM_WORD:
       snprintf(buf, buf_size, "(u16)%s", opr_reg_p(po, popr));
       break;
     case OPLM_BYTE:
-      snprintf(buf, buf_size, "(u8)%s", opr_reg_p(po, popr));
+      if (popr->name[1] == 'h') // XXX..
+        snprintf(buf, buf_size, "(u8)(%s >> 8)", opr_reg_p(po, popr));
+      else
+        snprintf(buf, buf_size, "(u8)%s", opr_reg_p(po, popr));
       break;
     default:
       ferr(po, "invalid src lmod: %d\n", popr->lmod);
@@ -645,9 +1392,11 @@ static char *out_src_opr(char *buf, size_t buf_size,
     break;
 
   case OPT_REGMEM:
-    if (g_bp_frame && !strncmp(popr->name, "ebp+", 4)) {
-      bg_frame_access(po, popr->lmod, buf, buf_size,
-        popr->name + 4, 1, is_lea);
+    if (parse_stack_el(popr->name, NULL)
+      || (g_bp_frame && IS_START(popr->name, "ebp-")))
+    {
+      stack_frame_access(po, popr, buf, buf_size,
+        popr->name, cast, 1, is_lea);
       break;
     }
 
@@ -657,7 +1406,7 @@ static char *out_src_opr(char *buf, size_t buf_size,
       ret = sscanf(expr, "%[^[][%[^]]]", tmp1, tmp2);
       if (ret != 2)
         ferr(po, "parse failure for '%s'\n", expr);
-      snprintf(expr, sizeof(expr), "(u32)%s + %s", tmp1, tmp2);
+      snprintf(expr, sizeof(expr), "(u32)&%s + %s", tmp1, tmp2);
     }
 
     // XXX: do we need more parsing?
@@ -666,40 +1415,44 @@ static char *out_src_opr(char *buf, size_t buf_size,
       break;
     }
 
-    switch (popr->lmod) {
-    case OPLM_DWORD:
-      cast = "*(u32 *)";
-      break;
-    case OPLM_WORD:
-      cast = "*(u16 *)";
-      break;
-    case OPLM_BYTE:
-      cast = "*(u8 *)";
-      break;
-    default:
-      ferr(po, "invalid lmod: %d\n", popr->lmod);
-    }
+    if (cast[0] == 0)
+      cast = lmod_cast_u_ptr(po, popr->lmod);
     snprintf(buf, buf_size, "%s(%s)", cast, expr);
     break;
 
   case OPT_LABEL:
+    check_label_read_ref(po, popr->name);
+    if (cast[0] == 0 && popr->is_ptr)
+      cast = "(u32)";
+
     if (is_lea)
       snprintf(buf, buf_size, "(u32)&%s", popr->name);
+    else if (popr->size_lt)
+      snprintf(buf, buf_size, "%s%s%s%s", cast,
+        lmod_cast_u_ptr(po, popr->lmod),
+        popr->is_array ? "" : "&",
+        popr->name);
     else
-      snprintf(buf, buf_size, "%s", popr->name);
+      snprintf(buf, buf_size, "%s%s%s", cast, popr->name,
+        popr->is_array ? "[0]" : "");
     break;
 
   case OPT_OFFSET:
+    check_label_read_ref(po, popr->name);
+    if (cast[0] == 0)
+      cast = "(u32)";
     if (is_lea)
       ferr(po, "lea an offset?\n");
-    snprintf(buf, buf_size, "(u32)&%s", popr->name);
+    snprintf(buf, buf_size, "%s&%s", cast, popr->name);
     break;
 
   case OPT_CONST:
     if (is_lea)
       ferr(po, "lea from const?\n");
 
-    snprintf(buf, buf_size, popr->val < 10 ? "%u" : "0x%02x", popr->val);
+    snprintf(buf, buf_size, "%s", cast);
+    ret = strlen(buf);
+    printf_number(buf + ret, buf_size - ret, popr->val);
     break;
 
   default:
@@ -709,6 +1462,7 @@ static char *out_src_opr(char *buf, size_t buf_size,
   return buf;
 }
 
+// note: may set is_ptr (we find that out late for ebp frame..)
 static char *out_dst_opr(char *buf, size_t buf_size,
        struct parsed_op *po, struct parsed_opr *popr)
 {
@@ -724,7 +1478,10 @@ static char *out_dst_opr(char *buf, size_t buf_size,
       break;
     case OPLM_BYTE:
       // ugh..
-      snprintf(buf, buf_size, "LOBYTE(%s)", opr_reg_p(po, popr));
+      if (popr->name[1] == 'h') // XXX..
+        snprintf(buf, buf_size, "BYTE1(%s)", opr_reg_p(po, popr));
+      else
+        snprintf(buf, buf_size, "LOBYTE(%s)", opr_reg_p(po, popr));
       break;
     default:
       ferr(po, "invalid dst lmod: %d\n", popr->lmod);
@@ -732,13 +1489,25 @@ static char *out_dst_opr(char *buf, size_t buf_size,
     break;
 
   case OPT_REGMEM:
-    if (g_bp_frame && !strncmp(popr->name, "ebp+", 4)) {
-      bg_frame_access(po, popr->lmod, buf, buf_size,
-        popr->name + 4, 0, 0);
+    if (parse_stack_el(popr->name, NULL)
+      || (g_bp_frame && IS_START(popr->name, "ebp-")))
+    {
+      stack_frame_access(po, popr, buf, buf_size,
+        popr->name, "", 0, 0);
       break;
     }
 
-    return out_src_opr(buf, buf_size, po, popr, 0);
+    return out_src_opr(buf, buf_size, po, popr, NULL, 0);
+
+  case OPT_LABEL:
+    if (popr->size_mismatch)
+      snprintf(buf, buf_size, "%s%s%s",
+        lmod_cast_u_ptr(po, popr->lmod),
+        popr->is_array ? "" : "&", popr->name);
+    else
+      snprintf(buf, buf_size, "%s%s", popr->name,
+        popr->is_array ? "[0]" : "");
+    break;
 
   default:
     ferr(po, "invalid dst type: %d\n", popr->type);
@@ -747,42 +1516,16 @@ static char *out_dst_opr(char *buf, size_t buf_size,
   return buf;
 }
 
-static const char *lmod_cast_u(struct parsed_op *po,
-  enum opr_lenmod lmod)
-{
-  switch (lmod) {
-  case OPLM_DWORD:
-    return "";
-  case OPLM_WORD:
-    return "(u16)";
-  case OPLM_BYTE:
-    return "(u8)";
-  default:
-    ferr(po, "invalid lmod: %d\n", lmod);
-    return "(_invalid_)";
-  }
-}
-
-static const char *lmod_cast_s(struct parsed_op *po,
-  enum opr_lenmod lmod)
+static char *out_src_opr_u32(char *buf, size_t buf_size,
+       struct parsed_op *po, struct parsed_opr *popr)
 {
-  switch (lmod) {
-  case OPLM_DWORD:
-    return "(s32)";
-  case OPLM_WORD:
-    return "(s16)";
-  case OPLM_BYTE:
-    return "(s8)";
-  default:
-    ferr(po, "invalid lmod: %d\n", lmod);
-    return "(_invalid_)";
-  }
+  return out_src_opr(buf, buf_size, po, popr, NULL, 0);
 }
 
 static enum parsed_flag_op split_cond(struct parsed_op *po,
-  enum op_op op, int *is_neg)
+  enum op_op op, int *is_inv)
 {
-  *is_neg = 0;
+  *is_inv = 0;
 
   switch (op) {
   case OP_JO:
@@ -803,28 +1546,28 @@ static enum parsed_flag_op split_cond(struct parsed_op *po,
     return PFO_LE;
 
   case OP_JNO:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_O;
   case OP_JNC:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_C;
   case OP_JNZ:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_Z;
   case OP_JA:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_BE;
   case OP_JNS:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_S;
   case OP_JNP:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_P;
   case OP_JGE:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_L;
   case OP_JG:
-    *is_neg = 1;
+    *is_inv = 1;
     return PFO_LE;
 
   case OP_ADC:
@@ -838,7 +1581,7 @@ static enum parsed_flag_op split_cond(struct parsed_op *po,
 }
 
 static void out_test_for_cc(char *buf, size_t buf_size,
-  struct parsed_op *po, enum parsed_flag_op pfo, int is_neg,
+  struct parsed_op *po, enum parsed_flag_op pfo, int is_inv,
   enum opr_lenmod lmod, const char *expr)
 {
   const char *cast, *scast;
@@ -848,13 +1591,20 @@ static void out_test_for_cc(char *buf, size_t buf_size,
 
   switch (pfo) {
   case PFO_Z:
+  case PFO_BE: // CF=1||ZF=1; CF=0
     snprintf(buf, buf_size, "(%s%s %s 0)",
-      cast, expr, is_neg ? "!=" : "==");
+      cast, expr, is_inv ? "!=" : "==");
     break;
 
-  case PFO_LE: // ZF=1||SF!=OF; OF=0 after test
+  case PFO_S:
+  case PFO_L: // SF!=OF; OF=0
     snprintf(buf, buf_size, "(%s%s %s 0)",
-      scast, expr, is_neg ? ">" : "<=");
+      scast, expr, is_inv ? ">=" : "<");
+    break;
+
+  case PFO_LE: // ZF=1||SF!=OF; OF=0
+    snprintf(buf, buf_size, "(%s%s %s 0)",
+      scast, expr, is_inv ? ">" : "<=");
     break;
 
   default:
@@ -863,7 +1613,7 @@ static void out_test_for_cc(char *buf, size_t buf_size,
 }
 
 static void out_cmp_for_cc(char *buf, size_t buf_size,
-  struct parsed_op *po, enum parsed_flag_op pfo, int is_neg,
+  struct parsed_op *po, enum parsed_flag_op pfo, int is_inv,
   enum opr_lenmod lmod, const char *expr1, const char *expr2)
 {
   const char *cast, *scast;
@@ -872,21 +1622,37 @@ static void out_cmp_for_cc(char *buf, size_t buf_size,
   scast = lmod_cast_s(po, lmod);
 
   switch (pfo) {
+  case PFO_C:
+    // note: must be unsigned compare
+    snprintf(buf, buf_size, "(%s%s %s %s%s)",
+      cast, expr1, is_inv ? ">=" : "<", cast, expr2);
+    break;
+
   case PFO_Z:
     snprintf(buf, buf_size, "(%s%s %s %s%s)",
-      cast, expr1, is_neg ? "!=" : "==", cast, expr2);
+      cast, expr1, is_inv ? "!=" : "==", cast, expr2);
     break;
 
-  case PFO_C:
+  case PFO_BE: // !a
     // note: must be unsigned compare
     snprintf(buf, buf_size, "(%s%s %s %s%s)",
-      cast, expr1, is_neg ? ">=" : "<", cast, expr2);
+      cast, expr1, is_inv ? ">" : "<=", cast, expr2);
+    break;
+
+  // note: must be signed compare
+  case PFO_S:
+    snprintf(buf, buf_size, "(%s(%s - %s) %s 0)",
+      scast, expr1, expr2, is_inv ? ">=" : "<");
+    break;
+
+  case PFO_L: // !ge
+    snprintf(buf, buf_size, "(%s%s %s %s%s)",
+      scast, expr1, is_inv ? ">=" : "<", scast, expr2);
     break;
 
-  case PFO_L:
-    // note: must be signed compare
+  case PFO_LE:
     snprintf(buf, buf_size, "(%s%s %s %s%s)",
-      scast, expr1, is_neg ? ">=" : "<", scast, expr2);
+      scast, expr1, is_inv ? ">" : "<=", scast, expr2);
     break;
 
   default:
@@ -895,26 +1661,26 @@ static void out_cmp_for_cc(char *buf, size_t buf_size,
 }
 
 static void out_cmp_test(char *buf, size_t buf_size,
-  struct parsed_op *po, enum parsed_flag_op pfo, int is_neg)
+  struct parsed_op *po, enum parsed_flag_op pfo, int is_inv)
 {
   char buf1[256], buf2[256], buf3[256];
 
   if (po->op == OP_TEST) {
     if (IS(opr_name(po, 0), opr_name(po, 1))) {
-      out_src_opr(buf3, sizeof(buf3), po, &po->operand[0], 0);
+      out_src_opr_u32(buf3, sizeof(buf3), po, &po->operand[0]);
     }
     else {
-      out_src_opr(buf1, sizeof(buf1), po, &po->operand[0], 0);
-      out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], 0);
+      out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]);
+      out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]);
       snprintf(buf3, sizeof(buf3), "(%s & %s)", buf1, buf2);
     }
-    out_test_for_cc(buf, buf_size, po, pfo, is_neg,
+    out_test_for_cc(buf, buf_size, po, pfo, is_inv,
       po->operand[0].lmod, buf3);
   }
   else if (po->op == OP_CMP) {
-    out_src_opr(buf2, sizeof(buf2), po, &po->operand[0], 0);
-    out_src_opr(buf3, sizeof(buf3), po, &po->operand[1], 0);
-    out_cmp_for_cc(buf, buf_size, po, pfo, is_neg,
+    out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[0]);
+    out_src_opr_u32(buf3, sizeof(buf3), po, &po->operand[1]);
+    out_cmp_for_cc(buf, buf_size, po, pfo, is_inv,
       po->operand[0].lmod, buf2, buf3);
   }
   else
@@ -924,20 +1690,6 @@ static void out_cmp_test(char *buf, size_t buf_size,
 static void propagate_lmod(struct parsed_op *po, struct parsed_opr *popr1,
        struct parsed_opr *popr2)
 {
-  struct parsed_equ *eq;
-
-  if (popr1->lmod == OPLM_UNSPEC && popr2->lmod == OPLM_UNSPEC) {
-    // lmod could be specified by equ..
-    if (!strncmp(popr1->name, "ebp+", 4)) {
-      eq = equ_find(po, popr1->name + 4);
-      popr1->lmod = eq->lmod;
-    }
-    if (!strncmp(popr2->name, "ebp+", 4)) {
-      eq = equ_find(po, popr2->name + 4);
-      popr2->lmod = eq->lmod;
-    }
-  }
-
   if (popr1->lmod == OPLM_UNSPEC && popr2->lmod == OPLM_UNSPEC)
     ferr(po, "missing lmod for both operands\n");
 
@@ -954,8 +1706,10 @@ static const char *op_to_c(struct parsed_op *po)
   switch (po->op)
   {
     case OP_ADD:
+    case OP_ADC:
       return "+";
     case OP_SUB:
+    case OP_SBB:
       return "-";
     case OP_AND:
       return "&";
@@ -975,31 +1729,110 @@ static const char *op_to_c(struct parsed_op *po)
   }
 }
 
-static int scan_for_pop(int i, int opcnt, const char *reg)
+static void set_flag_no_dup(struct parsed_op *po, enum op_flags flag,
+  enum op_flags flag_check)
+{
+  if (po->flags & flag)
+    ferr(po, "flag %x already set\n", flag);
+  if (po->flags & flag_check)
+    ferr(po, "flag_check %x already set\n", flag_check);
+
+  po->flags |= flag;
+}
+
+// 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)
+
+static int scan_for_pop(int i, int opcnt, const char *reg,
+  int magic, int depth, int *maxdepth, int do_flags)
 {
+  struct parsed_op *po;
+  int ret = 0;
+  int j;
+
   for (; i < opcnt; i++) {
-    if (ops[i].flags & OPF_RMD)
+    po = &ops[i];
+    if (po->cc_scratch == magic)
+      break; // already checked
+    po->cc_scratch = magic;
+
+    if (po->flags & OPF_TAIL)
+      return -1; // deadend
+
+    if ((po->flags & OPF_RMD)
+        || (po->op == OP_PUSH && po->argnum != 0)) // arg push
       continue;
 
-    if ((ops[i].flags & OPF_JMP) || g_labels[i][0] != 0)
-      return -1;
+    if ((po->flags & OPF_JMP) && po->op != OP_CALL) {
+      if (po->btj != NULL) {
+        // jumptable
+        for (j = 0; j < po->btj->count - 1; j++) {
+          ret |= scan_for_pop(po->btj->d[j].bt_i, opcnt, reg, magic,
+                   depth, maxdepth, do_flags);
+          if (ret < 0)
+            return ret; // dead end
+        }
+        // follow last jumptable entry
+        i = po->btj->d[j].bt_i - 1;
+        continue;
+      }
 
-    if (ops[i].op == OP_POP && ops[i].operand[0].type == OPT_REG
-        && IS(ops[i].operand[0].name, reg))
-      return i;
+      if (po->bt_i < 0) {
+        ferr(po, "dead branch\n");
+        return -1;
+      }
+
+      if (po->flags & OPF_CC) {
+        ret |= scan_for_pop(po->bt_i, opcnt, reg, magic,
+                 depth, maxdepth, do_flags);
+        if (ret < 0)
+          return ret; // dead end
+      }
+      else {
+        i = po->bt_i - 1;
+      }
+      continue;
+    }
+
+    if ((po->op == OP_POP || po->op == OP_PUSH)
+        && po->operand[0].type == OPT_REG
+        && IS(po->operand[0].name, reg))
+    {
+      if (po->op == OP_PUSH) {
+        depth++;
+        if (depth > *maxdepth)
+          *maxdepth = depth;
+        if (do_flags)
+          set_flag_no_dup(po, OPF_RSAVE, OPF_RMD);
+      }
+      else if (depth == 0) {
+        if (do_flags)
+          set_flag_no_dup(po, OPF_RMD, OPF_RSAVE);
+        return 1;
+      }
+      else {
+        depth--;
+        if (depth < 0) // should not happen
+          ferr(po, "fail with depth\n");
+        if (do_flags)
+          set_flag_no_dup(po, OPF_RSAVE, OPF_RMD);
+      }
+    }
   }
 
-  return -1;
+  return ret;
 }
 
 // scan for pop starting from 'ret' op (all paths)
-static int scan_for_pop_ret(int i, int opcnt, const char *reg, int do_patch)
+static int scan_for_pop_ret(int i, int opcnt, const char *reg,
+  int flag_set)
 {
   int found = 0;
   int j;
 
   for (; i < opcnt; i++) {
-    if (ops[i].op != OP_RET)
+    if (!(ops[i].flags & OPF_TAIL))
       continue;
 
     for (j = i - 1; j >= 0; j--) {
@@ -1012,8 +1845,7 @@ static int scan_for_pop_ret(int i, int opcnt, const char *reg, int do_patch)
           && IS(ops[j].operand[0].name, reg))
       {
         found = 1;
-        if (do_patch)
-          ops[j].flags |= OPF_RMD;
+        ops[j].flags |= flag_set;
         break;
       }
 
@@ -1025,8 +1857,8 @@ static int scan_for_pop_ret(int i, int opcnt, const char *reg, int do_patch)
   return found ? 0 : -1;
 }
 
-// is operand opr modified by parsed_op po?
-static int is_opr_modified(struct parsed_opr *opr,
+// is operand 'opr modified' by parsed_op 'po'?
+static int is_opr_modified(const struct parsed_opr *opr,
   const struct parsed_op *po)
 {
   if ((po->flags & OPF_RMD) || !(po->flags & OPF_DATA))
@@ -1042,24 +1874,103 @@ static int is_opr_modified(struct parsed_opr *opr,
   return IS(po->operand[0].name, opr->name);
 }
 
-// scan for provided opr modification in range given
-static int scan_for_mod(struct parsed_opr *opr, int i, int opcnt)
+// is any operand of parsed_op 'po_test' modified by parsed_op 'po'?
+static int is_any_opr_modified(const struct parsed_op *po_test,
+  const struct parsed_op *po)
+{
+  int i;
+
+  if ((po->flags & OPF_RMD) || !(po->flags & OPF_DATA))
+    return 0;
+
+  if (po_test->operand_cnt == 1 && po_test->operand[0].type == OPT_CONST)
+    return 0;
+
+  if ((po_test->regmask_src | po_test->regmask_dst) & po->regmask_dst)
+    return 1;
+
+  // in reality, it can wreck any register, but in decompiled C
+  // version it can only overwrite eax or edx:eax
+  if (po->op == OP_CALL
+   && ((po_test->regmask_src | po_test->regmask_dst)
+       & ((1 << xAX)|(1 << xDX))))
+    return 1;
+
+  for (i = 0; i < po_test->operand_cnt; i++)
+    if (IS(po_test->operand[i].name, po->operand[0].name))
+      return 1;
+
+  return 0;
+}
+
+// scan for any po_test operand modification in range given
+static int scan_for_mod(struct parsed_op *po_test, int i, int opcnt)
+{
+  if (po_test->operand_cnt == 1 && po_test->operand[0].type == OPT_CONST)
+    return -1;
+
+  for (; i < opcnt; i++) {
+    if (is_any_opr_modified(po_test, &ops[i]))
+      return i;
+  }
+
+  return -1;
+}
+
+// scan for po_test operand[0] modification in range given
+static int scan_for_mod_opr0(struct parsed_op *po_test,
+  int i, int opcnt)
 {
   for (; i < opcnt; i++) {
-    if (is_opr_modified(opr, &ops[i]))
+    if (is_opr_modified(&po_test->operand[0], &ops[i]))
       return i;
   }
 
   return -1;
 }
 
-static int scan_for_flag_set(int i, int opcnt)
+static int scan_for_flag_set(int i, int *branched, int *setters,
+  int *setter_cnt)
+{
+  int ret;
+
+  while (i >= 0) {
+    if (g_labels[i][0] != 0) {
+      *branched = 1;
+      if (g_label_refs[i].next != NULL)
+        return -1;
+      if (i > 0 && LAST_OP(i - 1)) {
+        i = g_label_refs[i].i;
+        continue;
+      }
+      ret = scan_for_flag_set(g_label_refs[i].i, branched,
+              setters, setter_cnt);
+      if (ret < 0)
+        return ret;
+    }
+    i--;
+
+    if (ops[i].flags & OPF_FLAGS) {
+      setters[*setter_cnt] = i;
+      (*setter_cnt)++;
+      return 0;
+    }
+
+    if ((ops[i].flags & OPF_JMP) && !(ops[i].flags & OPF_CC))
+      return -1;
+  }
+
+  return -1;
+}
+
+// scan back for cdq, if anything modifies edx, fail
+static int scan_for_cdq_edx(int i)
 {
   for (; i >= 0; i--) {
-    if (ops[i].flags & OPF_FLAGS)
+    if (ops[i].op == OP_CDQ)
       return i;
 
-    if ((ops[i].flags & OPF_JMP) && !(ops[i].flags & OPF_CC))
+    if (ops[i].regmask_dst & (1 << xDX))
       return -1;
     if (g_labels[i][0] != 0)
       return -1;
@@ -1068,104 +1979,531 @@ static int scan_for_flag_set(int i, int opcnt)
   return -1;
 }
 
-static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
+static int scan_for_reg_clear(int i, int reg)
 {
-  struct parsed_op *po, *delayed_flag_op = NULL, *tmp_op;
-  struct parsed_opr *last_arith_dst = NULL;
-  char buf1[256], buf2[256], buf3[256];
-  struct parsed_proto *pp;
+  for (; i >= 0; i--) {
+    if (ops[i].op == OP_XOR
+     && ops[i].operand[0].lmod == OPLM_DWORD
+     && ops[i].operand[0].reg == ops[i].operand[1].reg
+     && ops[i].operand[0].reg == reg)
+      return i;
+
+    if (ops[i].regmask_dst & (1 << reg))
+      return -1;
+    if (g_labels[i][0] != 0)
+      return -1;
+  }
+
+  return -1;
+}
+
+// scan for positive, constant esp adjust
+static int scan_for_esp_adjust(int i, int opcnt, int *adj)
+{
+  struct parsed_op *po;
+  *adj = 0;
+
+  for (; i < opcnt; i++) {
+    po = &ops[i];
+
+    if (po->op == OP_ADD && po->operand[0].reg == xSP) {
+      if (po->operand[1].type != OPT_CONST)
+        ferr(&ops[i], "non-const esp adjust?\n");
+      *adj += po->operand[1].val;
+      if (*adj & 3)
+        ferr(&ops[i], "unaligned esp adjust: %x\n", *adj);
+      return i;
+    }
+    else if (po->op == OP_PUSH)
+      *adj -= lmod_bytes(po, po->operand[0].lmod);
+    else if (po->op == OP_POP)
+      *adj += lmod_bytes(po, po->operand[0].lmod);
+    else if (po->flags & (OPF_JMP|OPF_TAIL)) {
+      if (po->op != OP_CALL)
+        return -1;
+      if (po->operand[0].type != OPT_LABEL)
+        return -1;
+      // TODO: should only allow combining __cdecl calls..
+    }
+
+    if (g_labels[i][0] != 0)
+      return -1;
+  }
+
+  return -1;
+}
+
+static int collect_call_args(struct parsed_op *po, int i,
+  struct parsed_proto *pp, int *save_arg_vars, int arg,
+  int need_op_saving, int branched)
+{
+  struct parsed_proto *pp_tmp;
+  struct label_ref *lr;
+  int need_to_save_current;
+  int ret = 0;
+  int j;
+
+  if (i < 0)
+    ferr(po, "no refs for '%s'?\n", g_labels[i]);
+
+  for (; arg < pp->argc; arg++)
+    if (pp->arg[arg].reg == NULL)
+      break;
+
+  for (j = i; j >= 0 && arg < pp->argc; )
+  {
+    if (g_labels[j][0] != 0) {
+      branched = 1;
+      lr = &g_label_refs[j];
+      if (lr->next != NULL)
+        need_op_saving = 1;
+      for (; lr->next; lr = lr->next)
+        ret |= collect_call_args(po, lr->i, pp, save_arg_vars,
+                 arg, need_op_saving, branched);
+
+      if (j > 0 && LAST_OP(j - 1)) {
+        // follow last branch in reverse
+        j = lr->i;
+        continue;
+      }
+      need_op_saving = 1;
+      ret |= collect_call_args(po, lr->i, pp, save_arg_vars,
+               arg, need_op_saving, branched);
+    }
+    j--;
+
+    if (ops[j].op == OP_CALL)
+    {
+      pp_tmp = ops[j].datap;
+      if (pp_tmp == NULL)
+        ferr(po, "arg collect hit unparsed call\n");
+      if (branched && pp_tmp->argc_stack > 0)
+        ferr(po, "arg collect %d/%d hit '%s' with %d stack args\n",
+          arg, pp->argc, opr_name(&ops[j], 0), pp_tmp->argc_stack);
+    }
+    else if (ops[j].op == OP_ADD && ops[j].operand[0].reg == xSP) {
+      ferr(po, "arg collect %d/%d hit esp adjust\n",
+        arg, pp->argc);
+    }
+    else if (ops[j].op == OP_POP) {
+      ferr(po, "arg collect %d/%d hit pop\n", arg, pp->argc);
+    }
+    else if (LAST_OP(j)) {
+      break;
+    }
+    else if ((ops[j].flags & (OPF_JMP|OPF_CC)) == (OPF_JMP|OPF_CC))
+    {
+      branched = 1;
+    }
+    else if (ops[j].op == OP_PUSH && !(ops[j].flags & OPF_FARG))
+    {
+      pp->arg[arg].datap = &ops[j];
+      need_to_save_current = 0;
+      if (!need_op_saving) {
+        ret = scan_for_mod(&ops[j], j + 1, i);
+        need_to_save_current = (ret >= 0);
+      }
+      if (need_op_saving || need_to_save_current) {
+        // mark this push as one that needs operand saving
+        ops[j].flags &= ~OPF_RMD;
+        if (ops[j].argnum == 0) {
+          ops[j].argnum = arg + 1;
+          *save_arg_vars |= 1 << arg;
+        }
+        else if (ops[j].argnum < arg + 1)
+          ferr(&ops[j], "argnum conflict (%d<%d) for '%s'\n",
+            ops[j].argnum, arg + 1, pp->name);
+      }
+      else if (ops[j].argnum == 0)
+        ops[j].flags |= OPF_RMD;
+
+      // some PUSHes are reused by calls on multiple branches,
+      // but that can't happen if we didn't branch, so they
+      // can be removed from future searches (handles nested calls)
+      if (!branched)
+        ops[j].flags |= OPF_FARG;
+
+      // next arg
+      for (arg++; arg < pp->argc; arg++)
+        if (pp->arg[arg].reg == NULL)
+          break;
+    }
+  }
+
+  if (arg < pp->argc) {
+    ferr(po, "arg collect failed for '%s': %d/%d\n",
+      pp->name, arg, pp->argc);
+    ret = -1;
+  }
+  return ret;
+}
+
+static void add_label_ref(struct label_ref *lr, int op_i)
+{
+  struct label_ref *lr_new;
+
+  if (lr->i == -1) {
+    lr->i = op_i;
+    return;
+  }
+
+  lr_new = calloc(1, sizeof(*lr_new));
+  lr_new->i = op_i;
+  lr->next = lr_new;
+}
+
+static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
+{
+  struct parsed_op *po, *delayed_flag_op = NULL, *tmp_op;
+  struct parsed_opr *last_arith_dst = NULL;
+  char buf1[256], buf2[256], buf3[256], cast[64];
+  const struct parsed_proto *pp_c;
+  struct parsed_proto *pp, *pp_tmp;
+  struct parsed_data *pd;
   const char *tmpname;
+  enum parsed_flag_op pfo;
   int save_arg_vars = 0;
   int cmp_result_vars = 0;
+  int need_mul_var = 0;
   int had_decl = 0;
+  int label_pending = 0;
+  int regmask_save = 0;
   int regmask_arg = 0;
   int regmask = 0;
-  int special_sbb = 0;
+  int pfomask = 0;
+  int found = 0;
+  int depth = 0;
   int no_output;
+  int i, j, l;
   int dummy;
   int arg;
-  int i, j;
   int reg;
   int ret;
 
-  g_bp_frame = g_bp_stack = 0;
+  g_bp_frame = g_sp_frame = g_stack_fsz = 0;
 
-  ret = proto_parse(fhdr, funcn, &g_func_pp);
-  if (ret)
+  g_func_pp = proto_parse(fhdr, funcn);
+  if (g_func_pp == NULL)
     ferr(ops, "proto_parse failed for '%s'\n", funcn);
 
-  fprintf(fout, "%s %s(", g_func_pp.ret_type, funcn);
-  for (i = 0; i < g_func_pp.argc; i++) {
+  fprintf(fout, "%s ", g_func_pp->ret_type.name);
+  if (g_ida_func_attr & IDAFA_NORETURN)
+    fprintf(fout, "noreturn ");
+  fprintf(fout, "%s(", funcn);
+  for (i = 0; i < g_func_pp->argc; i++) {
     if (i > 0)
       fprintf(fout, ", ");
-    fprintf(fout, "%s a%d", g_func_pp.arg[i].type, i + 1);
+    fprintf(fout, "%s a%d", g_func_pp->arg[i].type.name, i + 1);
+  }
+  if (g_func_pp->is_vararg) {
+    if (i > 0)
+      fprintf(fout, ", ");
+    fprintf(fout, "...");
   }
   fprintf(fout, ")\n{\n");
 
   // pass1:
-  // - handle ebp frame, remove ops related to it
+  // - handle ebp/esp frame, remove ops related to it
   if (ops[0].op == OP_PUSH && IS(opr_name(&ops[0], 0), "ebp")
       && ops[1].op == OP_MOV
       && IS(opr_name(&ops[1], 0), "ebp")
       && IS(opr_name(&ops[1], 1), "esp"))
   {
+    int ecx_push = 0;
+
     g_bp_frame = 1;
     ops[0].flags |= OPF_RMD;
     ops[1].flags |= OPF_RMD;
+    i = 2;
 
     if (ops[2].op == OP_SUB && IS(opr_name(&ops[2], 0), "esp")) {
-      g_bp_stack = opr_const(&ops[2], 1);
+      g_stack_fsz = opr_const(&ops[2], 1);
       ops[2].flags |= OPF_RMD;
+      i++;
+    }
+    else {
+      // another way msvc builds stack frame..
+      i = 2;
+      while (ops[i].op == OP_PUSH && IS(opr_name(&ops[i], 0), "ecx")) {
+        g_stack_fsz += 4;
+        ops[i].flags |= OPF_RMD;
+        ecx_push++;
+        i++;
+      }
+      // and another way..
+      if (i == 2 && ops[i].op == OP_MOV && ops[i].operand[0].reg == xAX
+          && ops[i].operand[1].type == OPT_CONST
+          && ops[i + 1].op == OP_CALL
+          && IS(opr_name(&ops[i + 1], 0), "__alloca_probe"))
+      {
+        g_stack_fsz += ops[i].operand[1].val;
+        ops[i].flags |= OPF_RMD;
+        i++;
+        ops[i].flags |= OPF_RMD;
+        i++;
+      }
     }
 
-    i = 2;
+    found = 0;
     do {
       for (; i < opcnt; i++)
         if (ops[i].op == OP_RET)
           break;
-      if (ops[i - 1].op != OP_POP || !IS(opr_name(&ops[i - 1], 0), "ebp"))
+      if (i == opcnt && (ops[i - 1].flags & OPF_JMP) && found)
+        break;
+
+      if (ops[i - 1].op == OP_POP && IS(opr_name(&ops[i - 1], 0), "ebp"))
+        ops[i - 1].flags |= OPF_RMD;
+      else if (!(g_ida_func_attr & IDAFA_NORETURN))
         ferr(&ops[i - 1], "'pop ebp' expected\n");
-      ops[i - 1].flags |= OPF_RMD;
 
-      if (g_bp_stack != 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 (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"))
         {
+          ops[i - 2].flags |= OPF_RMD;
+        }
+        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"))
+        {
+          ferr(&ops[i - 3], "unexpected ecx pop\n");
         }
-        ops[i - 2].flags |= OPF_RMD;
       }
+
+      found = 1;
       i++;
     } while (i < opcnt);
   }
+  else {
+    for (i = 0; i < opcnt; i++) {
+      if (ops[i].op == OP_PUSH || (ops[i].flags & (OPF_JMP|OPF_TAIL)))
+        break;
+      if (ops[i].op == OP_SUB && ops[i].operand[0].reg == xSP
+        && ops[i].operand[1].type == OPT_CONST)
+      {
+        g_sp_frame = 1;
+        break;
+      }
+    }
+
+    if (g_sp_frame)
+    {
+      g_stack_fsz = ops[i].operand[1].val;
+      ops[i].flags |= OPF_RMD;
+
+      i++;
+      do {
+        for (; i < opcnt; i++)
+          if (ops[i].op == OP_RET)
+            break;
+        if (ops[i - 1].op != OP_ADD
+            || !IS(opr_name(&ops[i - 1], 0), "esp")
+            || ops[i - 1].operand[1].type != OPT_CONST
+            || ops[i - 1].operand[1].val != g_stack_fsz)
+          ferr(&ops[i - 1], "'add esp' expected\n");
+        ops[i - 1].flags |= OPF_RMD;
+
+        i++;
+      } while (i < opcnt);
+    }
+  }
 
   // pass2:
+  // - resolve all branches
+  for (i = 0; i < opcnt; i++)
+  {
+    po = &ops[i];
+    po->bt_i = -1;
+    po->btj = NULL;
+
+    if ((po->flags & OPF_RMD) || !(po->flags & OPF_JMP)
+        || po->op == OP_CALL || po->op == OP_RET)
+      continue;
+
+    if (po->operand[0].type == OPT_REGMEM) {
+      char *p = strchr(po->operand[0].name, '[');
+      if (p == NULL)
+        ferr(po, "unhandled indirect branch\n");
+      ret = p - po->operand[0].name;
+      strncpy(buf1, po->operand[0].name, ret);
+      buf1[ret] = 0;
+
+      for (j = 0, pd = NULL; j < g_func_pd_cnt; j++) {
+        if (IS(g_func_pd[j].label, buf1)) {
+          pd = &g_func_pd[j];
+          break;
+        }
+      }
+      if (pd == NULL)
+        ferr(po, "label '%s' not parsed?\n", buf1);
+      if (pd->type != OPT_OFFSET)
+        ferr(po, "label '%s' with non-offset data?\n", buf1);
+
+      // find all labels, link
+      for (j = 0; j < pd->count; j++) {
+        for (l = 0; l < opcnt; l++) {
+          if (g_labels[l][0] && IS(g_labels[l], pd->d[j].u.label)) {
+            add_label_ref(&g_label_refs[l], i);
+            pd->d[j].bt_i = l;
+            break;
+          }
+        }
+      }
+
+      po->btj = pd;
+      continue;
+    }
+
+    for (l = 0; l < opcnt; l++) {
+      if (g_labels[l][0] && IS(po->operand[0].name, g_labels[l])) {
+        add_label_ref(&g_label_refs[l], i);
+        po->bt_i = l;
+        break;
+      }
+    }
+
+    if (po->bt_i != -1)
+      continue;
+
+    if (po->operand[0].type == OPT_LABEL) {
+      // assume tail call
+      po->op = OP_CALL;
+      po->flags |= OPF_TAIL;
+      continue;
+    }
+
+    ferr(po, "unhandled branch\n");
+  }
+
+  // pass3:
+  // - process calls
+  for (i = 0; i < opcnt; i++)
+  {
+    po = &ops[i];
+    if (po->flags & OPF_RMD)
+      continue;
+
+    if (po->op == OP_CALL)
+    {
+      pp = calloc(1, sizeof(*pp));
+      my_assert_not(pp, NULL);
+      tmpname = opr_name(po, 0);
+      if (po->operand[0].type != OPT_LABEL)
+      {
+        ret = scan_for_esp_adjust(i + 1, opcnt, &j);
+        if (ret < 0)
+          ferr(po, "non-__cdecl indirect call unhandled yet\n");
+        j /= 4;
+        if (j > ARRAY_SIZE(pp->arg))
+          ferr(po, "esp adjust too large?\n");
+        pp->ret_type.name = strdup("int");
+        pp->argc = pp->argc_stack = j;
+        for (arg = 0; arg < pp->argc; arg++)
+          pp->arg[arg].type.name = strdup("int");
+      }
+      else {
+        pp_c = proto_parse(fhdr, tmpname);
+        if (pp_c == NULL)
+          ferr(po, "proto_parse failed for call '%s'\n", tmpname);
+        pp = proto_clone(pp_c);
+      }
+
+      // 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);
+      if (ret >= 0) {
+        if (pp->is_vararg) {
+          if (j / 4 < pp->argc_stack)
+            ferr(po, "esp adjust is too small: %x < %x\n",
+              j, pp->argc_stack * 4);
+          // modify pp to make it have varargs as normal args
+          arg = pp->argc;
+          pp->argc += j / 4 - pp->argc_stack;
+          for (; arg < pp->argc; arg++) {
+            pp->arg[arg].type.name = strdup("int");
+            pp->argc_stack++;
+          }
+          if (pp->argc > ARRAY_SIZE(pp->arg))
+            ferr(po, "too many args for '%s'\n", tmpname);
+        }
+        if (pp->argc_stack != j / 4)
+          ferr(po, "stack tracking failed for '%s': %x %x\n",
+            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;
+      }
+      else if (pp->is_vararg)
+        ferr(po, "missing esp_adjust for vararg func '%s'\n",
+          pp->name);
+
+      // can't call functions with non-__cdecl callbacks yet
+      for (arg = 0; arg < pp->argc; arg++) {
+        if (pp->arg[arg].fptr != NULL) {
+          pp_tmp = pp->arg[arg].fptr;
+          if (pp_tmp->is_stdcall || pp_tmp->argc != pp_tmp->argc_stack)
+            ferr(po, "'%s' has a non-__cdecl callback\n", tmpname);
+        }
+      }
+
+      collect_call_args(po, i, pp, &save_arg_vars, 0, 0, 0);
+
+      if (strstr(pp->ret_type.name, "int64"))
+        need_mul_var = 1;
+      po->datap = pp;
+    }
+  }
+
+  // pass4:
   // - find POPs for PUSHes, rm both
   // - scan for all used registers
   // - find flag set ops for their users
-  // - process calls
+  // - declare indirect functions
   for (i = 0; i < opcnt; i++) {
     po = &ops[i];
     if (po->flags & OPF_RMD)
       continue;
 
-    if (po->op == OP_PUSH && po->operand[0].type == OPT_REG) {
-      if (po->operand[0].reg < 0)
+    if (po->op == OP_PUSH
+        && po->argnum == 0 && !(po->flags & OPF_RSAVE)
+        && po->operand[0].type == OPT_REG)
+    {
+      reg = po->operand[0].reg;
+      if (reg < 0)
         ferr(po, "reg not set for push?\n");
-      if (!(regmask & (1 << po->operand[0].reg))) { // reg save
-        ret = scan_for_pop(i + 1, opcnt, po->operand[0].name);
-        if (ret >= 0) {
-          po->flags |= OPF_RMD;
-          ops[ret].flags |= OPF_RMD;
-          continue;
-        }
-        ret = scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, 0);
-        if (ret == 0) {
-          po->flags |= OPF_RMD;
-          scan_for_pop_ret(i + 1, opcnt, po->operand[0].name, 1);
-          continue;
+
+      depth = 0;
+      ret = scan_for_pop(i + 1, opcnt,
+              po->operand[0].name, i + opcnt, 0, &depth, 0);
+      if (ret == 1) {
+        if (depth > 1)
+          ferr(po, "too much depth: %d\n", depth);
+        if (depth > 0)
+          regmask_save |= 1 << reg;
+
+        po->flags |= OPF_RMD;
+        scan_for_pop(i + 1, opcnt, po->operand[0].name,
+          i + opcnt * 2, 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;
       }
     }
 
@@ -1173,90 +2511,123 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
 
     if (po->flags & OPF_CC)
     {
-      ret = scan_for_flag_set(i - 1, opcnt);
-      if (ret < 0)
-        ferr(po, "unable to trace flag setter\n");
+      int setters[16], cnt = 0, branched = 0;
 
-      tmp_op = &ops[ret]; // flag setter
-      for (j = 0; j < tmp_op->operand_cnt; j++) {
-        ret = scan_for_mod(&tmp_op->operand[j], tmp_op - ops + 1, i);
-        if (ret >= 0) {
-          ret = 1 << split_cond(po, po->op, &dummy);
-          tmp_op->pfomask |= ret;
-          cmp_result_vars |= ret;
+      ret = scan_for_flag_set(i, &branched, setters, &cnt);
+      if (ret < 0 || cnt <= 0)
+        ferr(po, "unable to trace flag setter(s)\n");
+      if (cnt > ARRAY_SIZE(setters))
+        ferr(po, "too many flag setters\n");
+
+      pfo = split_cond(po, po->op, &dummy);
+      for (j = 0; j < cnt; j++)
+      {
+        tmp_op = &ops[setters[j]]; // flag setter
+        pfomask = 0;
+
+        // 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) {
+          if (branched || scan_for_mod(tmp_op, setters[j] + 1, i) >= 0)
+            pfomask = 1 << pfo;
+        }
+        else if (tmp_op->op == OP_CMPS) {
+          pfomask = 1 << PFO_Z;
+        }
+        else {
+          if ((pfo != PFO_Z && pfo != PFO_S && pfo != PFO_P)
+              || scan_for_mod_opr0(tmp_op, setters[j] + 1, i) >= 0)
+            pfomask = 1 << pfo;
+        }
+        if (pfomask) {
+          tmp_op->pfomask |= pfomask;
+          cmp_result_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;
     }
-    else if (po->op == OP_CALL)
+    else if (po->op == OP_MUL
+      || (po->op == OP_IMUL && po->operand_cnt == 1))
     {
-      pp = malloc(sizeof(*pp));
+      need_mul_var = 1;
+    }
+    else if (po->op == OP_CALL && po->operand[0].type != OPT_LABEL) {
+      pp = po->datap;
       my_assert_not(pp, NULL);
-      tmpname = opr_name(&ops[i], 0);
-      ret = proto_parse(fhdr, tmpname, pp);
-      if (ret)
-        ferr(po, "proto_parse failed for '%s'\n", tmpname);
-
-      for (arg = 0; arg < pp->argc; arg++)
-        if (pp->arg[arg].reg == NULL)
-          break;
-
-      for (j = i - 1; j >= 0 && arg < pp->argc; j--) {
-        if (ops[j].flags & OPF_RMD)
-          continue;
-        if (ops[j].op != OP_PUSH)
-          continue;
-        if (g_labels[j + 1][0] != 0)
-          ferr(po, "arg search interrupted by '%s'\n", g_labels[j + 1]);
+      fprintf(fout, "  %s (*icall%d)(", pp->ret_type.name, i);
+      for (j = 0; j < pp->argc; j++) {
+        if (j > 0)
+          fprintf(fout, ", ");
+        fprintf(fout, "%s a%d", pp->arg[j].type.name, j + 1);
+      }
+      fprintf(fout, ");\n");
+    }
+  }
 
-        pp->arg[arg].datap = &ops[j];
-        ret = scan_for_mod(&ops[j].operand[0], j + 1, i);
-        if (ret >= 0) {
-          // mark this push as one that needs operand saving
-          ops[j].datap = (void *)(long)(arg + 1);
-          save_arg_vars |= 1 << arg;
-        }
-        else
-          ops[j].flags |= OPF_RMD;
+  // output LUTs/jumptables
+  for (i = 0; i < g_func_pd_cnt; i++) {
+    pd = &g_func_pd[i];
+    fprintf(fout, "  static const ");
+    if (pd->type == OPT_OFFSET) {
+      fprintf(fout, "void *jt_%s[] =\n    { ", pd->label);
+
+      for (j = 0; j < pd->count; j++) {
+        if (j > 0)
+          fprintf(fout, ", ");
+        fprintf(fout, "&&%s", pd->d[j].u.label);
+      }
+    }
+    else {
+      fprintf(fout, "%s %s[] =\n    { ",
+        lmod_type_u(ops, pd->lmod), pd->label);
 
-        // next arg
-        for (arg++; arg < pp->argc; arg++)
-          if (pp->arg[arg].reg == NULL)
-            break;
+      for (j = 0; j < pd->count; j++) {
+        if (j > 0)
+          fprintf(fout, ", ");
+        fprintf(fout, "%u", pd->d[j].u.val);
       }
-      if (arg < pp->argc)
-        ferr(po, "arg collect failed for '%s'\n", tmpname);
-      po->datap = pp;
     }
+    fprintf(fout, " };\n");
   }
 
-  // declare stack frame
-  if (g_bp_stack)
+  // declare stack frame, va_arg
+  if (g_stack_fsz)
     fprintf(fout, "  union { u32 d[%d]; u16 w[%d]; u8 b[%d]; } sf;\n",
-      (g_bp_stack + 3) / 4, (g_bp_stack + 1) / 2, g_bp_stack);
+      (g_stack_fsz + 3) / 4, (g_stack_fsz + 1) / 2, g_stack_fsz);
+
+  if (g_func_pp->is_vararg)
+    fprintf(fout, "  va_list ap;\n");
 
-  // instantiate arg-registers
-  for (i = 0; i < g_func_pp.argc; i++) {
-    if (g_func_pp.arg[i].reg != NULL) {
+  // declare arg-registers
+  for (i = 0; i < g_func_pp->argc; i++) {
+    if (g_func_pp->arg[i].reg != NULL) {
       reg = char_array_i(regs_r32,
-              ARRAY_SIZE(regs_r32), g_func_pp.arg[i].reg);
+              ARRAY_SIZE(regs_r32), g_func_pp->arg[i].reg);
       if (reg < 0)
-        ferr(ops, "arg '%s' is not a reg?\n", g_func_pp.arg[i].reg);
+        ferr(ops, "arg '%s' is not a reg?\n", g_func_pp->arg[i].reg);
 
       regmask_arg |= 1 << reg;
       fprintf(fout, "  u32 %s = (u32)a%d;\n",
-        g_func_pp.arg[i].reg, i + 1);
+        g_func_pp->arg[i].reg, i + 1);
       had_decl = 1;
     }
   }
 
-  // instantiate other regs - special case for eax
-  if (!((regmask | regmask_arg) & 1) && !IS(g_func_pp.ret_type, "void")) {
+  // declare other regs - special case for eax
+  if (!((regmask | regmask_arg) & 1)
+   && !IS(g_func_pp->ret_type.name, "void"))
+  {
     fprintf(fout, "  u32 eax = 0;\n");
     had_decl = 1;
   }
 
   regmask &= ~regmask_arg;
+  regmask &= ~(1 << xSP);
   if (g_bp_frame)
     regmask &= ~(1 << xBP);
   if (regmask) {
@@ -1268,6 +2639,15 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
     }
   }
 
+  if (regmask_save) {
+    for (reg = 0; reg < 8; reg++) {
+      if (regmask_save & (1 << reg)) {
+        fprintf(fout, "  u32 s_%s;\n", regs_r32[reg]);
+        had_decl = 1;
+      }
+    }
+  }
+
   if (save_arg_vars) {
     for (reg = 0; reg < 32; reg++) {
       if (save_arg_vars & (1 << reg)) {
@@ -1286,14 +2666,30 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
     }
   }
 
+  if (need_mul_var) {
+    fprintf(fout, "  u64 mul_tmp;\n");
+    had_decl = 1;
+  }
+
   if (had_decl)
     fprintf(fout, "\n");
 
+  if (g_func_pp->is_vararg) {
+    if (g_func_pp->argc_stack == 0)
+      ferr(ops, "vararg func without stack args?\n");
+    fprintf(fout, "  va_start(ap, a%d);\n", g_func_pp->argc);
+  }
+
   // output ops
   for (i = 0; i < opcnt; i++)
   {
-    if (g_labels[i][0] != 0)
+    if (g_labels[i][0] != 0 && g_label_refs[i].i != -1) {
       fprintf(fout, "\n%s:\n", g_labels[i]);
+      label_pending = 1;
+
+      delayed_flag_op = NULL;
+      last_arith_dst = NULL;
+    }
 
     po = &ops[i];
     if (po->flags & OPF_RMD)
@@ -1305,56 +2701,38 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
       if (po->operand_cnt != n_) \
         ferr(po, "operand_cnt is %d/%d\n", po->operand_cnt, n_)
 
-    // see is delayed flag stuff is still valid
-    if (delayed_flag_op != NULL) {
-      if (po->regmask_dst & delayed_flag_op->regmask_src)
-        delayed_flag_op = NULL;
-      else {
-        for (j = 0; j < po->operand_cnt; j++) {
-          if (is_opr_modified(&delayed_flag_op->operand[0], po))
-            delayed_flag_op = NULL;
-        }
-      }
-    }
-
-    if (last_arith_dst != NULL) {
-      if (is_opr_modified(last_arith_dst, po))
-        last_arith_dst = NULL;
-    }
-
     // conditional/flag using op?
     if (po->flags & OPF_CC)
     {
-      enum parsed_flag_op pfo;
-      int is_neg = 0;
+      int is_delayed = 0;
+      int is_inv = 0;
 
-      pfo = split_cond(po, po->op, &is_neg);
-      special_sbb = 0;
-      if (po->op == OP_SBB && IS(opr_name(po, 0), opr_name(po, 1)))
-        special_sbb = 1;
+      pfo = split_cond(po, po->op, &is_inv);
 
       // we go through all this trouble to avoid using parsed_flag_op,
       // which makes generated code much nicer
       if (delayed_flag_op != NULL)
       {
-        out_cmp_test(buf1, sizeof(buf1), delayed_flag_op, pfo, is_neg);
+        out_cmp_test(buf1, sizeof(buf1), delayed_flag_op, pfo, is_inv);
+        is_delayed = 1;
       }
       else if (last_arith_dst != NULL
         && (pfo == PFO_Z || pfo == PFO_S || pfo == PFO_P))
       {
-        out_src_opr(buf3, sizeof(buf3), po, last_arith_dst, 0);
-        out_test_for_cc(buf1, sizeof(buf1), po, pfo, is_neg,
+        out_src_opr_u32(buf3, sizeof(buf3), po, last_arith_dst);
+        out_test_for_cc(buf1, sizeof(buf1), po, pfo, is_inv,
           last_arith_dst->lmod, buf3);
+        is_delayed = 1;
       }
       else if (po->datap != NULL) {
-        // use preprocessed results
+        // use preprocessed flag calc results
         tmp_op = po->datap;
         if (!tmp_op || !(tmp_op->pfomask & (1 << pfo)))
           ferr(po, "not prepared for pfo %d\n", pfo);
 
-        // note: is_neg was not yet applied
+        // note: is_inv was not yet applied
         snprintf(buf1, sizeof(buf1), "(%scond_%s)",
-          is_neg ? "!" : "", parsed_flag_op_names[pfo]);
+          is_inv ? "!" : "", parsed_flag_op_names[pfo]);
       }
       else {
         ferr(po, "all methods of finding comparison failed\n");
@@ -1363,11 +2741,12 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
       if (po->flags & OPF_JMP) {
         fprintf(fout, "  if %s\n", buf1);
       }
-      else if (special_sbb) {
-        out_dst_opr(buf2, sizeof(buf2), po, &po->operand[0]);
-        fprintf(fout, "  %s = %s * -1;", buf2, buf1);
+      else if (po->op == OP_ADC || po->op == OP_SBB) {
+        if (is_delayed)
+          fprintf(fout, "  cond_%s = %s;\n",
+            parsed_flag_op_names[pfo], buf1);
       }
-      else if (po->flags & OPF_JMP) { // setc
+      else if (po->flags & OPF_DATA) { // SETcc
         out_dst_opr(buf2, sizeof(buf2), po, &po->operand[0]);
         fprintf(fout, "  %s = %s;", buf2, buf1);
       }
@@ -1376,29 +2755,33 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
       }
     }
 
+    pfomask = po->pfomask;
+
     switch (po->op)
     {
       case OP_MOV:
         assert_operand_cnt(2);
         propagate_lmod(po, &po->operand[0], &po->operand[1]);
-        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));
+        out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+        fprintf(fout, "  %s = %s;", buf1,
+            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1],
+              po->operand[0].is_ptr ? "(void *)" : "", 0));
         break;
 
       case OP_LEA:
         assert_operand_cnt(2);
-        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        po->operand[1].lmod = OPLM_DWORD; // always
         fprintf(fout, "  %s = %s;",
             out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
-            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], 1));
+            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1],
+              NULL, 1));
         break;
 
       case OP_MOVZX:
         assert_operand_cnt(2);
         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));
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
         break;
 
       case OP_MOVSX:
@@ -1416,7 +2799,7 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
         fprintf(fout, "  %s = %s%s;",
             out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
             buf3,
-            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], 0));
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
         break;
 
       case OP_NOT:
@@ -1425,128 +2808,319 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
         fprintf(fout, "  %s = ~%s;", buf1, buf1);
         break;
 
+      case OP_CDQ:
+        assert_operand_cnt(2);
+        fprintf(fout, "  %s = (s32)%s >> 31;",
+            out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
+        strcpy(g_comment, "cdq");
+        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",
+            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;",
+            lmod_cast_u_ptr(po, po->operand[0].lmod),
+            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));
+        if (po->flags & OPF_REP) {
+          fprintf(fout,
+            "  for (; ecx != 0; ecx--, edi += %d, esi += %d)\n",
+            j, 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);
+          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));
+        if (po->flags & OPF_REP) {
+          fprintf(fout,
+            "  for (; ecx != 0; ecx--, edi += %d, esi += %d)\n",
+            j, j);
+          fprintf(fout,
+            "    if ((cond_z = (%sedi == %sesi)) %s 0)\n",
+              buf1, buf1, (po->flags & OPF_REPZ) ? "==" : "!=");
+          fprintf(fout,
+            "      break;");
+          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);
+          strcpy(g_comment, "cmps");
+        }
+        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:
-      case OP_XOR:
+        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        // fallthrough
       case OP_SHL:
       case OP_SHR:
       dualop_arith:
         assert_operand_cnt(2);
-        propagate_lmod(po, &po->operand[0], &po->operand[1]);
         fprintf(fout, "  %s %s= %s;",
             out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
             op_to_c(po),
-            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], 0));
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
         last_arith_dst = &po->operand[0];
         delayed_flag_op = NULL;
         break;
 
       case OP_SAR:
         assert_operand_cnt(2);
-        propagate_lmod(po, &po->operand[0], &po->operand[1]);
-        switch (po->operand[0].lmod) {
-        case OPLM_BYTE:
-          strcpy(buf3, "(s8)");
-          break;
-        case OPLM_WORD:
-          strcpy(buf3, "(s16)");
-          break;
-        case OPLM_DWORD:
-          strcpy(buf3, "(s32)");
-          break;
-        default:
-          ferr(po, "invalid dst lmod: %d\n", po->operand[0].lmod);
-        }
         out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
-        fprintf(fout, "  %s = %s%s >> %s;", buf1, buf3, buf1,
-            out_src_opr(buf2, sizeof(buf2), po, &po->operand[1], 0));
+        fprintf(fout, "  %s = %s%s >> %s;", buf1,
+          lmod_cast_s(po, po->operand[0].lmod), buf1,
+          out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
         last_arith_dst = &po->operand[0];
         delayed_flag_op = NULL;
         break;
 
-      case OP_SBB:
-        if (!special_sbb)
+      case OP_ROL:
+      case OP_ROR:
+        assert_operand_cnt(2);
+        out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+        if (po->operand[1].type == OPT_CONST) {
+          j = po->operand[1].val;
+          j %= lmod_bytes(po, po->operand[0].lmod) * 8;
+          fprintf(fout, po->op == OP_ROL ?
+            "  %s = (%s << %d) | (%s >> %d);" :
+            "  %s = (%s >> %d) | (%s << %d);",
+            buf1, buf1, j, buf1,
+            lmod_bytes(po, po->operand[0].lmod) * 8 - j);
+        }
+        else
           ferr(po, "TODO\n");
+        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
+          fprintf(fout, "  %s = 0;",
+            out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]));
+          last_arith_dst = &po->operand[0];
+          delayed_flag_op = NULL;
+          break;
+        }
+        goto dualop_arith;
+
+      case OP_ADC:
+      case OP_SBB:
+        assert_operand_cnt(2);
+        propagate_lmod(po, &po->operand[0], &po->operand[1]);
+        fprintf(fout, "  %s %s= %s + cond_c;",
+            out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]),
+            op_to_c(po),
+            out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[1]));
+        last_arith_dst = &po->operand[0];
+        delayed_flag_op = NULL;
         break;
 
       case OP_INC:
       case OP_DEC:
         out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
-        strcpy(buf2, po->op == OP_INC ? "++" : "--");
-        switch (po->operand[0].lmod) {
-        case OPLM_BYTE:
-          fprintf(fout, "  LOBYTE(%s)%s;", buf1, buf2);
-          break;
-        case OPLM_WORD:
-          fprintf(fout, "  LOWORD(%s)%s;", buf1, buf2);
-          break;
-        case OPLM_DWORD:
+        if (po->operand[0].type == OPT_REG) {
+          strcpy(buf2, po->op == OP_INC ? "++" : "--");
           fprintf(fout, "  %s%s;", buf1, buf2);
-          break;
-        default:
-          ferr(po, "invalid dst lmod: %d\n", po->operand[0].lmod);
         }
+        else {
+          strcpy(buf2, po->op == OP_INC ? "+" : "-");
+          fprintf(fout, "  %s %s= 1;", buf1, buf2);
+        }
+        last_arith_dst = &po->operand[0];
+        delayed_flag_op = NULL;
+        break;
+
+      case OP_NEG:
+        out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+        out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[0]);
+        fprintf(fout, "  %s = -%s%s;", buf1,
+          lmod_cast_s(po, po->operand[0].lmod), buf2);
         last_arith_dst = &po->operand[0];
         delayed_flag_op = NULL;
+        if (pfomask & (1 << PFO_C)) {
+          fprintf(fout, "\n  cond_c = (%s != 0);", buf1);
+          pfomask &= ~(1 << PFO_C);
+        }
         break;
 
       case OP_IMUL:
-        if (po->operand_cnt == 2)
+        if (po->operand_cnt == 2) {
+          propagate_lmod(po, &po->operand[0], &po->operand[1]);
           goto dualop_arith;
-        ferr(po, "TODO imul\n");
-        last_arith_dst = &po->operand[0];
+        }
+        if (po->operand_cnt == 3)
+          ferr(po, "TODO imul3\n");
+        // fallthrough
+      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,
+          out_src_opr_u32(buf2, sizeof(buf2), po, &po->operand[0]));
+        fprintf(fout, "  edx = mul_tmp >> 32;\n");
+        fprintf(fout, "  eax = mul_tmp;");
+        last_arith_dst = NULL;
+        delayed_flag_op = NULL;
+        break;
+
+      case OP_DIV:
+      case OP_IDIV:
+        assert_operand_cnt(1);
+        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 - 1, xDX);
+        else
+          ret = scan_for_cdq_edx(i - 1);
+        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);
+        }
+        else
+          ferr(po, "TODO 64bit divident\n");
+        last_arith_dst = NULL;
         delayed_flag_op = NULL;
         break;
 
       case OP_TEST:
       case OP_CMP:
         propagate_lmod(po, &po->operand[0], &po->operand[1]);
-        if (po->pfomask != 0) {
+        if (pfomask != 0) {
           for (j = 0; j < 8; j++) {
-            if (po->pfomask & (1 << j)) {
+            if (pfomask & (1 << j)) {
               out_cmp_test(buf1, sizeof(buf1), po, j, 0);
               fprintf(fout, "  cond_%s = %s;",
                 parsed_flag_op_names[j], buf1);
             }
           }
+          pfomask = 0;
         }
         else
           no_output = 1;
+        last_arith_dst = NULL;
         delayed_flag_op = po;
         break;
 
       // note: we reuse OP_Jcc for SETcc, only flags differ
       case OP_JO ... OP_JG:
-        if (po->flags & OPF_CC)
+        if (po->flags & OPF_JMP)
           fprintf(fout, "    goto %s;", po->operand[0].name);
-        else
-          ferr(po, "TODO SETcc\n");
+        // else SETcc - should already be handled
         break;
 
       case OP_JMP:
+        assert_operand_cnt(1);
+        last_arith_dst = NULL;
+        delayed_flag_op = NULL;
+
+        if (po->operand[0].type == OPT_REGMEM) {
+          ret = sscanf(po->operand[0].name, "%[^[][%[^*]*4]",
+                  buf1, buf2);
+          if (ret != 2)
+            ferr(po, "parse failure for jmp '%s'\n",
+              po->operand[0].name);
+          fprintf(fout, "  goto *jt_%s[%s];", buf1, buf2);
+          break;
+        }
+        else if (po->operand[0].type != OPT_LABEL)
+          ferr(po, "unhandled jmp type\n");
+
         fprintf(fout, "  goto %s;", po->operand[0].name);
         break;
 
       case OP_CALL:
+        assert_operand_cnt(1);
         pp = po->datap;
         if (pp == NULL)
           ferr(po, "NULL pp\n");
 
+        if (po->operand[0].type != OPT_LABEL)
+          fprintf(fout, "  icall%d = (void *)%s;\n", i,
+            out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]));
+
         fprintf(fout, "  ");
-        if (!IS(pp->ret_type, "void")) {
-          fprintf(fout, "eax = ");
-          if (strchr(pp->ret_type, '*'))
-            fprintf(fout, "(u32)");
+        if (strstr(pp->ret_type.name, "int64")) {
+          if (po->flags & OPF_TAIL)
+            ferr(po, "int64 and tail?\n");
+          fprintf(fout, "mul_tmp = ");
+        }
+        else if (!IS(pp->ret_type.name, "void")) {
+          if (po->flags & OPF_TAIL) {
+            fprintf(fout, "return ");
+            if (g_func_pp->ret_type.is_ptr != pp->ret_type.is_ptr)
+              fprintf(fout, "(%s)", g_func_pp->ret_type.name);
+          }
+          else {
+            fprintf(fout, "eax = ");
+            if (pp->ret_type.is_ptr)
+              fprintf(fout, "(u32)");
+          }
         }
-        fprintf(fout, "%s(", opr_name(po, 0));
+
+        if (po->operand[0].type != OPT_LABEL) {
+          fprintf(fout, "icall%d(", i);
+        }
+        else {
+          if (pp->name[0] == 0)
+            ferr(po, "missing pp->name\n");
+          fprintf(fout, "%s(", pp->name);
+        }
+
         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", pp->arg[arg].reg);
+            fprintf(fout, "%s%s", cast, pp->arg[arg].reg);
             continue;
           }
 
@@ -1554,37 +3128,83 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
           tmp_op = pp->arg[arg].datap;
           if (tmp_op == NULL)
             ferr(po, "parsed_op missing for arg%d\n", arg);
-          if (tmp_op->datap) {
-            fprintf(fout, "s_a%ld", (long)tmp_op->datap);
+          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], 0));
+                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;");
+        }
+
+        if (po->flags & OPF_TAIL) {
+          strcpy(g_comment, "tailcall");
+          if (IS(pp->ret_type.name, "void")
+           && !(g_ida_func_attr & IDAFA_NORETURN))
+          {
+            fprintf(fout, "\n  return;");
+            strcpy(g_comment, "^ tailcall");
+          }
+        }
+        delayed_flag_op = NULL;
+        last_arith_dst = NULL;
         break;
 
       case OP_RET:
-        if (IS(g_func_pp.ret_type, "void"))
-          fprintf(fout, "  return;");
+        if (g_func_pp->is_vararg)
+          fprintf(fout, "  va_end(ap);\n");
+        if (IS(g_func_pp->ret_type.name, "void")) {
+          if (i != opcnt - 1 || label_pending)
+            fprintf(fout, "  return;");
+        }
+        else if (g_func_pp->ret_type.is_ptr) {
+          fprintf(fout, "  return (%s)eax;",
+            g_func_pp->ret_type.name);
+        }
         else
           fprintf(fout, "  return eax;");
+
+        last_arith_dst = NULL;
+        delayed_flag_op = NULL;
         break;
 
       case OP_PUSH:
-        if (po->datap) {
+        if (po->argnum != 0) {
           // special case - saved func arg
-          fprintf(fout, "  s_a%ld = %s;", (long)po->datap,
-            out_src_opr(buf1, sizeof(buf1), po, &po->operand[0], 0));
+          out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]);
+          fprintf(fout, "  s_a%d = %s;", po->argnum, buf1);
+          break;
+        }
+        else if (po->flags & OPF_RSAVE) {
+          out_src_opr_u32(buf1, sizeof(buf1), po, &po->operand[0]);
+          fprintf(fout, "  s_%s = %s;", buf1, buf1);
           break;
         }
-        ferr(po, "push encountered\n");
+        if (!(g_ida_func_attr & IDAFA_NORETURN))
+          ferr(po, "stray push encountered\n");
+        no_output = 1;
         break;
 
       case OP_POP:
-        ferr(po, "pop encountered\n");
+        if (po->flags & OPF_RSAVE) {
+          out_dst_opr(buf1, sizeof(buf1), po, &po->operand[0]);
+          fprintf(fout, "  %s = s_%s;", buf1, buf1);
+          break;
+        }
+        ferr(po, "stray pop encountered\n");
+        break;
+
+      case OP_NOP:
+        no_output = 1;
         break;
 
       default:
@@ -1601,73 +3221,339 @@ static void gen_func(FILE *fout, FILE *fhdr, const char *funcn, int opcnt)
     }
     if (!no_output)
       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)
+      ferr(po, "unexpected repz/repnz\n");
+
+    if (pfomask != 0)
+      ferr(po, "missed flag calc, pfomask=%x\n", pfomask);
+
+    // see is delayed flag stuff is still valid
+    if (delayed_flag_op != NULL && delayed_flag_op != po) {
+      if (is_any_opr_modified(delayed_flag_op, po))
+        delayed_flag_op = NULL;
+    }
+
+    if (last_arith_dst != NULL && last_arith_dst != &po->operand[0]) {
+      if (is_opr_modified(last_arith_dst, po))
+        last_arith_dst = NULL;
+    }
+
+    label_pending = 0;
   }
 
   fprintf(fout, "}\n\n");
 
   // cleanup
   for (i = 0; i < opcnt; i++) {
+    struct label_ref *lr, *lr_del;
+
+    lr = g_label_refs[i].next;
+    while (lr != NULL) {
+      lr_del = lr;
+      lr = lr->next;
+      free(lr_del);
+    }
+    g_label_refs[i].i = -1;
+    g_label_refs[i].next = NULL;
+
     if (ops[i].op == OP_CALL) {
       pp = ops[i].datap;
-      if (pp) {
+      if (pp)
         proto_release(pp);
-        free(pp);
-      }
     }
   }
-  proto_release(&g_func_pp);
+  g_func_pp = NULL;
+}
+
+static void set_label(int i, const char *name)
+{
+  const char *p;
+  int len;
+
+  len = strlen(name);
+  p = strchr(name, ':');
+  if (p != NULL)
+    len = p - name;
+
+  if (len > sizeof(g_labels[0]) - 1)
+    aerr("label '%s' too long: %d\n", name, len);
+  if (g_labels[i][0] != 0 && !IS_START(g_labels[i], "algn_"))
+    aerr("dupe label '%s' vs '%s'?\n", name, g_labels[i]);
+  memcpy(g_labels[i], name, len);
+  g_labels[i][len] = 0;
+}
+
+// '=' needs special treatment..
+static char *next_word_s(char *w, size_t wsize, char *s)
+{
+       size_t i;
+
+       s = sskip(s);
+
+       for (i = 0; i < wsize - 1; i++) {
+               if (s[i] == 0 || my_isblank(s[i]) || (s[i] == '=' && i > 0))
+                       break;
+               w[i] = s[i];
+       }
+       w[i] = 0;
+
+       if (s[i] != 0 && !my_isblank(s[i]) && s[i] != '=')
+               printf("warning: '%s' truncated\n", w);
+
+       return s + i;
+}
+
+struct chunk_item {
+  char *name;
+  long fptr;
+  int asmln;
+};
+
+static int cmp_chunks(const void *p1, const void *p2)
+{
+  const struct chunk_item *c1 = p1, *c2 = p2;
+  return strcmp(c1->name, c2->name);
+}
+
+static int cmpstringp(const void *p1, const void *p2)
+{
+  return strcmp(*(char * const *)p1, *(char * const *)p2);
 }
 
 int main(int argc, char *argv[])
 {
-  FILE *fout, *fasm, *fhdr;
+  FILE *fout, *fasm, *frlist;
+  struct parsed_data *pd = NULL;
+  int pd_alloc = 0;
+  char **rlist = NULL;
+  int rlist_len = 0;
+  int rlist_alloc = 0;
+  struct chunk_item *func_chunks;
+  int func_chunks_used = 0;
+  int func_chunks_sorted = 0;
+  int func_chunk_cnt = 0;
+  int func_chunk_alloc;
+  int func_chunk_i = -1;
+  long func_chunk_ret = 0;
+  int func_chunk_ret_ln = 0;
   char line[256];
   char words[16][256];
+  enum opr_lenmod lmod;
   int in_func = 0;
+  int pending_endp = 0;
+  int skip_func = 0;
+  int skip_warned = 0;
   int eq_alloc;
+  int verbose = 0;
+  int arg_out;
+  int arg = 1;
   int pi = 0;
-  int len;
+  int i, j;
+  int ret, len;
   char *p;
   int wordc;
 
-  if (argc != 4) {
-    printf("usage:\n%s <.c> <.asm> <hdrf>\n",
+  if (argv[1] && IS(argv[1], "-v")) {
+    verbose = 1;
+    arg++;
+  }
+
+  if (argc < arg + 3) {
+    printf("usage:\n%s [-v] <.c> <.asm> <hdrf> [rlist]*\n",
       argv[0]);
     return 1;
   }
 
-  hdrfn = argv[3];
-  fhdr = fopen(hdrfn, "r");
-  my_assert_not(fhdr, NULL);
+  arg_out = arg++;
 
-  asmfn = argv[2];
+  asmfn = argv[arg++];
   fasm = fopen(asmfn, "r");
   my_assert_not(fasm, NULL);
 
-  fout = fopen(argv[1], "w");
+  hdrfn = argv[arg++];
+  g_fhdr = fopen(hdrfn, "r");
+  my_assert_not(g_fhdr, NULL);
+
+  rlist_alloc = 64;
+  rlist = malloc(rlist_alloc * sizeof(rlist[0]));
+  my_assert_not(rlist, NULL);
+  // needs special handling..
+  rlist[rlist_len++] = "__alloca_probe";
+
+  func_chunk_alloc = 32;
+  func_chunks = malloc(func_chunk_alloc * sizeof(func_chunks[0]));
+  my_assert_not(func_chunks, NULL);
+
+  for (; arg < argc; arg++) {
+    frlist = fopen(argv[arg], "r");
+    my_assert_not(frlist, NULL);
+
+    while (fgets(line, sizeof(line), frlist)) {
+      p = sskip(line);
+      if (*p == 0 || *p == ';' || *p == '#')
+        continue;
+
+      p = next_word(words[0], sizeof(words[0]), p);
+      if (words[0][0] == 0)
+        continue;
+
+      if (rlist_len >= rlist_alloc) {
+        rlist_alloc = rlist_alloc * 2 + 64;
+        rlist = realloc(rlist, rlist_alloc * sizeof(rlist[0]));
+        my_assert_not(rlist, NULL);
+      }
+      rlist[rlist_len++] = strdup(words[0]);
+    }
+
+    fclose(frlist);
+    frlist = NULL;
+  }
+
+  if (rlist_len > 0)
+    qsort(rlist, rlist_len, sizeof(rlist[0]), cmpstringp);
+
+  fout = fopen(argv[arg_out], "w");
   my_assert_not(fout, NULL);
 
   eq_alloc = 128;
   g_eqs = malloc(eq_alloc * sizeof(g_eqs[0]));
   my_assert_not(g_eqs, NULL);
 
+  for (i = 0; i < ARRAY_SIZE(g_label_refs); i++) {
+    g_label_refs[i].i = -1;
+    g_label_refs[i].next = NULL;
+  }
+
   while (fgets(line, sizeof(line), fasm))
   {
+    wordc = 0;
     asmln++;
 
     p = sskip(line);
-    if (*p == 0 || *p == ';')
+    if (*p == 0)
       continue;
 
+    // get rid of random tabs
+    for (i = 0; line[i] != 0; i++)
+      if (line[i] == '\t')
+        line[i] = ' ';
+
+    if (*p == ';')
+    {
+      if (p[2] == '=' && IS_START(p, "; =============== S U B"))
+        goto do_pending_endp; // eww..
+
+      if (p[2] == 'A' && IS_START(p, "; Attributes:"))
+      {
+        static const char *attrs[] = {
+          "bp-based frame",
+          "library function",
+          "static",
+          "noreturn",
+          "thunk",
+          "fpd=",
+        };
+
+        // parse IDA's attribute-list comment
+        g_ida_func_attr = 0;
+        p = sskip(p + 13);
+
+        for (; *p != 0; p = sskip(p)) {
+          for (i = 0; i < ARRAY_SIZE(attrs); i++) {
+            if (!strncmp(p, attrs[i], strlen(attrs[i]))) {
+              g_ida_func_attr |= 1 << i;
+              p += strlen(attrs[i]);
+              break;
+            }
+          }
+          if (i == ARRAY_SIZE(attrs)) {
+            anote("unparsed IDA attr: %s\n", p);
+            break;
+          }
+          if (IS(attrs[i], "fpd=")) {
+            p = next_word(words[0], sizeof(words[0]), p);
+            // ignore for now..
+          }
+        }
+      }
+      else if (p[2] == 'S' && IS_START(p, "; START OF FUNCTION CHUNK FOR "))
+      {
+        p += 30;
+        next_word(words[0], sizeof(words[0]), p);
+        if (words[0][0] == 0)
+          aerr("missing nam for func chunk?\n");
+        if (func_chunk_cnt >= func_chunk_alloc) {
+          func_chunk_alloc *= 2;
+          func_chunks = realloc(func_chunks,
+            func_chunk_alloc * sizeof(func_chunks[0]));
+          my_assert_not(func_chunks, NULL);
+        }
+        func_chunks[func_chunk_cnt].fptr = ftell(fasm);
+        func_chunks[func_chunk_cnt].name = strdup(words[0]);
+        func_chunks[func_chunk_cnt].asmln = asmln;
+        func_chunk_cnt++;
+        func_chunks_sorted = 0;
+      }
+      else if (p[2] == 'E' && IS_START(p, "; END OF FUNCTION CHUNK"))
+      {
+        if (func_chunk_i >= 0) {
+          if (func_chunk_i < func_chunk_cnt
+            && IS(func_chunks[func_chunk_i].name, g_func))
+          {
+            // move on to next chunk
+            ret = fseek(fasm, func_chunks[func_chunk_i].fptr, SEEK_SET);
+            if (ret)
+              aerr("seek failed for '%s' chunk #%d\n",
+                g_func, func_chunk_i);
+            asmln = func_chunks[func_chunk_i].asmln;
+            func_chunk_i++;
+          }
+          else {
+            if (func_chunk_ret == 0)
+              aerr("no return from chunk?\n");
+            fseek(fasm, func_chunk_ret, SEEK_SET);
+            asmln = func_chunk_ret_ln;
+            func_chunk_ret = 0;
+            pending_endp = 1;
+          }
+        }
+      }
+      else if (p[2] == 'F' && IS_START(p, "; FUNCTION CHUNK AT ")) {
+        func_chunks_used = 1;
+        p += 20;
+        if (IS_START(g_func, "sub_")) {
+          unsigned long addr = strtoul(p, NULL, 16);
+          unsigned long f_addr = strtoul(g_func + 4, NULL, 16);
+          if (addr > f_addr)
+            aerr("need a chunk %lX that is after %s\n", addr, g_func);
+        }
+      }
+      continue;
+    } // *p == ';'
+
+parse_words:
     memset(words, 0, sizeof(words));
     for (wordc = 0; wordc < 16; wordc++) {
-      p = sskip(next_word(words[wordc], sizeof(words[0]), p));
+      p = sskip(next_word_s(words[wordc], sizeof(words[0]), p));
       if (*p == 0 || *p == ';') {
         wordc++;
         break;
       }
     }
 
+    // alow asm patches in comments
+    if (*p == ';' && IS_START(p, "; sctpatch:")) {
+      p = sskip(p + 11);
+      if (*p == 0 || *p == ';')
+        continue;
+      goto parse_words; // lame
+    }
+
     if (wordc == 0) {
       // shouldn't happen
       awarn("wordc == 0?\n");
@@ -1683,34 +3569,161 @@ int main(int argc, char *argv[])
       continue;
     }
 
+do_pending_endp:
+    // do delayed endp processing to collect switch jumptables
+    if (pending_endp) {
+      if (in_func && !skip_func && wordc >= 2
+          && ((words[0][0] == 'd' && words[0][2] == 0)
+              || (words[1][0] == 'd' && words[1][2] == 0)))
+      {
+        i = 1;
+        if (words[1][0] == 'd' && words[1][2] == 0) {
+          // label
+          if (g_func_pd_cnt >= pd_alloc) {
+            pd_alloc = pd_alloc * 2 + 16;
+            g_func_pd = realloc(g_func_pd,
+              sizeof(g_func_pd[0]) * pd_alloc);
+            my_assert_not(g_func_pd, NULL);
+          }
+          pd = &g_func_pd[g_func_pd_cnt];
+          g_func_pd_cnt++;
+          memset(pd, 0, sizeof(*pd));
+          strcpy(pd->label, words[0]);
+          pd->type = OPT_CONST;
+          pd->lmod = lmod_from_directive(words[1]);
+          i = 2;
+        }
+        else {
+          lmod = lmod_from_directive(words[0]);
+          if (lmod != pd->lmod)
+            aerr("lmod change? %d->%d\n", pd->lmod, lmod);
+        }
+
+        if (pd->count_alloc < pd->count + wordc) {
+          pd->count_alloc = pd->count_alloc * 2 + 14 + wordc;
+          pd->d = realloc(pd->d, sizeof(pd->d[0]) * pd->count_alloc);
+          my_assert_not(pd->d, NULL);
+        }
+        for (; i < wordc; i++) {
+          if (IS(words[i], "offset")) {
+            pd->type = OPT_OFFSET;
+            i++;
+          }
+          p = strchr(words[i], ',');
+          if (p != NULL)
+            *p = 0;
+          if (pd->type == OPT_OFFSET)
+            pd->d[pd->count].u.label = strdup(words[i]);
+          else
+            pd->d[pd->count].u.val = parse_number(words[i]);
+          pd->d[pd->count].bt_i = -1;
+          pd->count++;
+        }
+        continue;
+      }
+
+      if (in_func && !skip_func)
+        gen_func(fout, g_fhdr, g_func, pi);
+
+      pending_endp = 0;
+      in_func = 0;
+      g_ida_func_attr = 0;
+      skip_warned = 0;
+      skip_func = 0;
+      g_func[0] = 0;
+      func_chunks_used = 0;
+      func_chunk_i = -1;
+      if (pi != 0) {
+        memset(&ops, 0, pi * sizeof(ops[0]));
+        memset(g_labels, 0, pi * sizeof(g_labels[0]));
+        pi = 0;
+      }
+      g_eqcnt = 0;
+      for (i = 0; i < g_func_pd_cnt; i++) {
+        pd = &g_func_pd[i];
+        if (pd->type == OPT_OFFSET) {
+          for (j = 0; j < pd->count; j++)
+            free(pd->d[j].u.label);
+        }
+        free(pd->d);
+        pd->d = NULL;
+      }
+      g_func_pd_cnt = 0;
+      pd = NULL;
+      if (wordc == 0)
+        continue;
+    }
+
     if (IS(words[1], "proc")) {
       if (in_func)
         aerr("proc '%s' while in_func '%s'?\n",
           words[0], g_func);
+      p = words[0];
+      if ((g_ida_func_attr & IDAFA_THUNK)
+       || bsearch(&p, rlist, rlist_len, sizeof(rlist[0]), cmpstringp))
+        skip_func = 1;
       strcpy(g_func, words[0]);
+      set_label(0, words[0]);
       in_func = 1;
       continue;
     }
 
-    if (IS(words[1], "endp")) {
+    if (IS(words[1], "endp"))
+    {
       if (!in_func)
         aerr("endp '%s' while not in_func?\n", words[0]);
       if (!IS(g_func, words[0]))
         aerr("endp '%s' while in_func '%s'?\n",
           words[0], g_func);
-      gen_func(fout, fhdr, g_func, pi);
-      in_func = 0;
-      g_func[0] = 0;
-      if (pi != 0) {
-        memset(&ops, 0, pi * sizeof(ops[0]));
-        memset(g_labels, 0, pi * sizeof(g_labels[0]));
-        pi = 0;
+
+      if (!skip_func && func_chunks_used) {
+        // start processing chunks
+        struct chunk_item *ci, key = { g_func, 0 };
+
+        func_chunk_ret = ftell(fasm);
+        func_chunk_ret_ln = asmln;
+        if (!func_chunks_sorted) {
+          qsort(func_chunks, func_chunk_cnt,
+            sizeof(func_chunks[0]), cmp_chunks);
+          func_chunks_sorted = 1;
+        }
+        ci = bsearch(&key, func_chunks, func_chunk_cnt,
+               sizeof(func_chunks[0]), cmp_chunks);
+        if (ci == NULL)
+          aerr("'%s' needs chunks, but none found\n", g_func);
+        func_chunk_i = ci - func_chunks;
+        for (; func_chunk_i > 0; func_chunk_i--)
+          if (!IS(func_chunks[func_chunk_i - 1].name, g_func))
+            break;
+
+        ret = fseek(fasm, func_chunks[func_chunk_i].fptr, SEEK_SET);
+        if (ret)
+          aerr("seek failed for '%s' chunk #%d\n", g_func, func_chunk_i);
+        asmln = func_chunks[func_chunk_i].asmln;
+        func_chunk_i++;
+        continue;
       }
-      g_eqcnt = 0;
+      pending_endp = 1;
       continue;
     }
 
-    if (IS(words[1], "=")) {
+    p = strchr(words[0], ':');
+    if (p != NULL) {
+      set_label(pi, words[0]);
+      continue;
+    }
+
+    if (!in_func || skip_func) {
+      if (!skip_warned && !skip_func && g_labels[pi][0] != 0) {
+        if (verbose)
+          anote("skipping from '%s'\n", g_labels[pi]);
+        skip_warned = 1;
+      }
+      g_labels[pi][0] = 0;
+      continue;
+    }
+
+    if (wordc > 1 && IS(words[1], "=")) {
       if (wordc != 5)
         aerr("unhandled equ, wc=%d\n", wordc);
       if (g_eqcnt >= eq_alloc) {
@@ -1743,27 +3756,13 @@ int main(int argc, char *argv[])
     if (pi >= ARRAY_SIZE(ops))
       aerr("too many ops\n");
 
-    p = strchr(words[0], ':');
-    if (p != NULL) {
-      len = p - words[0];
-      if (len > sizeof(g_labels[0]) - 1)
-        aerr("label too long: %d\n", len);
-      if (g_labels[pi][0] != 0)
-        aerr("dupe label?\n");
-      memcpy(g_labels[pi], words[0], len);
-      g_labels[pi][len] = 0;
-      continue;
-    }
-
     parse_op(&ops[pi], words, wordc);
     pi++;
-
-    (void)proto_parse;
   }
 
   fclose(fout);
   fclose(fasm);
-  fclose(fhdr);
+  fclose(g_fhdr);
 
   return 0;
 }