Reduce code duplication and resolve inconsistencies (due to code duplication) in...
[libfirm] / ir / be / ia32 / ia32_emitter.c
index 82b8ef6..d2dddc2 100644 (file)
  * @version     $Id$
  *
  * Summary table for x86 floatingpoint compares:
+ * (remember effect of unordered on x86: ZF=1, PF=1, CF=1)
+ *
  *   pnc_Eq  => !P && E
  *   pnc_Lt  => !P && B
  *   pnc_Le  => !P && BE
  *   pnc_Gt  => A
  *   pnc_Ge  => AE
- *   pnc_Lg  => P || NE
+ *   pnc_Lg  => NE
  *   pnc_Leg => NP  (ordered)
  *   pnc_Uo  => P
  *   pnc_Ue  => E
@@ -37,7 +39,7 @@
  *   pnc_Ule => BE
  *   pnc_Ug  => P || A
  *   pnc_Uge => P || AE
- *   pnc_Ne  => NE
+ *   pnc_Ne  => P || NE
  */
 #include "config.h"
 
@@ -497,53 +499,43 @@ static void ia32_emit_cfop_target(const ir_node *node)
        be_gas_emit_block_name(block);
 }
 
-/*
- * positive conditions for signed compares
- */
-static const char *const cmp2condition_s[] = {
-       NULL, /* always false */
-       "e",  /* == */
-       "l",  /* <  */
-       "le", /* <= */
-       "g",  /* >  */
-       "ge", /* >= */
-       "ne", /* != */
-       NULL  /* always true */
-};
-
-/*
- * positive conditions for unsigned compares
- */
-static const char *const cmp2condition_u[] = {
-       NULL, /* always false */
-       "e",  /* == */
-       "b",  /* <  */
-       "be", /* <= */
-       "a",  /* >  */
-       "ae", /* >= */
-       "ne", /* != */
-       NULL  /* always true */
-};
-
 /**
  * Emit the suffix for a compare instruction.
  */
