fix sse/x87 fixup code added at wrong places
[libfirm] / ir / be / ia32 / ia32_emitter.c
index bb1648c..1ccd8a1 100644 (file)
@@ -86,8 +86,11 @@ const arch_register_t *get_in_reg(ia32_emit_env_t *env, const ir_node *irn,
 
        assert(reg && "no in register found");
 
-       /* in case of a joker register: just return a valid register */
-       if (arch_register_type_is(reg, joker)) {
+       if(reg == &ia32_gp_regs[REG_GP_NOREG])
+               panic("trying to emit noreg");
+
+       /* in case of unknown register: just return a valid register */
+       if (reg == &ia32_gp_regs[REG_GP_UKNWN]) {
                const arch_register_req_t *req;
 
                /* ask for the requirements */
@@ -155,10 +158,11 @@ char get_mode_suffix(const ir_mode *mode) {
                case 64:
                        return 'l';
                case 80:
+               case 96:
                        return 't';
                }
        } else {
-               assert(mode_is_int(mode) || mode_is_reference(mode) || mode_is_character(mode));
+               assert(mode_is_int(mode) || mode_is_reference(mode));
                switch(get_mode_size_bits(mode)) {
                case 64:
                        return 'q';
@@ -175,12 +179,18 @@ char get_mode_suffix(const ir_mode *mode) {
 
 static
 int produces_result(const ir_node *node) {
-       return !(is_ia32_St(node) ||
-               is_ia32_CondJmp(node) ||
-               is_ia32_xCondJmp(node) ||
-               is_ia32_CmpSet(node) ||
-               is_ia32_xCmpSet(node) ||
-               is_ia32_SwitchJmp(node));
+       return
+               !is_ia32_CmpSet(node)    &&
+               !is_ia32_CondJmp(node)   &&
+               !is_ia32_St(node)        &&
+               !is_ia32_SwitchJmp(node) &&
+               !is_ia32_TestJmp(node)   &&
+               !is_ia32_xCmpSet(node)   &&
+               !is_ia32_xCondJmp(node)  &&
+               !is_ia32_CmpCMov(node)   &&
+               !is_ia32_TestCMov(node)  &&
+               !is_ia32_CmpSet(node)    && /* this is correct, the Cmp has no result */
+               !is_ia32_TestSet(node);
 }
 
 static
@@ -240,6 +250,29 @@ void ia32_emit_source_register(ia32_emit_env_t *env, const ir_node *node, int po
        be_emit_string(env, reg_name);
 }
 
+void ia32_emit_8bit_source_register(ia32_emit_env_t *env, const ir_node *node, int pos)
+{
+       const arch_register_t *reg = get_in_reg(env, node, pos);
+       const char *reg_name = arch_register_get_name(reg);
+
+       assert(pos < get_irn_arity(node));
+
+       be_emit_char(env, '%');
+       be_emit_char(env, reg_name[1]);
+       be_emit_char(env, 'l');
+}
+
+void ia32_emit_16bit_source_register(ia32_emit_env_t *env, const ir_node *node, int pos)
+{
+       const arch_register_t *reg = get_in_reg(env, node, pos);
+       const char *reg_name = arch_register_get_name(reg);
+
+       assert(pos < get_irn_arity(node));
+
+       be_emit_char(env, '%');
+       be_emit_string(env, &reg_name[1]);
+}
+
 void ia32_emit_dest_register(ia32_emit_env_t *env, const ir_node *node, int pos) {
        const arch_register_t *reg = get_out_reg(env, node, pos);
        const char *reg_name = arch_register_get_name(reg);
@@ -248,6 +281,14 @@ void ia32_emit_dest_register(ia32_emit_env_t *env, const ir_node *node, int pos)
        be_emit_string(env, reg_name);
 }
 
+static void ia32_emit_register(ia32_emit_env_t *env, const arch_register_t *reg)
+{
+       const char *reg_name = arch_register_get_name(reg);
+
+       be_emit_char(env, '%');
+       be_emit_string(env, reg_name);
+}
+
 void ia32_emit_x87_name(ia32_emit_env_t *env, const ir_node *node, int pos)
 {
        const ia32_x87_attr_t *attr = get_ia32_x87_attr_const(node);
@@ -272,11 +313,10 @@ void ia32_emit_immediate(ia32_emit_env_t *env, const ir_node *node)
                return;
        case ia32_ImmSymConst:
                ent = get_ia32_Immop_symconst(node);
-               mark_entity_visited(ent);
+               set_entity_backend_marked(ent, 1);
                id = get_entity_ld_ident(ent);
                be_emit_ident(env, id);
                return;
-       case ia32_ImmAsm:
        case ia32_ImmNone:
                break;
        }
@@ -388,77 +428,105 @@ void ia32_emit_function_size(ia32_emit_env_t *env, const char *name)
 }
 
 
+static
+void emit_ia32_Immediate(ia32_emit_env_t *env, const ir_node *node);
 
 /**
  * Emits registers and/or address mode of a binary operation.
  */
 void ia32_emit_binop(ia32_emit_env_t *env, const ir_node *node) {
-       switch(get_ia32_op_type(node)) {
-               case ia32_Normal:
-                       if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
-                               ia32_emit_immediate(env, node);
-                               be_emit_cstring(env, ", ");
-                               ia32_emit_source_register(env, node, 2);
-                       } else {
-                               const arch_register_t *in1 = get_in_reg(env, node, 2);
-                               const arch_register_t *in2 = get_in_reg(env, node, 3);
-                               const arch_register_t *out = produces_result(node) ? get_out_reg(env, node, 0) : NULL;
-                               const arch_register_t *in;
-                               const char            *in_name;
+       int            right_pos;
+       const ir_node *right_op = get_irn_n(node, 3);
 
-                               in      = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
-                               out     = out ? out : in1;
-                               in_name = arch_register_get_name(in);
+       switch(get_ia32_op_type(node)) {
+       case ia32_Normal:
+               if(is_ia32_Immediate(right_op)) {
+                       emit_ia32_Immediate(env, right_op);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_source_register(env, node, 2);
+                       break;
+               } else if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
+                       ia32_emit_immediate(env, node);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_source_register(env, node, 2);
+               } else {
+                       const arch_register_t *in1 = get_in_reg(env, node, 2);
+                       const arch_register_t *in2 = get_in_reg(env, node, 3);
+                       const arch_register_t *out = produces_result(node) ? get_out_reg(env, node, 0) : NULL;
+                       const arch_register_t *in;
+                       const char            *in_name;
+
+                       in      = out ? ((out == in2) ? in1 : in2) : in2;
+                       out     = out ? out : in1;
+                       in_name = arch_register_get_name(in);
+
+                       if (is_ia32_emit_cl(node)) {
+                               assert(in == &ia32_gp_regs[REG_ECX]);
+                               in_name = "cl";
+                       }
 
-                               if (is_ia32_emit_cl(node)) {
-                                       assert(REGS_ARE_EQUAL(&ia32_gp_regs[REG_ECX], in) && "shift operation needs ecx");
-                                       in_name = "cl";
-                               }
+                       be_emit_char(env, '%');
+                       be_emit_string(env, in_name);
+                       be_emit_cstring(env, ", %");
+                       be_emit_string(env, arch_register_get_name(out));
+               }
+               break;
+       case ia32_AddrModeS:
+               if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
+                       assert(!produces_result(node) &&
+                                       "Source AM with Const must not produce result");
+                       ia32_emit_immediate(env, node);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_am(env, node);
+               } else if(is_ia32_Immediate(right_op)) {
+                       assert(!produces_result(node) &&
+                                       "Source AM with Const must not produce result");
 
-                               be_emit_char(env, '%');
-                               be_emit_string(env, in_name);
-                               be_emit_cstring(env, ", %");
-                               be_emit_string(env, arch_register_get_name(out));
-                       }
-                       break;
-               case ia32_AddrModeS:
+                       emit_ia32_Immediate(env, right_op);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_am(env, node);
+               } else if (produces_result(node)) {
                        ia32_emit_am(env, node);
                        be_emit_cstring(env, ", ");
-                       if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
-                               assert(!produces_result(node) && "Source AM with Const must not produce result");
-                               ia32_emit_immediate(env, node);
-                       } else if (produces_result(node)) {
-                               ia32_emit_dest_register(env, node, 0);
-                       } else {
-                               ia32_emit_source_register(env, node, 2);
-                       }
+                       ia32_emit_dest_register(env, node, 0);
+               } else {
+                       ia32_emit_am(env, node);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_source_register(env, node, 2);
+               }
+               break;
+       case ia32_AddrModeD:
+               right_pos = get_irn_arity(node) >= 5 ? 3 : 2;
+               right_op  = get_irn_n(node, right_pos);
+               if(is_ia32_Immediate(right_op)) {
+                       emit_ia32_Immediate(env, right_op);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_am(env, node);
                        break;
-               case ia32_AddrModeD:
-                       if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
-                               ia32_emit_immediate(env, node);
-                               be_emit_cstring(env, ", ");
-                               ia32_emit_am(env, node);
-                       } else {
-                               const arch_register_t *in1 = get_in_reg(env, node,
-                                                                       get_irn_arity(node) == 5 ? 3 : 2);
-                               ir_mode               *mode = get_ia32_ls_mode(node);
-                               const char            *in_name;
-
-                               in_name = ia32_get_reg_name_for_mode(env, mode, in1);
+               } else if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
+                       ia32_emit_immediate(env, node);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_am(env, node);
+               } else {
+                       const arch_register_t *in1 = get_in_reg(env, node, right_pos);
+                       ir_mode               *mode = get_ia32_ls_mode(node);
+                       const char            *in_name;
 
-                               if (is_ia32_emit_cl(node)) {
-                                       assert(REGS_ARE_EQUAL(&ia32_gp_regs[REG_ECX], in1) && "shift operation needs ecx");
-                                       in_name = "cl";
-                               }
+                       in_name = ia32_get_reg_name_for_mode(env, mode, in1);
 
-                               be_emit_char(env, '%');
-                               be_emit_string(env, in_name);
-                               be_emit_cstring(env, ", ");
-                               ia32_emit_am(env, node);
+                       if (is_ia32_emit_cl(node)) {
+                               assert(in1 == &ia32_gp_regs[REG_ECX]);
+                               in_name = "cl";
                        }
-                       break;
-               default:
-                       assert(0 && "unsupported op type");
+
+                       be_emit_char(env, '%');
+                       be_emit_string(env, in_name);
+                       be_emit_cstring(env, ", ");
+                       ia32_emit_am(env, node);
+               }
+               break;
+       default:
+               assert(0 && "unsupported op type");
        }
 }
 