-static void ia32_emit_cmp_suffix(int pnc)
-{
-       const char *str;
-
-       if (pnc == ia32_pn_Cmp_parity) {
-               be_emit_char('p');
-               return;
-       }
-
-       if (pnc & ia32_pn_Cmp_float || pnc & ia32_pn_Cmp_unsigned) {
-               str = cmp2condition_u[pnc & 7];
-       } else {
-               str = cmp2condition_s[pnc & 7];
+static void ia32_emit_condition_code(ia32_condition_code_t cc)
+{
+       switch (cc) {
+       case ia32_cc_overflow:      be_emit_cstring("o");  return;
+       case ia32_cc_not_overflow:  be_emit_cstring("no"); return;
+       case ia32_cc_float_below:
+       case ia32_cc_float_unordered_below:
+       case ia32_cc_below:         be_emit_cstring("b");  return;
+       case ia32_cc_float_above_equal:
+       case ia32_cc_float_unordered_above_equal:
+       case ia32_cc_above_equal:   be_emit_cstring("ae"); return;
+       case ia32_cc_float_equal:
+       case ia32_cc_equal:         be_emit_cstring("e");  return;
+       case ia32_cc_float_not_equal:
+       case ia32_cc_not_equal:     be_emit_cstring("ne"); return;
+       case ia32_cc_float_below_equal:
+       case ia32_cc_float_unordered_below_equal:
+       case ia32_cc_below_equal:   be_emit_cstring("be"); return;
+       case ia32_cc_float_above:
+       case ia32_cc_float_unordered_above:
+       case ia32_cc_above:         be_emit_cstring("a");  return;
+       case ia32_cc_sign:          be_emit_cstring("s");  return;
+       case ia32_cc_not_sign:      be_emit_cstring("ns"); return;
+       case ia32_cc_parity:        be_emit_cstring("p");  return;
+       case ia32_cc_not_parity:    be_emit_cstring("np"); return;
+       case ia32_cc_less:          be_emit_cstring("l");  return;
+       case ia32_cc_greater_equal: be_emit_cstring("ge"); return;
+       case ia32_cc_less_equal:    be_emit_cstring("le"); return;
+       case ia32_cc_greater:       be_emit_cstring("g");  return;
+       case ia32_cc_float_parity_cases:
+       case ia32_cc_additional_float_cases:
+               break;
        }
-
-       be_emit_string(str);
+       panic("Invalid ia32 condition code");
 }
 
 typedef enum ia32_emit_mod_t {
@@ -565,12 +557,15 @@ void ia32_emit_am(const ir_node *node)
        int        offs      = get_ia32_am_offs_int(node);
        ir_node   *base      = get_irn_n(node, n_ia32_base);
        int        has_base  = !is_ia32_NoReg_GP(base);
-       ir_node   *index     = get_irn_n(node, n_ia32_index);
-       int        has_index = !is_ia32_NoReg_GP(index);
+       ir_node   *idx       = get_irn_n(node, n_ia32_index);
+       int        has_index = !is_ia32_NoReg_GP(idx);
 
        /* just to be sure... */
        assert(!is_ia32_use_frame(node) || get_ia32_frame_ent(node) != NULL);
 
+       if (get_ia32_am_tls_segment(node))
+               be_emit_cstring("%gs:");
+
        /* emit offset */
        if (ent != NULL) {
                const ia32_attr_t *attr = get_ia32_attr_const(node);
@@ -680,6 +675,9 @@ static void ia32_emitf(const ir_node *node, const char *fmt, ...)
 end_of_mods:
 
                switch (*fmt++) {
+                       arch_register_t const *reg;
+                       ir_node         const *imm;
+
                        case '%':
                                be_emit_char('%');
                                break;
@@ -693,17 +691,13 @@ emit_AM:
                                                ia32_emit_am(node);
                                                break;
 
-                                       case 'R': {
-                                               const arch_register_t *reg = va_arg(ap, const arch_register_t*);
+                                       case 'R':
+                                               reg = va_arg(ap, const arch_register_t*);
                                                if (get_ia32_op_type(node) == ia32_AddrModeS) {
                                                        goto emit_AM;
                                                } else {
-                                                       if (mod & EMIT_ALTERNATE_AM)
-                                                               be_emit_char('*');
-                                                       emit_register(reg, NULL);
+                                                       goto emit_R;
                                                }
-                                               break;
-                                       }
 
                                        case 'S':
                                                if (get_ia32_op_type(node) == ia32_AddrModeS) {
@@ -713,49 +707,45 @@ emit_AM:
                                                        assert(get_ia32_op_type(node) == ia32_Normal);
                                                        goto emit_S;
                                                }
-                                               break;
 
                                        default: goto unknown;
                                }
                                break;
                        }
 
-                       case 'D': {
-                               unsigned               pos;
-                               const arch_register_t *reg;
-
+                       case 'D':
                                if (*fmt < '0' || '9' <= *fmt)
                                        goto unknown;
-
-                               pos = *fmt++ - '0';
-                               reg = get_out_reg(node, pos);
-                               emit_register(reg, mod & EMIT_RESPECT_LS ? get_ia32_ls_mode(node) : NULL);
-                               break;
-                       }
+                               reg = get_out_reg(node, *fmt++ - '0');
+                               goto emit_R;
 
                        case 'I':
+                               imm = node;
+emit_I:
                                if (!(mod & EMIT_ALTERNATE_AM))
                                        be_emit_char('$');
-                               emit_ia32_Immediate_no_prefix(node);
+                               emit_ia32_Immediate_no_prefix(imm);
                                break;
 
                        case 'L':
                                ia32_emit_cfop_target(node);
                                break;
 
-                       case 'M': {
+                       case 'M':
                                ia32_emit_mode_suffix_mode(get_ia32_ls_mode(node));
                                break;
-                       }
 
                        case 'P': {
-                               int pnc = va_arg(ap, int);
-                               ia32_emit_cmp_suffix(pnc);
+                               ia32_condition_code_t cc = va_arg(ap, ia32_condition_code_t);
+                               ia32_emit_condition_code(cc);
                                break;
                        }
 
-                       case 'R': {
-                               const arch_register_t *reg = va_arg(ap, const arch_register_t*);
+                       case 'R':
+                               reg = va_arg(ap, const arch_register_t*);
+emit_R:
+                               if (mod & EMIT_ALTERNATE_AM)
+                                       be_emit_char('*');
                                if (mod & EMIT_HIGH_REG) {
                                        emit_8bit_register_high(reg);
                                } else if (mod & EMIT_LOW_REG) {
@@ -764,31 +754,22 @@ emit_AM:
                                        emit_register(reg, mod & EMIT_RESPECT_LS ? get_ia32_ls_mode(node) : NULL);
                                }
                                break;
-                       }
 
 emit_S:
                        case 'S': {
-                               unsigned       pos;
-                               const ir_node *in;
+                               unsigned pos;
 
                                if (*fmt < '0' || '9' <= *fmt)
                                        goto unknown;
 
                                pos = *fmt++ - '0';
-                               in  = get_irn_n(node, pos);
-                               if (is_ia32_Immediate(in)) {
-                                       if (!(mod & EMIT_ALTERNATE_AM))
-                                               be_emit_char('$');
-                                       emit_ia32_Immediate_no_prefix(in);
+                               imm = get_irn_n(node, pos);
+                               if (is_ia32_Immediate(imm)) {
+                                       goto emit_I;
                                } else {
-                                       const arch_register_t *reg;
-
-                                       if (mod & EMIT_ALTERNATE_AM)
-                                               be_emit_char('*');
                                        reg = get_in_reg(node, pos);
-                                       emit_register(reg, mod & EMIT_RESPECT_LS ? get_ia32_ls_mode(node) : NULL);
+                                       goto emit_R;
                                }
-                               break;
                        }
 
                        case 's': {
@@ -939,7 +920,7 @@ static ir_node *find_original_value(ir_node *node)
        }
 }
 
-static int determine_final_pnc(const ir_node *node, int flags_pos, int pnc)
+static int determine_final_cc(const ir_node *node, int flags_pos, int cc)
 {
        ir_node           *flags = get_irn_n(node, flags_pos);
        const ia32_attr_t *flags_attr;
@@ -957,40 +938,21 @@ static int determine_final_pnc(const ir_node *node, int flags_pos, int pnc)
                }
 
                flags_attr = get_ia32_attr_const(cmp);
-               if (flags_attr->data.ins_permuted)
-                       pnc = get_mirrored_pnc(pnc);
-               pnc |= ia32_pn_Cmp_float;
-       } else if (is_ia32_Ucomi(flags) || is_ia32_Fucomi(flags)
-                       || is_ia32_Fucompi(flags)) {
-               flags_attr = get_ia32_attr_const(flags);
-
-               if (flags_attr->data.ins_permuted)
-                       pnc = get_mirrored_pnc(pnc);
-               pnc |= ia32_pn_Cmp_float;
        } else {
                flags_attr = get_ia32_attr_const(flags);
-
-               if (flags_attr->data.ins_permuted)
-                       pnc = get_mirrored_pnc(pnc);
-               if (flags_attr->data.cmp_unsigned)
-                       pnc |= ia32_pn_Cmp_unsigned;
        }
 
-       return pnc;
-}
-
-static int ia32_get_negated_pnc(int pnc)
-{
-       ir_mode *mode = pnc & ia32_pn_Cmp_float ? mode_F : mode_Iu;
-       return get_negated_pnc(pnc, mode);
+       if (flags_attr->data.ins_permuted)
+               cc = ia32_invert_condition_code(cc);
+       return cc;
 }
 
 void ia32_emit_cmp_suffix_node(const ir_node *node, int flags_pos)
 {
-       int pnc = get_ia32_condcode(node);
-       pnc = determine_final_pnc(node, flags_pos, pnc);
+       ia32_condition_code_t cc = get_ia32_condcode(node);
+       cc = determine_final_cc(node, flags_pos, cc);
 
-       ia32_emit_cmp_suffix(pnc);
+       ia32_emit_condition_code(cc);
 }
 
 /**
@@ -1037,12 +999,12 @@ static int can_be_fallthrough(const ir_node *node)
  */
 static void emit_ia32_Jcc(const ir_node *node)
 {
-       int            need_parity_label = 0;
-       const ir_node *proj_true;
-       const ir_node *proj_false;
-       int            pnc = get_ia32_condcode(node);
+       int                   need_parity_label = 0;
+       ia32_condition_code_t cc                = get_ia32_condcode(node);
+       const ir_node        *proj_true;
+       const ir_node        *proj_false;
 
-       pnc = determine_final_pnc(node, 0, pnc);
+       cc = determine_final_cc(node, 0, cc);
 
        /* get both Projs */
        proj_true = get_proj(node, pn_ia32_Jcc_true);
@@ -1057,25 +1019,15 @@ static void emit_ia32_Jcc(const ir_node *node)
 
                proj_true  = proj_false;
                proj_false = t;
-               pnc        = ia32_get_negated_pnc(pnc);
+               cc         = ia32_negate_condition_code(cc);
        }
 
-       if (pnc & ia32_pn_Cmp_float) {
+       if (cc & ia32_cc_float_parity_cases) {
                /* Some floating point comparisons require a test of the parity flag,
                 * which indicates that the result is unordered */
-               switch (pnc & 0x0f) {
-               case pn_Cmp_Uo: {
+               if (cc & ia32_cc_negated) {
                        ia32_emitf(proj_true, "\tjp %L\n");
-                       break;
-               }
-
-               case pn_Cmp_Leg:
-                       ia32_emitf(proj_true, "\tjnp %L\n");
-                       break;
-
-               case pn_Cmp_Eq:
-               case pn_Cmp_Lt:
-               case pn_Cmp_Le:
+               } else {
                        /* we need a local label if the false proj is a fallthrough
                         * as the falseblock might have no label emitted then */
                        if (can_be_fallthrough(proj_false)) {
@@ -1084,22 +1036,9 @@ static void emit_ia32_Jcc(const ir_node *node)
                        } else {
                                ia32_emitf(proj_false, "\tjp %L\n");
                        }
-                       goto emit_jcc;
-
-               case pn_Cmp_Ug:
-               case pn_Cmp_Uge:
-               case pn_Cmp_Ne:
-                       ia32_emitf(proj_true, "\tjp %L\n");
-                       goto emit_jcc;
-
-               default:
-                       goto emit_jcc;
                }
-       } else {
-emit_jcc:
-               ia32_emitf(proj_true, "\tj%P %L\n", pnc);
        }
-
+       ia32_emitf(proj_true, "\tj%P %L\n", cc);
        if (need_parity_label) {
                ia32_emitf(NULL, "1:\n");
        }
@@ -1120,56 +1059,38 @@ static void emit_ia32_Setcc(const ir_node *node)
 {
        const arch_register_t *dreg = get_out_reg(node, pn_ia32_Setcc_res);
 
-       int pnc = get_ia32_condcode(node);
-       pnc     = determine_final_pnc(node, n_ia32_Setcc_eflags, pnc);
-       if (pnc & ia32_pn_Cmp_float) {
-               switch (pnc & 0x0f) {
-               case pn_Cmp_Uo:
-                       ia32_emitf(node, "\tsetp %#R\n", dreg);
-                       return;
-
-               case pn_Cmp_Leg:
-                       ia32_emitf(node, "\tsetnp %#R\n", dreg);
-                       return;
-
-               case pn_Cmp_Eq:
-               case pn_Cmp_Lt:
-               case pn_Cmp_Le:
-                       ia32_emitf(node, "\tset%P %<R\n", pnc, dreg);
-                       ia32_emitf(node, "\tsetnp %>R\n", dreg);
-                       ia32_emitf(node, "\tandb %>R, %<R\n", dreg, dreg);
-                       return;
-
-               case pn_Cmp_Ug:
-               case pn_Cmp_Uge:
-               case pn_Cmp_Ne:
-                       ia32_emitf(node, "\tset%P %<R\n", pnc, dreg);
+       ia32_condition_code_t cc = get_ia32_condcode(node);
+       cc = determine_final_cc(node, n_ia32_Setcc_eflags, cc);
+       if (cc & ia32_cc_float_parity_cases) {
+               if (cc & ia32_cc_negated) {
+                       ia32_emitf(node, "\tset%P %<R\n", cc, dreg);
                        ia32_emitf(node, "\tsetp %>R\n", dreg);
                        ia32_emitf(node, "\torb %>R, %<R\n", dreg, dreg);
-                       return;
-
-               default:
-                       break;
+               } else {
+                       ia32_emitf(node, "\tset%P %<R\n", cc, dreg);
+                       ia32_emitf(node, "\tsetnp %>R\n", dreg);
+                       ia32_emitf(node, "\tandb %>R, %<R\n", dreg, dreg);
                }
+       } else {
+               ia32_emitf(node, "\tset%P %#R\n", cc, dreg);
        }
-       ia32_emitf(node, "\tset%P %#R\n", pnc, dreg);
 }
 
 static void emit_ia32_CMovcc(const ir_node *node)
 {
        const ia32_attr_t     *attr = get_ia32_attr_const(node);
        const arch_register_t *out  = arch_irn_get_register(node, pn_ia32_res);
-       int                    pnc  = get_ia32_condcode(node);
+       ia32_condition_code_t  cc   = get_ia32_condcode(node);
        const arch_register_t *in_true;
        const arch_register_t *in_false;
 
-       pnc = determine_final_pnc(node, n_ia32_CMovcc_eflags, pnc);
+       cc = determine_final_cc(node, n_ia32_CMovcc_eflags, cc);
        /* although you can't set ins_permuted in the constructor it might still
         * be set by memory operand folding
         * Permuting inputs of a cmov means the condition is negated!
         */
        if (attr->data.ins_permuted)
-               pnc = ia32_get_negated_pnc(pnc);
+               cc = ia32_negate_condition_code(cc);
 
        in_true  = arch_get_irn_register(get_irn_n(node, n_ia32_CMovcc_val_true));
        in_false = arch_get_irn_register(get_irn_n(node, n_ia32_CMovcc_val_false));
@@ -1182,7 +1103,7 @@ static void emit_ia32_CMovcc(const ir_node *node)
 
                assert(get_ia32_op_type(node) == ia32_Normal);
 
-               pnc = ia32_get_negated_pnc(pnc);
+               cc = ia32_negate_condition_code(cc);
 
                tmp      = in_true;
                in_true  = in_false;
@@ -1192,146 +1113,24 @@ static void emit_ia32_CMovcc(const ir_node *node)
                ia32_emitf(node, "\tmovl %R, %R\n", in_false, out);
        }
 
-       /* TODO: handling of Nans isn't correct yet */
-       if (pnc & ia32_pn_Cmp_float) {
-               switch (pnc & 0x0f) {
-               case pn_Cmp_Uo:
-               case pn_Cmp_Leg:
-               case pn_Cmp_Eq:
-               case pn_Cmp_Lt:
-               case pn_Cmp_Le:
-               case pn_Cmp_Ug:
-               case pn_Cmp_Uge:
-               case pn_Cmp_Ne:
-                       panic("CMov with floatingpoint compare/parity not supported yet");
-               }
-       }
-
-       ia32_emitf(node, "\tcmov%P %#AR, %#R\n", pnc, in_true, out);
-}
-
-
-/* jump table entry (target and corresponding number) */
-typedef struct branch_t {
-       ir_node *target;
-       int      value;
-} branch_t;
-
-/* jump table for switch generation */
-typedef struct jmp_tbl_t {
-       ir_node  *defProj;                 /**< default target */
-       long      min_value;               /**< smallest switch case */
-       long      max_value;               /**< largest switch case */
-       long      num_branches;            /**< number of jumps */
-       char      label[SNPRINTF_BUF_LEN]; /**< label of the jump table */
-       branch_t *branches;                /**< jump array */
-} jmp_tbl_t;
-
-/**
- * Compare two variables of type branch_t. Used to sort all switch cases
- */
-static int ia32_cmp_branch_t(const void *a, const void *b)
-{
-       branch_t *b1 = (branch_t *)a;
-       branch_t *b2 = (branch_t *)b;
-
-       if (b1->value <= b2->value)
-               return -1;
-       else
-               return 1;
-}
-
-static void generate_jump_table(jmp_tbl_t *tbl, const ir_node *node)
-{
-       int                 i;
-       long                pnc;
-       long                default_pn;
-       ir_node            *proj;
-       const ir_edge_t    *edge;
-
-       /* fill the table structure */
-       get_unique_label(tbl->label, SNPRINTF_BUF_LEN, "TBL_");
-       tbl->defProj      = NULL;
-       tbl->num_branches = get_irn_n_edges(node) - 1;
-       tbl->branches     = XMALLOCNZ(branch_t, tbl->num_branches);
-       tbl->min_value    = LONG_MAX;
-       tbl->max_value    = LONG_MIN;
-
-       default_pn = get_ia32_condcode(node);
-       i = 0;
-       /* go over all proj's and collect them */
-       foreach_out_edge(node, edge) {
-               proj = get_edge_src_irn(edge);
-               assert(is_Proj(proj) && "Only proj allowed at SwitchJmp");
-
-               pnc = get_Proj_proj(proj);
-
-               /* check for default proj */
-               if (pnc == default_pn) {
-                       assert(tbl->defProj == NULL && "found two default Projs at SwitchJmp");
-                       tbl->defProj = proj;
-               } else {
-                       tbl->min_value = pnc < tbl->min_value ? pnc : tbl->min_value;
-                       tbl->max_value = pnc > tbl->max_value ? pnc : tbl->max_value;
-
-                       /* create branch entry */
-                       tbl->branches[i].target = proj;
-                       tbl->branches[i].value  = pnc;
-                       ++i;
-               }
-
+       if (cc & ia32_cc_float_parity_cases) {
+               panic("CMov with floatingpoint compare/parity not supported yet");
        }
-       assert(i == tbl->num_branches);
 
-       /* sort the branches by their number */
-       qsort(tbl->branches, tbl->num_branches, sizeof(tbl->branches[0]), ia32_cmp_branch_t);
+       ia32_emitf(node, "\tcmov%P %#AR, %#R\n", cc, in_true, out);
 }
 
 /**
- * Emits code for a SwitchJmp (creates a jump table if
- * possible otherwise a cmp-jmp cascade). Port from
- * cggg ia32 backend
+ * Emits code for a SwitchJmp
  */
 static void emit_ia32_SwitchJmp(const ir_node *node)
 {
-       unsigned long       interval;
-       int                 last_value, i;
-       jmp_tbl_t           tbl;
-
-       /* fill the table structure */
-       generate_jump_table(&tbl, node);
-
-       /* two-complement's magic make this work without overflow */
-       interval = tbl.max_value - tbl.min_value;
-
-       /* emit the table */
-       ia32_emitf(node,        "\tcmpl $%u, %S0\n", interval);
-       ia32_emitf(tbl.defProj, "\tja %L\n");
-
-       if (tbl.num_branches > 1) {
-               /* create table */
-               ia32_emitf(node, "\tjmp *%s(,%S0,4)\n", tbl.label);
-
-               be_gas_emit_switch_section(GAS_SECTION_RODATA);
-               ia32_emitf(NULL, "\t.align 4\n");
-               ia32_emitf(NULL, "%s:\n", tbl.label);
+       ir_entity *jump_table = get_ia32_am_sc(node);
+       long       default_pn = get_ia32_default_pn(node);
 
-               last_value = tbl.branches[0].value;
-               for (i = 0; i != tbl.num_branches; ++i) {
-                       while (last_value != tbl.branches[i].value) {
-                               ia32_emitf(tbl.defProj, ".long %L\n");
-                               ++last_value;
-                       }
-                       ia32_emitf(tbl.branches[i].target, ".long %L\n");
-                       ++last_value;
-               }
-               be_gas_emit_switch_section(GAS_SECTION_TEXT);
-       } else {
-               /* one jump is enough */
-               ia32_emitf(tbl.branches[0].target, "\tjmp %L\n");
-       }
+       ia32_emitf(node, "\tjmp *%AM\n", node);
 
-       free(tbl.branches);
+       emit_jump_table(node, default_pn, jump_table, get_cfop_target_block);
 }
 
 /**
@@ -1339,11 +1138,6 @@ static void emit_ia32_SwitchJmp(const ir_node *node)
  */
 static void emit_ia32_Jmp(const ir_node *node)
 {
-       ir_node *block;
-
-       /* for now, the code works for scheduled and non-schedules blocks */
-       block = get_nodes_block(node);
-
        /* we have a block schedule */
        if (can_be_fallthrough(node)) {
                ia32_emitf(node, "\t/* fallthrough to %L */\n");
@@ -1370,7 +1164,7 @@ static const char* emit_asm_operand(const ir_node *node, const char *s)
        const ia32_asm_reg_t  *asm_reg;
        char                   c;
        char                   modifier = 0;
-       int                    num      = -1;
+       int                    num;
        int                    p;
 
        assert(*s == '%');
@@ -1411,8 +1205,7 @@ static const char* emit_asm_operand(const ir_node *node, const char *s)
        }
 
        /* parse number */
-       sscanf(s, "%d%n", &num, &p);
-       if (num < 0) {
+       if (sscanf(s, "%d%n", &num, &p) != 1) {
                ir_fprintf(stderr, "Warning: Couldn't parse assembler operand (%+F)\n",
                           node);
                return s;
@@ -1578,8 +1371,8 @@ static void emit_ia32_Conv_FP2FP(const ir_node *node)
  */
 static void emit_ia32_Conv_I2I(const ir_node *node)
 {
-       ir_mode *smaller_mode = get_ia32_ls_mode(node);
-       int      signed_mode  = mode_is_signed(smaller_mode);
+       ir_mode    *smaller_mode = get_ia32_ls_mode(node);
+       int         signed_mode  = mode_is_signed(smaller_mode);
        const char *sign_suffix;
 
        assert(!mode_is_float(smaller_mode));
@@ -1687,14 +1480,6 @@ static void emit_ia32_Const(const ir_node *node)
        ia32_emitf(node, "\tmovl %I, %D0\n");
 }
 
-/**
- * Emits code to load the TLS base
- */
-static void emit_ia32_LdTls(const ir_node *node)
-{
-       ia32_emitf(node, "\tmovl %%gs:0, %D0\n");
-}
-
 /* helper function for emit_ia32_Minus64Bit */
 static void emit_mov(const ir_node* node, const arch_register_t *src, const arch_register_t *dst)
 {
@@ -1860,7 +1645,6 @@ static void ia32_register_emitters(void)
        IA32_EMIT(IMul);
        IA32_EMIT(Jcc);
        IA32_EMIT(Setcc);
-       IA32_EMIT(LdTls);
        IA32_EMIT(Minus64Bit);
        IA32_EMIT(SwitchJmp);
        IA32_EMIT(ClimbFrame);
@@ -1873,7 +1657,6 @@ static void ia32_register_emitters(void)
        BE_EMIT(Perm);
        BE_EMIT(Return);
 
-       BE_IGN(Barrier);
        BE_IGN(Keep);
        BE_IGN(Start);
 
@@ -2017,10 +1800,10 @@ static int should_align_block(const ir_node *block)
  */
 static void ia32_emit_block_header(ir_node *block)
 {
-       ir_graph     *irg = current_ir_graph;
+       ir_graph     *irg        = current_ir_graph;
        int           need_label = block_needs_label(block);
-       int           i, arity;
-       ir_exec_freq *exec_freq = be_get_irg_exec_freq(irg);
+       ir_exec_freq *exec_freq  = be_get_irg_exec_freq(irg);
+       int           arity;
 
        if (block == get_irg_end_block(irg))
                return;
@@ -2070,6 +1853,7 @@ static void ia32_emit_block_header(ir_node *block)
        if (arity <= 0) {
                be_emit_cstring(" none");
        } else {
+               int i;
                for (i = 0; i < arity; ++i) {
                        ir_node *predblock = get_Block_cfgpred_block(block, i);
                        be_emit_irprintf(" %d", get_irn_node_nr(predblock));
@@ -2199,14 +1983,14 @@ void ia32_gen_routine(ir_graph *irg)
           Those are ascending with ascending addresses. */
        qsort(exc_list, ARR_LEN(exc_list), sizeof(exc_list[0]), cmp_exc_entry);
        {
-               size_t i;
+               size_t e;
 
-               for (i = 0; i < ARR_LEN(exc_list); ++i) {
+               for (e = 0; e < ARR_LEN(exc_list); ++e) {
                        be_emit_cstring("\t.long ");
-                       ia32_emit_exc_label(exc_list[i].exc_instr);
+                       ia32_emit_exc_label(exc_list[e].exc_instr);
                        be_emit_char('\n');
                        be_emit_cstring("\t.long ");
-                       be_gas_emit_block_name(exc_list[i].block);
+                       be_gas_emit_block_name(exc_list[e].block);
                        be_emit_char('\n');
                }
        }
@@ -2223,8 +2007,6 @@ static const lc_opt_table_entry_t ia32_emitter_options[] = {
 static unsigned char reg_gp_map[N_ia32_gp_REGS];
 //static unsigned char reg_mmx_map[N_ia32_mmx_REGS];
 //static unsigned char reg_sse_map[N_ia32_xmm_REGS];
-static unsigned char pnc_map_signed[8];
-static unsigned char pnc_map_unsigned[8];
 
 static void build_reg_map(void)
 {
@@ -2236,35 +2018,12 @@ static void build_reg_map(void)
        reg_gp_map[REG_GP_EBP] = 0x5;
        reg_gp_map[REG_GP_ESI] = 0x6;
        reg_gp_map[REG_GP_EDI] = 0x7;
-
-       pnc_map_signed[pn_Cmp_Eq]    = 0x04;
-       pnc_map_signed[pn_Cmp_Lt]    = 0x0C;
-       pnc_map_signed[pn_Cmp_Le]    = 0x0E;
-       pnc_map_signed[pn_Cmp_Gt]    = 0x0F;
-       pnc_map_signed[pn_Cmp_Ge]    = 0x0D;
-       pnc_map_signed[pn_Cmp_Lg]    = 0x05;
-
-       pnc_map_unsigned[pn_Cmp_Eq]    = 0x04;
-       pnc_map_unsigned[pn_Cmp_Lt]    = 0x02;
-       pnc_map_unsigned[pn_Cmp_Le]    = 0x06;
-       pnc_map_unsigned[pn_Cmp_Gt]    = 0x07;
-       pnc_map_unsigned[pn_Cmp_Ge]    = 0x03;
-       pnc_map_unsigned[pn_Cmp_Lg]    = 0x05;
 }
 
 /** Returns the encoding for a pnc field. */
-static unsigned char pnc2cc(int pnc)
+static unsigned char pnc2cc(ia32_condition_code_t cc)
 {
-       unsigned char cc;
-       if (pnc == ia32_pn_Cmp_parity) {
-               cc = 0x0A;
-       } else if (pnc & ia32_pn_Cmp_float || pnc & ia32_pn_Cmp_unsigned) {
-               cc = pnc_map_unsigned[pnc & 0x07];
-       } else {
-               cc = pnc_map_signed[pnc & 0x07];
-       }
-       assert(cc != 0);
-       return cc;
+       return cc & 0xf;
 }
 
 /** Sign extension bit values for binops */
@@ -2437,8 +2196,8 @@ static void bemit_mod_am(unsigned reg, const ir_node *node)
        int        offs      = get_ia32_am_offs_int(node);
        ir_node   *base      = get_irn_n(node, n_ia32_base);
        int        has_base  = !is_ia32_NoReg_GP(base);
-       ir_node   *index     = get_irn_n(node, n_ia32_index);
-       int        has_index = !is_ia32_NoReg_GP(index);
+       ir_node   *idx       = get_irn_n(node, n_ia32_index);
+       int        has_index = !is_ia32_NoReg_GP(idx);
        unsigned   modrm     = 0;
        unsigned   sib       = 0;
        unsigned   emitoffs  = 0;
@@ -2473,7 +2232,7 @@ static void bemit_mod_am(unsigned reg, const ir_node *node)
 
        /* Determine if we need a SIB byte. */
        if (has_index) {
-               const arch_register_t *reg_index = arch_get_irn_register(index);
+               const arch_register_t *reg_index = arch_get_irn_register(idx);
                int                    scale     = get_ia32_am_scale(node);
                assert(scale < 4);
                /* R/M set to ESP means SIB in 32bit mode. */
@@ -2876,30 +2635,27 @@ static void bemit_setcc(const ir_node *node)
 {
        const arch_register_t *dreg = get_out_reg(node, pn_ia32_Setcc_res);
 
-       int pnc = get_ia32_condcode(node);
-       pnc     = determine_final_pnc(node, n_ia32_Setcc_eflags, pnc);
-       if (pnc & ia32_pn_Cmp_float) {
-               switch (pnc & 0x0f) {
-               case pn_Cmp_Uo:
-                        /* setp <dreg */
+       ia32_condition_code_t cc = get_ia32_condcode(node);
+       cc = determine_final_cc(node, n_ia32_Setcc_eflags, cc);
+       if (cc & ia32_cc_float_parity_cases) {
+               if (cc & ia32_cc_negated) {
+                       /* set%PNC <dreg */
                        bemit8(0x0F);
-                       bemit8(0x9A);
+                       bemit8(0x90 | pnc2cc(cc));
                        bemit_modrm8(REG_LOW, dreg);
-                       return;
 
-               case pn_Cmp_Leg:
-                        /* setnp <dreg*/
+                       /* setp >dreg */
                        bemit8(0x0F);
-                       bemit8(0x9B);
-                       bemit_modrm8(REG_LOW, dreg);
-                       return;
+                       bemit8(0x9A);
+                       bemit_modrm8(REG_HIGH, dreg);
 
-               case pn_Cmp_Eq:
-               case pn_Cmp_Lt:
-               case pn_Cmp_Le:
+                       /* orb %>dreg, %<dreg */
+                       bemit8(0x08);
+                       bemit_modrr8(REG_LOW, dreg, REG_HIGH, dreg);
+               } else {
                         /* set%PNC <dreg */
                        bemit8(0x0F);
-                       bemit8(0x90 | pnc2cc(pnc));
+                       bemit8(0x90 | pnc2cc(cc));
                        bemit_modrm8(REG_LOW, dreg);
 
                        /* setnp >dreg */
@@ -2910,34 +2666,13 @@ static void bemit_setcc(const ir_node *node)
                        /* andb %>dreg, %<dreg */
                        bemit8(0x20);
                        bemit_modrr8(REG_LOW, dreg, REG_HIGH, dreg);
-                       return;
-
-               case pn_Cmp_Ug:
-               case pn_Cmp_Uge:
-               case pn_Cmp_Ne:
-                       /* set%PNC <dreg */
-                       bemit8(0x0F);
-                       bemit8(0x90 | pnc2cc(pnc));
-                       bemit_modrm8(REG_LOW, dreg);
-
-                       /* setp >dreg */
-                       bemit8(0x0F);
-                       bemit8(0x9A);
-                       bemit_modrm8(REG_HIGH, dreg);
-
-                       /* orb %>dreg, %<dreg */
-                       bemit8(0x08);
-                       bemit_modrr8(REG_LOW, dreg, REG_HIGH, dreg);
-                       return;
-
-               default:
-                       break;
                }
+       } else {
+               /* set%PNC <dreg */
+               bemit8(0x0F);
+               bemit8(0x90 | pnc2cc(cc));
+               bemit_modrm8(REG_LOW, dreg);
        }
-       /* set%PNC <dreg */
-       bemit8(0x0F);
-       bemit8(0x90 | pnc2cc(pnc));
-       bemit_modrm8(REG_LOW, dreg);
 }
 
 static void bemit_cmovcc(const ir_node *node)
@@ -2945,11 +2680,11 @@ static void bemit_cmovcc(const ir_node *node)
        const ia32_attr_t     *attr         = get_ia32_attr_const(node);
        int                    ins_permuted = attr->data.ins_permuted;
        const arch_register_t *out          = arch_irn_get_register(node, pn_ia32_res);
-       int                    pnc          = get_ia32_condcode(node);
+       ia32_condition_code_t  cc           = get_ia32_condcode(node);
        const arch_register_t *in_true;
        const arch_register_t *in_false;
 
-       pnc = determine_final_pnc(node, n_ia32_CMovcc_eflags, pnc);
+       cc = determine_final_cc(node, n_ia32_CMovcc_eflags, cc);
 
        in_true  = arch_get_irn_register(get_irn_n(node, n_ia32_CMovcc_val_true));
        in_false = arch_get_irn_register(get_irn_n(node, n_ia32_CMovcc_val_false));
@@ -2968,12 +2703,13 @@ static void bemit_cmovcc(const ir_node *node)
        }
 
        if (ins_permuted)
-               pnc = ia32_get_negated_pnc(pnc);
+               cc = ia32_negate_condition_code(cc);
 
-       /* TODO: handling of Nans isn't correct yet */
+       if (cc & ia32_cc_float_parity_cases)
+               panic("cmov can't handle parity float cases");
 
        bemit8(0x0F);
-       bemit8(0x40 | pnc2cc(pnc));
+       bemit8(0x40 | pnc2cc(cc));
        if (get_ia32_op_type(node) == ia32_Normal) {
                bemit_modrr(in_true, out);
        } else {
@@ -3319,8 +3055,8 @@ static void bemit_load(const ir_node *node)
        if (out->index == REG_GP_EAX) {
                ir_node   *base      = get_irn_n(node, n_ia32_base);
                int        has_base  = !is_ia32_NoReg_GP(base);
-               ir_node   *index     = get_irn_n(node, n_ia32_index);
-               int        has_index = !is_ia32_NoReg_GP(index);
+               ir_node   *idx       = get_irn_n(node, n_ia32_index);
+               int        has_index = !is_ia32_NoReg_GP(idx);
                if (!has_base && !has_index) {
                        ir_entity *ent  = get_ia32_am_sc(node);
                        int        offs = get_ia32_am_offs_int(node);
@@ -3364,8 +3100,8 @@ static void bemit_store(const ir_node *node)
                if (in->index == REG_GP_EAX) {
                        ir_node   *base      = get_irn_n(node, n_ia32_base);
                        int        has_base  = !is_ia32_NoReg_GP(base);
-                       ir_node   *index     = get_irn_n(node, n_ia32_index);
-                       int        has_index = !is_ia32_NoReg_GP(index);
+                       ir_node   *idx       = get_irn_n(node, n_ia32_index);
+                       int        has_index = !is_ia32_NoReg_GP(idx);
                        if (!has_base && !has_index) {
                                ir_entity *ent  = get_ia32_am_sc(node);
                                int        offs = get_ia32_am_offs_int(node);
@@ -3501,14 +3237,13 @@ static void bemit_jp(bool odd, const ir_node *dest_block)
 
 static void bemit_ia32_jcc(const ir_node *node)
 {
-       int            pnc = get_ia32_condcode(node);
-       const ir_node *proj_true;
-       const ir_node *proj_false;
-       const ir_node *dest_true;
-       const ir_node *dest_false;
-       const ir_node *block;
+       ia32_condition_code_t cc = get_ia32_condcode(node);
+       const ir_node        *proj_true;
+       const ir_node        *proj_false;
+       const ir_node        *dest_true;
+       const ir_node        *dest_false;
 
-       pnc = determine_final_pnc(node, 0, pnc);
+       cc = determine_final_cc(node, 0, cc);
 
        /* get both Projs */
        proj_true = get_proj(node, pn_ia32_Jcc_true);
@@ -3517,59 +3252,35 @@ static void bemit_ia32_jcc(const ir_node *node)
        proj_false = get_proj(node, pn_ia32_Jcc_false);
        assert(proj_false && "Jcc without false Proj");
 
-       block = get_nodes_block(node);
-
        if (can_be_fallthrough(proj_true)) {
                /* exchange both proj's so the second one can be omitted */
                const ir_node *t = proj_true;
 
                proj_true  = proj_false;
                proj_false = t;
-               pnc        = ia32_get_negated_pnc(pnc);
+               cc         = ia32_negate_condition_code(cc);
        }
 
        dest_true  = get_cfop_target_block(proj_true);
        dest_false = get_cfop_target_block(proj_false);
 
-       if (pnc & ia32_pn_Cmp_float) {
+       if (cc & ia32_cc_float_parity_cases) {
                /* Some floating point comparisons require a test of the parity flag,
                 * which indicates that the result is unordered */
-               switch (pnc & 15) {
-                       case pn_Cmp_Uo: {
-                               bemit_jp(false, dest_true);
-                               break;
+               if (cc & ia32_cc_negated) {
+                       bemit_jp(false, dest_true);
+               } else {
+                       /* we need a local label if the false proj is a fallthrough
+                        * as the falseblock might have no label emitted then */
+                       if (can_be_fallthrough(proj_false)) {
+                               bemit8(0x7A);
+                               bemit8(0x06);  // jp + 6
+                       } else {
+                               bemit_jp(false, dest_false);
                        }
-
-                       case pn_Cmp_Leg:
-                               bemit_jp(true, dest_true);
-                               break;
-
-                       case pn_Cmp_Eq:
-                       case pn_Cmp_Lt:
-                       case pn_Cmp_Le:
-                               /* we need a local label if the false proj is a fallthrough
-                                * as the falseblock might have no label emitted then */
-                               if (can_be_fallthrough(proj_false)) {
-                                       bemit8(0x7A);
-                                       bemit8(0x06);  // jp + 6
-                               } else {
-                                       bemit_jp(false, dest_false);
-                               }
-                               goto emit_jcc;
-
-                       case pn_Cmp_Ug:
-                       case pn_Cmp_Uge:
-                       case pn_Cmp_Ne:
-                               bemit_jp(false, dest_true);
-                               goto emit_jcc;
-
-                       default:
-                               goto emit_jcc;
                }
-       } else {
-emit_jcc:
-               bemit_jcc(pnc, dest_true);
        }
+       bemit_jcc(cc, dest_true);
 
        /* the second Proj might be a fallthrough */
        if (can_be_fallthrough(proj_false)) {
@@ -3581,63 +3292,13 @@ emit_jcc:
 
 static void bemit_switchjmp(const ir_node *node)
 {
-       unsigned long          interval;
-       int                    last_value;
-       int                    i;
-       jmp_tbl_t              tbl;
-       const arch_register_t *in;
+       ir_entity *jump_table = get_ia32_am_sc(node);
+       long       default_pn = get_ia32_default_pn(node);
 
-       /* fill the table structure */
-       generate_jump_table(&tbl, node);
-
-       /* two-complement's magic make this work without overflow */
-       interval = tbl.max_value - tbl.min_value;
-
-       in = get_in_reg(node, 0);
-       /* emit the table */
-       if (get_signed_imm_size(interval) == 1) {
-               bemit8(0x83); // cmpl $imm8, %in
-               bemit_modru(in, 7);
-               bemit8(interval);
-       } else {
-               bemit8(0x81); // cmpl $imm32, %in
-               bemit_modru(in, 7);
-               bemit32(interval);
-       }
-       bemit8(0x0F); // ja tbl.defProj
-       bemit8(0x87);
-       ia32_emitf(tbl.defProj, ".long %L - . - 4\n");
-
-       if (tbl.num_branches > 1) {
-               /* create table */
-               bemit8(0xFF); // jmp *tbl.label(,%in,4)
-               bemit8(MOD_IND | ENC_REG(4) | ENC_RM(0x04));
-               bemit8(ENC_SIB(2, reg_gp_map[in->index], 0x05));
-               be_emit_irprintf("\t.long %s\n", tbl.label);
-
-               be_gas_emit_switch_section(GAS_SECTION_RODATA);
-               be_emit_cstring(".align 4\n");
-               be_emit_irprintf("%s:\n", tbl.label);
-
-               last_value = tbl.branches[0].value;
-               for (i = 0; i != tbl.num_branches; ++i) {
-                       while (last_value != tbl.branches[i].value) {
-                               ia32_emitf(tbl.defProj, ".long %L\n");
-                               ++last_value;
-                       }
-                       ia32_emitf(tbl.branches[i].target, ".long %L\n");
-                       ++last_value;
-               }
-               be_gas_emit_switch_section(GAS_SECTION_TEXT);
-       } else {
-               /* one jump is enough */
-               panic("switch only has one case");
-               //ia32_emitf(tbl.branches[0].target, "\tjmp %L\n");
-       }
-
-       be_emit_write_line();
+       bemit8(0xFF); // jmp *tbl.label(,%in,4)
+       bemit_mod_am(0x05, node);
 
-       free(tbl.branches);
+       emit_jump_table(node, default_pn, jump_table, get_cfop_target_block);
 }
 
 /**
@@ -4201,7 +3862,6 @@ static void ia32_register_binary_emitters(void)
 
        /* ignore the following nodes */
        register_emitter(op_ia32_ProduceVal,   emit_Nothing);
-       register_emitter(op_be_Barrier,        emit_Nothing);
        register_emitter(op_be_Keep,           emit_Nothing);
        register_emitter(op_be_Start,          emit_Nothing);
        register_emitter(op_Phi,               emit_Nothing);