@@ -478,7 +546,7 @@ void ia32_emit_x87_binop(ia32_emit_env_t *env, const ir_node *node) {
                                const arch_register_t *out      = x87_attr->x87[2];
                                const arch_register_t *in;
 
-                               in  = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
+                               in  = out ? ((out == in2) ? in1 : in2) : in2;
                                out = out ? out : in1;
 
                                be_emit_char(env, '%');
@@ -496,34 +564,39 @@ void ia32_emit_x87_binop(ia32_emit_env_t *env, const ir_node *node) {
        }
 }
 
+void ia32_emit_am_or_dest_register(ia32_emit_env_t *env, const ir_node *node,
+                                   int pos) {
+       if(get_ia32_op_type(node) == ia32_Normal) {
+               ia32_emit_dest_register(env, node, pos);
+       } else {
+               assert(get_ia32_op_type(node) == ia32_AddrModeD);
+               ia32_emit_am(env, node);
+       }
+}
+
 /**
  * Emits registers and/or address mode of a unary operation.
  */
-void ia32_emit_unop(ia32_emit_env_t *env, const ir_node *node) {
+void ia32_emit_unop(ia32_emit_env_t *env, const ir_node *node, int pos) {
+       const ir_node *op;
+
        switch(get_ia32_op_type(node)) {
-               case ia32_Normal:
-                       if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
-                               ia32_emit_immediate(env, node);
-                       } else {
-                               if (is_ia32_Mul(node) || is_ia32_IMul1OP(node)) {
-                                       ia32_emit_source_register(env, node, 3);
-                               } else if(is_ia32_IDiv(node) || is_ia32_Div(node)) {
-                                       ia32_emit_source_register(env, node, 4);
-                               } else if(is_ia32_Push(node)) {
-                                       ia32_emit_source_register(env, node, 2);
-                               } else if(is_ia32_Pop(node)) {
-                                       ia32_emit_dest_register(env, node, 1);
-                               } else {
-                                       ia32_emit_dest_register(env, node, 0);
-                               }
-                       }
-                       break;
-               case ia32_AddrModeS:
-               case ia32_AddrModeD:
-                       ia32_emit_am(env, node);
-                       break;
-               default:
-                       assert(0 && "unsupported op type");
+       case ia32_Normal:
+               op = get_irn_n(node, pos);
+               if (is_ia32_Immediate(op)) {
+                       emit_ia32_Immediate(env, op);
+               } else if (is_ia32_ImmConst(node) || is_ia32_ImmSymConst(node)) {
+                       ia32_emit_immediate(env, node);
+               } else {
+                       ia32_emit_source_register(env, node, pos);
+               }
+               break;
+       case ia32_AddrModeS:
+       case ia32_AddrModeD:
+               ia32_emit_am(env, node);
+               break;
+       default:
+               assert(0 && "unsupported op type");
        }
 }
 
@@ -545,7 +618,7 @@ void ia32_emit_am(ia32_emit_env_t *env, const ir_node *node) {
        if (ent != NULL) {
                ident *id;
 
-               mark_entity_visited(ent);
+               set_entity_backend_marked(ent, 1);
                id = get_entity_ld_ident(ent);
                if (is_ia32_am_sc_sign(node))
                        be_emit_char(env, '-');
@@ -589,6 +662,11 @@ void ia32_emit_am(ia32_emit_env_t *env, const ir_node *node) {
                }
                be_emit_char(env, ')');
        }
+
+       /* special case if nothing is set */
+       if(ent == NULL && offs == 0 && !has_base && !has_index) {
+               be_emit_char(env, '0');
+       }
 }
 
 /*************************************************
@@ -625,14 +703,6 @@ const struct cmp2conditon_t cmp2condition_s[] = {
        { "ge",              pn_Cmp_Ge },     /* >= */
        { "ne",              pn_Cmp_Lg },     /* != */
        { NULL,              pn_Cmp_Leg},     /* Floating point: ordered */
-       { NULL,              pn_Cmp_Uo },     /* Floating point: unordered */
-       { "e",               pn_Cmp_Ue },     /* Floating point: unordered or == */
-       { "b",               pn_Cmp_Ul },     /* Floating point: unordered or < */
-       { "be",              pn_Cmp_Ule },    /* Floating point: unordered or <= */
-       { "a",               pn_Cmp_Ug },     /* Floating point: unordered or > */
-       { "ae",              pn_Cmp_Uge },    /* Floating point: unordered or >= */
-       { "ne",              pn_Cmp_Ne },     /* Floating point: unordered or != */
-       { NULL,              pn_Cmp_True },   /* always true */
 };
 
 /*
@@ -654,15 +724,15 @@ const struct cmp2conditon_t cmp2condition_u[] = {
  * returns the condition code
  */
 static
-const char *get_cmp_suffix(int cmp_code)
+const char *get_cmp_suffix(pn_Cmp cmp_code)
 {
-       assert( (cmp2condition_s[cmp_code & 15].num) == (cmp_code & 15));
+       assert( (cmp2condition_s[cmp_code & 7].num) == (cmp_code & 7));
        assert( (cmp2condition_u[cmp_code & 7].num) == (cmp_code & 7));
 
        if((cmp_code & ia32_pn_Cmp_Unsigned)) {
                return cmp2condition_u[cmp_code & 7].name;
        } else {
-               return cmp2condition_s[cmp_code & 15].name;
+               return cmp2condition_s[cmp_code & 7].name;
        }
 }
 
@@ -680,15 +750,23 @@ ir_node *get_cfop_target_block(const ir_node *irn) {
        return get_irn_link(irn);
 }
 
+/**
+ * Emits a block label for the given block.
+ */
 static
 void ia32_emit_block_name(ia32_emit_env_t *env, const ir_node *block)
 {
-       be_emit_cstring(env, BLOCK_PREFIX);
-       be_emit_irprintf(env->emit, "%d", get_irn_node_nr(block));
+       if (has_Block_label(block)) {
+               be_emit_string(env, be_gas_label_prefix());
+               be_emit_irprintf(env->emit, "%u", (unsigned)get_Block_label(block));
+       } else {
+               be_emit_cstring(env, BLOCK_PREFIX);
+               be_emit_irprintf(env->emit, "%d", get_irn_node_nr(block));
+       }
 }
 
 /**
- * Returns the target label for a control flow node.
+ * Emits the target label for a control flow node.
  */
 static
 void ia32_emit_cfop_target(ia32_emit_env_t * env, const ir_node *node) {
@@ -760,19 +838,54 @@ void finish_CondJmp(ia32_emit_env_t *env, const ir_node *node, ir_mode *mode,
                pnc        = get_negated_pnc(pnc, mode);
        }
 
-       /* in case of unordered compare, check for parity */
-       if (pnc & pn_Cmp_Uo) {
-               be_emit_cstring(env, "\tjp ");
+       if (mode_is_float(mode)) {
+               /* Some floating point comparisons require a test of the parity flag, which
+                * indicates that the result is unordered */
+               switch (pnc) {
+                       case pn_Cmp_Uo:
+                               be_emit_cstring(env, "\tjp ");
+                               ia32_emit_cfop_target(env, proj_true);
+                               be_emit_finish_line_gas(env, proj_true);
+                               break;
+
+                       case pn_Cmp_Leg:
+                               be_emit_cstring(env, "\tjnp ");
+                               ia32_emit_cfop_target(env, proj_true);
+                               be_emit_finish_line_gas(env, proj_true);
+                               break;
+
+                       case pn_Cmp_Eq:
+                       case pn_Cmp_Lt:
+                       case pn_Cmp_Le:
+                               be_emit_cstring(env, "\tjp ");
+                               ia32_emit_cfop_target(env, proj_false);
+                               be_emit_finish_line_gas(env, proj_false);
+                               goto float_jcc;
+
+                       case pn_Cmp_Ug:
+                       case pn_Cmp_Uge:
+                       case pn_Cmp_Ne:
+                               be_emit_cstring(env, "\tjp ");
+                               ia32_emit_cfop_target(env, proj_true);
+                               be_emit_finish_line_gas(env, proj_true);
+                               goto float_jcc;
+
+                       default:
+                       float_jcc:
+                               /* The bits set by floating point compares correspond to unsigned
+                                * comparisons */
+                               pnc |= ia32_pn_Cmp_Unsigned;
+                               goto emit_jcc;
+               }
+       } else {
+emit_jcc:
+               be_emit_cstring(env, "\tj");
+               ia32_emit_cmp_suffix(env, pnc);
+               be_emit_char(env, ' ');
                ia32_emit_cfop_target(env, proj_true);
                be_emit_finish_line_gas(env, proj_true);
        }
 
-       be_emit_cstring(env, "\tj");
-       ia32_emit_cmp_suffix(env, pnc);
-       be_emit_char(env, ' ');
-       ia32_emit_cfop_target(env, proj_true);
-       be_emit_finish_line_gas(env, proj_true);
-
        /* the second Proj might be a fallthrough */
        if (get_cfop_target_block(proj_false) != next_block) {
                be_emit_cstring(env, "\tjmp ");
@@ -791,7 +904,9 @@ void finish_CondJmp(ia32_emit_env_t *env, const ir_node *node, ir_mode *mode,
  */
 static
 void CondJmp_emitter(ia32_emit_env_t *env, const ir_node *node) {
-       be_emit_cstring(env, "\tcmp ");
+       be_emit_cstring(env, "\tcmp");
+       ia32_emit_mode_suffix(env, node);
+       be_emit_char(env, ' ');
        ia32_emit_binop(env, node);
        be_emit_finish_line_gas(env, node);
 
@@ -811,19 +926,13 @@ void emit_ia32_CondJmp(ia32_emit_env_t *env, const ir_node *node) {
  */
 static
 void TestJmp_emitter(ia32_emit_env_t *env, const ir_node *node) {
-       if(is_ia32_ImmSymConst(node) || is_ia32_ImmConst(node)) {
-               be_emit_cstring(env, "\ttest ");
-               ia32_emit_immediate(env, node);
-               be_emit_cstring(env, ", ");
-               ia32_emit_source_register(env, node, 0);
-               be_emit_finish_line_gas(env, node);
-       } else {
-               be_emit_cstring(env, "\ttest ");
-               ia32_emit_source_register(env, node, 1);
-               be_emit_cstring(env, ", ");
-               ia32_emit_source_register(env, node, 0);
-               be_emit_finish_line_gas(env, node);
-       }
+       be_emit_cstring(env, "\ttest");
+       ia32_emit_mode_suffix(env, node);
+       be_emit_char(env, ' ');
+
+       ia32_emit_binop(env, node);
+       be_emit_finish_line_gas(env, node);
+
        finish_CondJmp(env, node, mode_Iu, get_ia32_pncode(node));
 }
 
@@ -835,22 +944,6 @@ void emit_ia32_TestJmp(ia32_emit_env_t *env, const ir_node *node) {
        TestJmp_emitter(env, node);
 }
 
-static
-void emit_ia32_CJmp(ia32_emit_env_t *env, const ir_node *node) {
-       be_emit_cstring(env, "/* omitted redundant test */");
-       be_emit_finish_line_gas(env, node);
-
-       finish_CondJmp(env, node, mode_Is, get_ia32_pncode(node));
-}
-
-static
-void emit_ia32_CJmpAM(ia32_emit_env_t *env, const ir_node *node) {
-       be_emit_cstring(env, "/* omitted redundant test/cmp */");
-       be_emit_finish_line_gas(env, node);
-
-       finish_CondJmp(env, node, mode_Is, get_ia32_pncode(node));
-}
-
 /**
  * Emits code for conditional SSE floating point jump with two variables.
  */
@@ -911,123 +1004,111 @@ void emit_ia32_x87CondJmp(ia32_emit_env_t *env, const ir_node *node) {
 }
 
 static
-void CMov_emitter(ia32_emit_env_t *env, const ir_node *node) {
-       long pnc = get_ia32_pncode(node);
-       int is_PsiCondCMov = is_ia32_PsiCondCMov(node);
-       int idx_left  = 2 - is_PsiCondCMov;
-       int idx_right = 3 - is_PsiCondCMov;
+void CMov_emitter(ia32_emit_env_t *env, const ir_node *node)
+{
        const arch_register_t *in1, *in2, *out;
+       long  pnc = get_ia32_pncode(node);
 
        out = arch_get_irn_register(env->arch_env, node);
-       in1 = arch_get_irn_register(env->arch_env, get_irn_n(node, idx_left));
-       in2 = arch_get_irn_register(env->arch_env, get_irn_n(node, idx_right));
 
        /* we have to emit the cmp first, because the destination register */
        /* could be one of the compare registers                           */
-       if (is_ia32_CmpCMov(node)) {
-               be_emit_cstring(env, "\tcmp ");
-               ia32_emit_source_register(env, node, 1);
-               be_emit_cstring(env, ", ");
-               ia32_emit_source_register(env, node, 0);
-       } else if (is_ia32_xCmpCMov(node)) {
+       if (is_ia32_xCmpCMov(node)) {
                be_emit_cstring(env, "\tucomis");
                ia32_emit_mode_suffix_mode(env, get_irn_mode(node));
                be_emit_char(env, ' ');
                ia32_emit_source_register(env, node, 1);
                be_emit_cstring(env, ", ");
                ia32_emit_source_register(env, node, 0);
-       } else if (is_PsiCondCMov) {
-               /* omit compare because flags are already set by And/Or */
-               be_emit_cstring(env, "\ttest ");
-               ia32_emit_source_register(env, node, 0);
-               be_emit_cstring(env, ", ");
-               ia32_emit_source_register(env, node, 0);
+
+               in1 = arch_get_irn_register(env->arch_env, get_irn_n(node, 2));
+               in2 = arch_get_irn_register(env->arch_env, get_irn_n(node, 3));
        } else {
-               assert(0 && "unsupported CMov");
+               if (is_ia32_CmpCMov(node)) {
+                       be_emit_cstring(env, "\tcmp ");
+               } else {
+                       assert(is_ia32_TestCMov(node));
+                       be_emit_cstring(env, "\ttest ");
+               }
+               ia32_emit_binop(env, node);
+
+               in1 = arch_get_irn_register(env->arch_env, get_irn_n(node, 5));
+               in2 = arch_get_irn_register(env->arch_env, get_irn_n(node, 6));
        }
        be_emit_finish_line_gas(env, node);
 
-       if (REGS_ARE_EQUAL(out, in2)) {
+       if (out == in2) {
                /* best case: default in == out -> do nothing */
-       } else if (REGS_ARE_EQUAL(out, in1)) {
-               ir_node *n = (ir_node*) node;
-               /* true in == out -> need complement compare and exchange true and default in */
-               ir_node *t = get_irn_n(n, idx_left);
-               set_irn_n(n, idx_left, get_irn_n(n, idx_right));
-               set_irn_n(n, idx_right, t);
-
+       } else if(in2 == &ia32_gp_regs[REG_GP_UKNWN]) {
+               /* also nothign to do for unknown regs */
+       } else if (out == in1) {
+               const arch_register_t *t;
+               /* true in == out -> need complement compare and exchange true and
+                * default in */
+               t   = in1;
+               in1 = in2;
+               in2 = t;
                pnc = get_negated_pnc(pnc, get_irn_mode(node));
        } else {
-               /* out is different from in: need copy default -> out */
-               if (is_PsiCondCMov) {
-                       be_emit_cstring(env, "\tmovl ");
-                       ia32_emit_dest_register(env, node, 2);
-                       be_emit_cstring(env, ", ");
-                       ia32_emit_dest_register(env, node, 0);
-               } else {
-                       be_emit_cstring(env, "\tmovl ");
-                       ia32_emit_source_register(env, node, 3);
-                       be_emit_cstring(env, ", ");
-                       ia32_emit_dest_register(env, node, 0);
-               }
+               /* out is different from both ins: need copy default -> out */
+               be_emit_cstring(env, "\tmovl ");
+               ia32_emit_register(env, in2);
+               be_emit_cstring(env, ", ");
+               ia32_emit_register(env, out);
                be_emit_finish_line_gas(env, node);
        }
 
-       if (is_PsiCondCMov) {
-               be_emit_cstring(env, "\tcmov");
-               ia32_emit_cmp_suffix(env, pnc);
-               be_emit_cstring(env, "l ");
-               ia32_emit_source_register(env, node, 1);
-               be_emit_cstring(env, ", ");
-               ia32_emit_dest_register(env, node, 0);
-       } else {
-               be_emit_cstring(env, "\tcmov");
-               ia32_emit_cmp_suffix(env, pnc);
-               be_emit_cstring(env, "l ");
-               ia32_emit_source_register(env, node, 2);
-               be_emit_cstring(env, ", ");
-               ia32_emit_dest_register(env, node, 0);
-       }
+       be_emit_cstring(env, "\tcmov");
+       ia32_emit_cmp_suffix(env, pnc );
+       be_emit_cstring(env, "l ");
+       ia32_emit_register(env, in1);
+       be_emit_cstring(env, ", ");
+       ia32_emit_register(env, out);
+
        be_emit_finish_line_gas(env, node);
 }
 
 static
-void emit_ia32_CmpCMov(ia32_emit_env_t *env, const ir_node *node) {
+void emit_ia32_CmpCMov(ia32_emit_env_t *env, const ir_node *node)
+{
        CMov_emitter(env, node);
 }
 
 static
-void emit_ia32_PsiCondCMov(ia32_emit_env_t *env, const ir_node *node) {
+void emit_ia32_TestCMov(ia32_emit_env_t *env, const ir_node *node)
+{
        CMov_emitter(env, node);
 }
 
 static
-void emit_ia32_xCmpCMov(ia32_emit_env_t *env, const ir_node *node) {
+void emit_ia32_xCmpCMov(ia32_emit_env_t *env, const ir_node *node)
+{
        CMov_emitter(env, node);
 }
 
 static
-void Set_emitter(ia32_emit_env_t *env, const ir_node *node, ir_mode *mode) {
-       int pnc = get_ia32_pncode(node);
+void Set_emitter(ia32_emit_env_t *env, const ir_node *node)
+{
+       long pnc = get_ia32_pncode(node);
        const char *reg8bit;
        const arch_register_t *out;
 
        out     = arch_get_irn_register(env->arch_env, node);
        reg8bit = ia32_get_mapped_reg_name(env->isa->regs_8bit, out);
 
-       if (is_ia32_CmpSet(node)) {
-               be_emit_cstring(env, "\tcmp ");
-               ia32_emit_binop(env, node);
-       } else if (is_ia32_xCmpSet(node)) {
+       if(is_ia32_xCmpSet(node)) {
                be_emit_cstring(env, "\tucomis");
                ia32_emit_mode_suffix_mode(env, get_irn_mode(get_irn_n(node, 2)));
                be_emit_char(env, ' ');
                ia32_emit_binop(env, node);
-       } else if (is_ia32_PsiCondSet(node)) {
-               be_emit_cstring(env, "\tcmp $0, ");
-               ia32_emit_source_register(env, node, 0);
        } else {
-               assert(0 && "unsupported Set");
+               if (is_ia32_CmpSet(node)) {
+                       be_emit_cstring(env, "\tcmp ");
+               } else {
+                       assert(is_ia32_TestSet(node));
+                       be_emit_cstring(env, "\ttest ");
+               }
+               ia32_emit_binop(env, node);
        }
        be_emit_finish_line_gas(env, node);
 
@@ -1045,17 +1126,17 @@ void Set_emitter(ia32_emit_env_t *env, const ir_node *node, ir_mode *mode) {
 
 static
 void emit_ia32_CmpSet(ia32_emit_env_t *env, const ir_node *node) {
-       Set_emitter(env, node, get_irn_mode(get_irn_n(node, 2)));
+       Set_emitter(env, node);
 }
 
 static
-void emit_ia32_PsiCondSet(ia32_emit_env_t *env, const ir_node *node) {
-       Set_emitter(env, node, get_irn_mode(get_irn_n(node, 0)));
+void emit_ia32_TestSet(ia32_emit_env_t *env, const ir_node *node) {
+       Set_emitter(env, node);
 }
 
 static
 void emit_ia32_xCmpSet(ia32_emit_env_t *env, const ir_node *node) {
-       Set_emitter(env, node, get_irn_mode(get_irn_n(node, 2)));
+       Set_emitter(env, node);
 }
 
 static
@@ -1160,9 +1241,9 @@ typedef struct _branch_t {
 /* jump table for switch generation */
 typedef struct _jmp_tbl_t {
        ir_node  *defProj;         /**< default target */
-       int       min_value;       /**< smallest switch case */
-       int       max_value;       /**< largest switch case */
-       int       num_branches;    /**< number of jumps */
+       long      min_value;       /**< smallest switch case */
+       long      max_value;       /**< largest switch case */
+       long      num_branches;    /**< number of jumps */
        char     *label;           /**< label of the jump table */
        branch_t *branches;        /**< jump array */
 } jmp_tbl_t;
@@ -1316,21 +1397,20 @@ void emit_Jmp(ia32_emit_env_t *env, const ir_node *node) {
 static
 void emit_ia32_Immediate(ia32_emit_env_t *env, const ir_node *node)
 {
-       const ia32_attr_t *attr = get_ia32_attr_const(node);
+       const ia32_immediate_attr_t *attr = get_ia32_immediate_attr_const(node);
 
-       if(attr->am_sc != NULL) {
-               ident *id = get_entity_ld_ident(attr->am_sc);
+       be_emit_char(env, '$');
+       if(attr->symconst != NULL) {
+               ident *id = get_entity_ld_ident(attr->symconst);
 
-               if(attr->data.am_sc_sign)
+               if(attr->attr.data.am_sc_sign)
                        be_emit_char(env, '-');
                be_emit_ident(env, id);
        }
-       if(attr->cnst_val.tv != NULL) {
-               if(attr->am_sc != NULL)
+       if(attr->symconst == NULL || attr->offset != 0) {
+               if(attr->symconst != NULL)
                        be_emit_char(env, '+');
-               else
-                       be_emit_char(env, '$');
-               be_emit_tarval(env, attr->cnst_val.tv);
+               be_emit_irprintf(env->emit, "0x%X", attr->offset);
        }
 }
 
@@ -1449,9 +1529,11 @@ const char* emit_asm_operand(ia32_emit_env_t *env, const ir_node *node,
 static
 void emit_ia32_Asm(ia32_emit_env_t *env, const ir_node *node)
 {
-       const ia32_attr_t *attr     = get_ia32_attr_const(node);
-       ident             *asm_text = attr->cnst_val.asm_text;
-       const char        *s        = get_id_str(asm_text);
+       const void            *gen_attr = get_irn_generic_attr_const(node);
+       const ia32_asm_attr_t *attr
+               = CONST_CAST_IA32_ATTR(ia32_asm_attr_t, gen_attr);
+       ident                 *asm_text = attr->asm_text;
+       const char            *s        = get_id_str(asm_text);
 
        be_emit_cstring(env, "# Begin ASM \t");
        be_emit_finish_line_gas(env, node);
@@ -1648,19 +1730,14 @@ void emit_ia32_Conv_I2I(ia32_emit_env_t *env, const ir_node *node) {
                        in_reg  = get_in_reg(env, node, 2);
                        out_reg = get_out_reg(env, node, 0);
 
-                       if (REGS_ARE_EQUAL(in_reg, &ia32_gp_regs[REG_EAX]) &&
-                               REGS_ARE_EQUAL(out_reg, in_reg)                &&
-                               signed_mode)
+                       if (in_reg  == &ia32_gp_regs[REG_EAX] &&
+                               out_reg == &ia32_gp_regs[REG_EAX] &&
+                               signed_mode &&
+                               smaller_bits == 16)
                        {
                                /* argument and result are both in EAX and */
-                               /* signedness is ok: -> use converts       */
-                               if (smaller_bits == 8) {
-                                       be_emit_cstring(env, "\tcbtw");
-                               } else if (smaller_bits == 16) {
-                                       be_emit_cstring(env, "\tcwtl");
-                               } else {
-                                       assert(0);
-                               }
+                               /* signedness is ok: -> use the smaller cwtl opcode */
+                               be_emit_cstring(env, "\tcwtl");
                        } else {
                                const char *sreg = ia32_get_reg_name_for_mode(env, smaller_mode, in_reg);
 
@@ -1677,7 +1754,7 @@ void emit_ia32_Conv_I2I(ia32_emit_env_t *env, const ir_node *node) {
                        be_emit_cstring(env, "\tmov");
                        be_emit_string(env, sign_suffix);
                        ia32_emit_mode_suffix_mode(env, smaller_mode);
-                       be_emit_cstring(env, "l %");
+                       be_emit_cstring(env, "l ");
                        ia32_emit_am(env, node);
                        be_emit_cstring(env, ", ");
                        ia32_emit_dest_register(env, node, 0);
@@ -1716,7 +1793,7 @@ void emit_be_Call(ia32_emit_env_t *env, const ir_node *node) {
 
        be_emit_cstring(env, "\tcall ");
        if (ent) {
-               mark_entity_visited(ent);
+               set_entity_backend_marked(ent, 1);
                be_emit_string(env, get_entity_ld_name(ent));
        } else {
                be_emit_char(env, '*');
@@ -1765,24 +1842,31 @@ void emit_be_SetSP(ia32_emit_env_t *env, const ir_node *node) {
 static
 void Copy_emitter(ia32_emit_env_t *env, const ir_node *node, const ir_node *op)
 {
-       const arch_env_t *aenv = env->arch_env;
+       const arch_env_t      *arch_env = env->arch_env;
+       const arch_register_t *in       = arch_get_irn_register(arch_env, op);
+       const arch_register_t *out      = arch_get_irn_register(arch_env, node);
        ir_mode *mode;
 
-       if (REGS_ARE_EQUAL(arch_get_irn_register(aenv, node), arch_get_irn_register(aenv, op)) ||
-               arch_register_type_is(arch_get_irn_register(aenv, op), virtual))
+       if(in == out) {
+               return;
+       }
+       if(is_unknown_reg(in))
+               return;
+       /* copies of vf nodes aren't real... */
+       if(arch_register_get_class(in) == &ia32_reg_classes[CLASS_ia32_vfp])
                return;
 
        mode = get_irn_mode(node);
        if (mode == mode_E) {
                be_emit_cstring(env, "\tmovsd ");
-               ia32_emit_source_register(env, node, 0);
+               ia32_emit_register(env, in);
                be_emit_cstring(env, ", ");
-               ia32_emit_dest_register(env, node, 0);
+               ia32_emit_register(env, out);
        } else {
                be_emit_cstring(env, "\tmovl ");
-               ia32_emit_source_register(env, node, 0);
+               ia32_emit_register(env, in);
                be_emit_cstring(env, ", ");
-               ia32_emit_dest_register(env, node, 0);
+               ia32_emit_register(env, out);
        }
        be_emit_finish_line_gas(env, node);
 }
@@ -1891,13 +1975,17 @@ void emit_ia32_LdTls(ia32_emit_env_t *env, const ir_node *node) {
 }
 
 static
-void emit_be_Return(ia32_emit_env_t *env, const ir_node *node) {
+void emit_be_Return(ia32_emit_env_t *env, const ir_node *node)
+{
        be_emit_cstring(env, "\tret");
        be_emit_finish_line_gas(env, node);
 }
 
 static
-void emit_Nothing(ia32_emit_env_t *env, const ir_node *node) {
+void emit_Nothing(ia32_emit_env_t *env, const ir_node *node)
+{
+       (void) env;
+       (void) node;
 }
 
 
@@ -1935,12 +2023,10 @@ void ia32_register_emitters(void) {
        IA32_EMIT(Asm);
        IA32_EMIT(CondJmp);
        IA32_EMIT(TestJmp);
-       IA32_EMIT(CJmp);
-       IA32_EMIT(CJmpAM);
        IA32_EMIT(CmpCMov);
-       IA32_EMIT(PsiCondCMov);
+       IA32_EMIT(TestCMov);
        IA32_EMIT(CmpSet);
-       IA32_EMIT(PsiCondSet);
+       IA32_EMIT(TestSet);
        IA32_EMIT(SwitchJmp);
        IA32_EMIT(CopyB);
        IA32_EMIT(CopyB_i);
@@ -2040,7 +2126,8 @@ void ia32_emit_node(ia32_emit_env_t *env, const ir_node *node) {
                (*func) (env, node);
        } else {
                emit_Nothing(env, node);
-               ir_fprintf(stderr, "Warning: No emit handler for node %+F (%+G)\n", node, node);
+               ir_fprintf(stderr, "Error: No emit handler for node %+F (%+G, graph %+F)\n", node, node, current_ir_graph);
+               abort();
        }
 }
 
@@ -2163,25 +2250,8 @@ void ia32_emit_block_header(ia32_emit_env_t *env, ir_node *block, ir_node *prev)
        int           i, arity;
        ir_exec_freq  *exec_freq = env->cg->birg->exec_freq;
 
-       need_label = 1;
        n_cfgpreds = get_Block_n_cfgpreds(block);
-       if (n_cfgpreds == 0) {
-               need_label = 0;
-       } else if (n_cfgpreds == 1) {
-               ir_node *pred       = get_Block_cfgpred(block, 0);
-               ir_node *pred_block = get_nodes_block(pred);
-
-               /* we don't need labels for fallthrough blocks, however switch-jmps
-                * are no fallthoughs */
-               if(pred_block == prev &&
-                               !(is_Proj(pred) && is_ia32_SwitchJmp(get_Proj_pred(pred)))) {
-                       need_label = 0;
-               } else {
-                       need_label = 1;
-               }
-       } else {
-               need_label = 1;
-       }
+       need_label = (n_cfgpreds != 0);
 
        if (should_align_block(env, block, prev)) {
                assert(need_label);
@@ -2277,9 +2347,11 @@ void ia32_emit_func_epilog(ia32_emit_env_t *env, ir_graph *irg) {
  * Sets labels for control flow nodes (jump target)
  */
 static
-void ia32_gen_labels(ir_node *block, void *data) {
+void ia32_gen_labels(ir_node *block, void *data)
+{
        ir_node *pred;
        int n = get_Block_n_cfgpreds(block);
+       (void) data;
 
        for (n--; n >= 0; n--) {
                pred = get_Block_cfgpred(block, n);