fixed indents
[libfirm] / ir / be / ia32 / ia32_emitter.c
index 12997a2..949e838 100644 (file)
 #include "irprog_t.h"
 #include "iredges_t.h"
 
-#include "../besched.h"
+#include "../besched_t.h"
 #include "../benode_t.h"
 
 #include "ia32_emitter.h"
 #include "gen_ia32_emitter.h"
+#include "gen_ia32_regalloc_if.h"
 #include "ia32_nodes_attr.h"
 #include "ia32_new_nodes.h"
 #include "ia32_map_regs.h"
 # define obstack_chunk_free free
 #endif
 
+#define BLOCK_PREFIX(x) ".L" x
+
 extern int obstack_printf(struct obstack *obst, char *fmt, ...);
 
 #define SNPRINTF_BUF_LEN 128
 
+/* global arch_env for lc_printf functions */
 static const arch_env_t *arch_env = NULL;
 
 /*************************************************************
@@ -55,6 +59,14 @@ static const arch_env_t *arch_env = NULL;
  * |_|                                       |_|
  *************************************************************/
 
+/**
+ * returns true if a node has x87 registers
+ */
+static int has_x87_register(const ir_node *n)
+{
+       return get_irn_op(n)->flags & (irop_flag_machine << 1);
+}
+
 /* We always pass the ir_node which is a pointer. */
 static int ia32_get_arg_type(const lc_arg_occ_t *occ) {
        return lc_arg_type_ptr;
@@ -128,6 +140,12 @@ static const char *get_ia32_reg_name(ir_node *irn, int pos, enum io_direction in
 
        if (in_out == IN_REG) {
                reg = get_in_reg(irn, pos);
+
+               if (reg->reg_class == &ia32_reg_classes[CLASS_ia32_vfp]) {
+                       /* FIXME: works for binop only */
+                       assert(2 <= pos && pos <= 3);
+                       reg = get_ia32_attr(irn)->x87[pos - 2];
+               }
        }
        else {
                /* destination address mode nodes don't have outputs */
@@ -136,8 +154,9 @@ static const char *get_ia32_reg_name(ir_node *irn, int pos, enum io_direction in
                }
 
                reg = get_out_reg(irn, pos);
+               if (reg->reg_class == &ia32_reg_classes[CLASS_ia32_vfp])
+                       reg = get_ia32_attr(irn)->x87[pos + 2];
        }
-
        return arch_register_get_name(reg);
 }
 
@@ -152,11 +171,33 @@ static int ia32_get_reg_name(lc_appendable_t *app,
        int         nr = occ->width - 1;
 
        if (!X)
-               return lc_arg_append(app, occ, "(null)", 6);
+               return lc_appendable_snadd(app, "(null)", 6);
 
        buf = get_ia32_reg_name(X, nr, occ->conversion == 'S' ? IN_REG : OUT_REG);
 
-       return lc_arg_append(app, occ, buf, strlen(buf));
+       /* append the stupid % to register names */
+       lc_appendable_chadd(app, '%');
+       return lc_appendable_snadd(app, buf, strlen(buf));
+}
+
+/**
+ * Get the x87 register name for a node.
+ */
+static int ia32_get_x87_name(lc_appendable_t *app,
+    const lc_arg_occ_t *occ, const lc_arg_value_t *arg)
+{
+       const char *buf;
+       ir_node     *X  = arg->v_ptr;
+       int         nr = occ->width - 1;
+       ia32_attr_t *attr;
+
+       if (!X)
+               return lc_appendable_snadd(app, "(null)", 6);
+
+       attr = get_ia32_attr(X);
+       buf = attr->x87[nr]->name;
+       lc_appendable_chadd(app, '%');
+       return lc_appendable_snadd(app, buf, strlen(buf));
 }
 
 /**
@@ -178,7 +219,7 @@ static int ia32_const_to_str(lc_appendable_t *app,
                buf = get_ia32_am_offs(X);
        }
 
-       return buf ? lc_arg_append(app, occ, buf, strlen(buf)) : 0;
+       return buf ? lc_appendable_snadd(app, buf, strlen(buf)) : 0;
 }
 
 /**
@@ -201,7 +242,6 @@ static int ia32_get_mode_suffix(lc_appendable_t *app,
                return lc_appendable_chadd(app, get_mode_size_bits(mode) == 32 ? 's' : 'd');
        }
        else {
-
                return lc_appendable_chadd(app, mode_is_signed(mode) ? 's' : 'z');
        }
 }
@@ -216,6 +256,7 @@ const lc_arg_env_t *ia32_get_arg_env(void) {
        static const lc_arg_handler_t ia32_reg_handler   = { ia32_get_arg_type, ia32_get_reg_name };
        static const lc_arg_handler_t ia32_const_handler = { ia32_get_arg_type, ia32_const_to_str };
        static const lc_arg_handler_t ia32_mode_handler  = { ia32_get_arg_type, ia32_get_mode_suffix };
+       static const lc_arg_handler_t ia32_x87_handler   = { ia32_get_arg_type, ia32_get_x87_name };
 
        if(env == NULL) {
                /* extend the firm printer */
@@ -226,11 +267,23 @@ const lc_arg_env_t *ia32_get_arg_env(void) {
                lc_arg_register(env, "ia32:cnst", 'C', &ia32_const_handler);
                lc_arg_register(env, "ia32:offs", 'O', &ia32_const_handler);
                lc_arg_register(env, "ia32:mode", 'M', &ia32_mode_handler);
+               lc_arg_register(env, "ia32:x87",  'X', &ia32_x87_handler);
        }
 
        return env;
 }
 
+static char *ia32_get_reg_name_for_mode(ia32_emit_env_t *env, ir_mode *mode, const arch_register_t *reg) {
+       switch(get_mode_size_bits(mode)) {
+               case 8:
+                       return ia32_get_mapped_reg_name(env->isa->regs_8bit, reg);
+               case 16:
+                       return ia32_get_mapped_reg_name(env->isa->regs_16bit, reg);
+               default:
+                       return (char *)arch_register_get_name(reg);
+       }
+}
+
 /**
  * Emits registers and/or address mode of a binary operation.
  */
@@ -238,9 +291,14 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *env) {
        static char *buf = NULL;
 
        /* verify that this function is never called on non-AM supporting operations */
-       assert(get_ia32_am_support(n) != ia32_am_None && "emit binop expects addressmode support");
+       //assert(get_ia32_am_support(n) != ia32_am_None && "emit binop expects addressmode support");
 
-#define PRODUCES_RESULT(n) !(is_ia32_St(n) || is_ia32_CondJmp(n) || is_ia32_fCondJmp(n) || is_ia32_SwitchJmp(n))
+#define PRODUCES_RESULT(n)   \
+       (!(is_ia32_St(n)      || \
+       is_ia32_Store8Bit(n)  || \
+       is_ia32_CondJmp(n)    || \
+       is_ia32_fCondJmp(n)   || \
+       is_ia32_SwitchJmp(n)))
 
        if (! buf) {
                buf = xcalloc(1, SNPRINTF_BUF_LEN);
@@ -251,7 +309,7 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *env) {
 
        switch(get_ia32_op_type(n)) {
                case ia32_Normal:
-                       if (get_ia32_cnst(n)) {
+                       if (is_ia32_ImmConst(n) || is_ia32_ImmSymConst(n)) {
                                lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%3S, %s", n, get_ia32_cnst(n));
                        }
                        else {
@@ -259,46 +317,101 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *env) {
                                const arch_register_t *in2 = get_in_reg(n, 3);
                                const arch_register_t *out = PRODUCES_RESULT(n) ? get_out_reg(n, 0) : NULL;
                                const arch_register_t *in;
+                               const char            *in_name;
+
+                               in      = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
+                               out     = out ? out : in1;
+                               in_name = arch_register_get_name(in);
 
-                               in  = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
-                               out = out ? out : in1;
+                               if (is_ia32_emit_cl(n)) {
+                                       assert(REGS_ARE_EQUAL(&ia32_gp_regs[REG_ECX], in) && "shift operation needs ecx");
+                                       in_name = "cl";
+                               }
 
-                               snprintf(buf, SNPRINTF_BUF_LEN, "%s, %s", \
-                                       arch_register_get_name(out), arch_register_get_name(in));
+                               snprintf(buf, SNPRINTF_BUF_LEN, "%%%s, %%%s", arch_register_get_name(out), in_name);
                        }
                        break;
                case ia32_AddrModeS:
-                       lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%4S, %s", n, ia32_emit_am(n, env));
+                       if (is_ia32_ImmConst(n) || is_ia32_ImmSymConst(n)) {
+                               assert(! PRODUCES_RESULT(n) && "Source AM with Const must not produce result");
+                               snprintf(buf, SNPRINTF_BUF_LEN, "%s, %s", get_ia32_cnst(n), ia32_emit_am(n, env));
+                       }
+                       else {
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%4S, %s", n, ia32_emit_am(n, env));
+                       }
                        break;
                case ia32_AddrModeD:
-                       if (get_ia32_cnst(n)) {
-                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s, %s", ia32_emit_am(n, env), get_ia32_cnst(n));
+                       if (is_ia32_ImmConst(n) || is_ia32_ImmSymConst(n)) {
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s,%s%s",
+                                       ia32_emit_am(n, env),
+                                       is_ia32_ImmSymConst(n) ? " OFFSET FLAT:" : " ",  /* In case of a symconst we must add OFFSET to */
+                                       get_ia32_cnst(n));                               /* tell the assembler to store it's address.   */
                        }
                        else {
                                const arch_register_t *in1 = get_in_reg(n, 2);
-                               const char *reg_name;
-                               ir_mode *mode = get_ia32_res_mode(n);
-
-                               mode = mode ? mode : get_ia32_ls_mode(n);
-
-                               switch(get_mode_size_bits(mode)) {
-                                       case 8:
-                                               reg_name = ia32_get_mapped_reg_name(env->isa->regs_8bit, in1);
-                                               break;
-                                       case 16:
-                                               reg_name = ia32_get_mapped_reg_name(env->isa->regs_16bit, in1);
-                                               break;
-                                       case 32:
-                                               reg_name = arch_register_get_name(in1);
-                                               break;
-                                       default:
-                                               assert(0 && "unsupported mode size");
-                                               break;
+                               ir_mode              *mode = get_ia32_res_mode(n);
+                               const char           *in_name;
+
+                               mode    = mode ? mode : get_ia32_ls_mode(n);
+                               in_name = ia32_get_reg_name_for_mode(env, mode, in1);
+
+                               if (is_ia32_emit_cl(n)) {
+                                       assert(REGS_ARE_EQUAL(&ia32_gp_regs[REG_ECX], in1) && "shift operation needs ecx");
+                                       in_name = "cl";
                                }
 
-                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s, %s", ia32_emit_am(n, env), reg_name);
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s, %%%s", ia32_emit_am(n, env), in_name);
+                       }
+                       break;
+               default:
+                       assert(0 && "unsupported op type");
+       }
+
+#undef PRODUCES_RESULT
+
+       return buf;
+}
+
+/**
+ * Emits registers and/or address mode of a binary operation.
+ */
+char *ia32_emit_x87_binop(const ir_node *n, ia32_emit_env_t *env) {
+       static char *buf = NULL;
+
+       /* verify that this function is never called on non-AM supporting operations */
+       //assert(get_ia32_am_support(n) != ia32_am_None && "emit binop expects addressmode support");
+
+       if (! buf) {
+               buf = xcalloc(1, SNPRINTF_BUF_LEN);
+       }
+       else {
+               memset(buf, 0, SNPRINTF_BUF_LEN);
+       }
+
+       switch(get_ia32_op_type(n)) {
+               case ia32_Normal:
+                       if (is_ia32_ImmConst(n) || is_ia32_ImmSymConst(n)) {
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%3S, %s", n, get_ia32_cnst(n));
+                       }
+                       else {
+                               ia32_attr_t *attr = get_ia32_attr(n);
+                               const arch_register_t *in1 = attr->x87[0];
+                               const arch_register_t *in2 = attr->x87[1];
+                               const arch_register_t *out = attr->x87[2];
+                               const arch_register_t *in;
+                               const char            *in_name;
+
+                               in      = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
+                               out     = out ? out : in1;
+                               in_name = arch_register_get_name(in);
+
+                               snprintf(buf, SNPRINTF_BUF_LEN, "%%%s, %%%s", arch_register_get_name(out), in_name);
                        }
                        break;
+               case ia32_AddrModeS:
+               case ia32_AddrModeD:
+                       lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s", ia32_emit_am(n, env));
+                       break;
                default:
                        assert(0 && "unsupported op type");
        }
@@ -323,7 +436,12 @@ char *ia32_emit_unop(const ir_node *n, ia32_emit_env_t *env) {
 
        switch(get_ia32_op_type(n)) {
                case ia32_Normal:
-                       lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%1D", n);
+                       if (is_ia32_ImmConst(n) || is_ia32_ImmSymConst(n)) {
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%C", n);
+                       }
+                       else {
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%1D", n);
+                       }
                        break;
                case ia32_am_Dest:
                        snprintf(buf, SNPRINTF_BUF_LEN, ia32_emit_am(n, env));
@@ -362,19 +480,37 @@ char *ia32_emit_am(const ir_node *n, ia32_emit_env_t *env) {
        if (mode) {
                switch (get_mode_size_bits(mode)) {
                        case 8:
-                               obstack_printf(obst, "BYTE ");
+                               obstack_printf(obst, "BYTE PTR ");
                                break;
                        case 16:
-                               obstack_printf(obst, "WORD ");
+                               obstack_printf(obst, "WORD PTR ");
+                               break;
+                       case 32:
+                               obstack_printf(obst, "DWORD PTR ");
+                               break;
+                       case 64:
+                               if (has_x87_register(n))
+                                       /* ARGHHH: stupid gas x87 wants QWORD PTR but SSE must be WITHOUT */
+                                       obstack_printf(obst, "QWORD PTR ");
+                               break;
+                       case 80:
+                       case 96:
+                               obstack_printf(obst, "XWORD PTR ");
                                break;
                        default:
                                break;
                }
        }
 
-       obstack_printf(obst, "[");
+       /* emit address mode symconst */
+       if (get_ia32_am_sc(n)) {
+               if (is_ia32_am_sc_sign(n))
+                       obstack_printf(obst, "-");
+               obstack_printf(obst, "%s", get_id_str(get_ia32_am_sc(n)));
+       }
 
        if (am_flav & ia32_B) {
+               obstack_printf(obst, "[");
                lc_eoprintf(ia32_get_arg_env(), obst, "%1S", n);
                had_output = 1;
        }
@@ -383,6 +519,9 @@ char *ia32_emit_am(const ir_node *n, ia32_emit_env_t *env) {
                if (had_output) {
                        obstack_printf(obst, "+");
                }
+               else {
+                       obstack_printf(obst, "[");
+               }
 
                lc_eoprintf(ia32_get_arg_env(), obst, "%2S", n);
 
@@ -394,10 +533,23 @@ char *ia32_emit_am(const ir_node *n, ia32_emit_env_t *env) {
        }
 
        if (am_flav & ia32_O) {
-               obstack_printf(obst, get_ia32_am_offs(n));
+               s = get_ia32_am_offs(n);
+
+               if (s) {
+                       /* omit explicit + if there was no base or index */
+                       if (! had_output) {
+                               obstack_printf(obst, "[");
+                               if (s[0] == '+')
+                                       s++;
+                       }
+
+                       obstack_printf(obst, s);
+                       had_output = 1;
+               }
        }
 
-       obstack_printf(obst, "] ");
+       if (had_output)
+               obstack_printf(obst, "] ");
 
        size        = obstack_object_size(obst);
        s           = obstack_finish(obst);
@@ -411,8 +563,14 @@ char *ia32_emit_am(const ir_node *n, ia32_emit_env_t *env) {
 /**
  * Formated print of commands and comments.
  */
-static void ia32_fprintf_format(FILE *F, char *cmd_buf, char *cmnt_buf) {
-       fprintf(F, "\t%-35s %-60s\n", cmd_buf, cmnt_buf);
+static void ia32_fprintf_format(FILE *F, const ir_node *irn, char *cmd_buf, char *cmnt_buf) {
+       unsigned lineno;
+       const char *name = irn ? be_retrieve_dbg_info(get_irn_dbg_info((ir_node *)irn), &lineno) : NULL;
+
+       if (name)
+               fprintf(F, "\t%-35s %-60s /* %s:%u */\n", cmd_buf, cmnt_buf, name, lineno);
+       else
+               fprintf(F, "\t%-35s %-60s\n", cmd_buf, cmnt_buf);
 }
 
 
@@ -439,7 +597,7 @@ static char *get_unique_label(char *buf, size_t buflen, const char *prefix) {
  *************************************************/
 
 #undef IA32_DO_EMIT
-#define IA32_DO_EMIT ia32_fprintf_format(F, cmd_buf, cmnt_buf)
+#define IA32_DO_EMIT(irn) ia32_fprintf_format(F, irn, cmd_buf, cmnt_buf)
 
 /*
  * coding of conditions
@@ -475,22 +633,22 @@ static const struct cmp2conditon_t cmp2condition_s[] = {
  * positive conditions for unsigned compares
  */
 static const struct cmp2conditon_t cmp2condition_u[] = {
-  { NULL,              pn_Cmp_False },  /* always false */
-  { "e",               pn_Cmp_Eq },     /* == */
-  { "b",               pn_Cmp_Lt },     /* < */
-  { "be",              pn_Cmp_Le },     /* <= */
-  { "a",               pn_Cmp_Gt },     /* > */
-  { "ae",              pn_Cmp_Ge },     /* >= */
-  { "ne",              pn_Cmp_Lg },     /* != */
-  { "ordered",         pn_Cmp_Leg },    /* Floating point: ordered */
-  { "unordered",       pn_Cmp_Uo },     /* FLoting point: unordered */
-  { "unordered or ==", pn_Cmp_Ue },     /* Floating point: unordered or == */
-  { "unordered or <",  pn_Cmp_Ul },     /* Floating point: unordered or < */
-  { "unordered or <=", pn_Cmp_Ule },    /* Floating point: unordered or <= */
-  { "unordered or >",  pn_Cmp_Ug },     /* Floating point: unordered or > */
-  { "unordered or >=", pn_Cmp_Uge },    /* Floating point: unordered or >= */
-  { "unordered or !=", pn_Cmp_Ne },     /* Floating point: unordered or != */
-  { NULL,              pn_Cmp_True },   /* always true */
+       { NULL,              pn_Cmp_False },  /* always false */
+       { "e",               pn_Cmp_Eq },     /* == */
+       { "b",               pn_Cmp_Lt },     /* < */
+       { "be",              pn_Cmp_Le },     /* <= */
+       { "a",               pn_Cmp_Gt },     /* > */
+       { "ae",              pn_Cmp_Ge },     /* >= */
+       { "ne",              pn_Cmp_Lg },     /* != */
+       { "ordered",         pn_Cmp_Leg },    /* Floating point: ordered */
+       { "unordered",       pn_Cmp_Uo },     /* FLoting point: unordered */
+       { "unordered or ==", pn_Cmp_Ue },     /* Floating point: unordered or == */
+       { "unordered or <",  pn_Cmp_Ul },     /* Floating point: unordered or < */
+       { "unordered or <=", pn_Cmp_Ule },    /* Floating point: unordered or <= */
+       { "unordered or >",  pn_Cmp_Ug },     /* Floating point: unordered or > */
+       { "unordered or >=", pn_Cmp_Uge },    /* Floating point: unordered or >= */
+       { "unordered or !=", pn_Cmp_Ne },     /* Floating point: unordered or != */
+       { NULL,              pn_Cmp_True },   /* always true */
 };
 
 /*
@@ -504,53 +662,91 @@ static const char *get_cmp_suffix(int cmp_code, int unsigned_cmp)
        return unsigned_cmp ? cmp2condition_u[cmp_code & 7].name : cmp2condition_s[cmp_code & 7].name;
 }
 
+/**
+ * Returns the target block for a control flow node.
+ */
+static ir_node *get_cfop_target_block(const ir_node *irn) {
+       return get_irn_link(irn);
+}
+
 /**
  * Returns the target label for a control flow node.
  */
 static char *get_cfop_target(const ir_node *irn, char *buf) {
-       ir_node *bl = get_irn_link(irn);
+       ir_node *bl = get_cfop_target_block(irn);
 
-       snprintf(buf, SNPRINTF_BUF_LEN, "BLOCK_%ld", get_irn_node_nr(bl));
+       snprintf(buf, SNPRINTF_BUF_LEN, BLOCK_PREFIX("%ld"), get_irn_node_nr(bl));
        return buf;
 }
 
+/** Return the next block in Block schedule */
+static ir_node *next_blk_sched(const ir_node *block) {
+       return get_irn_link(block);
+}
+
 /**
  * Emits the jump sequence for a conditional jump (cmp + jmp_true + jmp_false)
  */
-static void finish_CondJmp(FILE *F, const ir_node *irn) {
-       const ir_node   *proj;
+static void finish_CondJmp(FILE *F, const ir_node *irn, ir_mode *mode) {
+       const ir_node   *proj1, *proj2 = NULL;
+       const ir_node   *block, *next_bl = NULL;
        const ir_edge_t *edge;
        char buf[SNPRINTF_BUF_LEN];
        char cmd_buf[SNPRINTF_BUF_LEN];
        char cmnt_buf[SNPRINTF_BUF_LEN];
 
+       /* get both Proj's */
        edge = get_irn_out_edge_first(irn);
-       proj = get_edge_src_irn(edge);
-       assert(is_Proj(proj) && "CondJmp with a non-Proj");
+       proj1 = get_edge_src_irn(edge);
+       assert(is_Proj(proj1) && "CondJmp with a non-Proj");
+
+       edge = get_irn_out_edge_next(irn, edge);
+       if (edge) {
+               proj2 = get_edge_src_irn(edge);
+               assert(is_Proj(proj2) && "CondJmp with a non-Proj");
+       }
+
+       /* for now, the code works for scheduled and non-schedules blocks */
+       block = get_nodes_block(irn);
+       if (proj2) {
+               /* we have a block schedule */
+               next_bl = next_blk_sched(block);
+
+               if (get_cfop_target_block(proj1) == next_bl) {
+                       /* exchange both proj's so the second one can be omitted */
+                       const ir_node *t = proj1;
+                       proj1 = proj2;
+                       proj2 = t;
+               }
+       }
 
-       if (get_Proj_proj(proj) == 1) {
+       /* the first Proj must always be created */
+       if (get_Proj_proj(proj1) == pn_Cond_true) {
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, "j%s %s",
                                        get_cmp_suffix(get_ia32_pncode(irn), !mode_is_signed(get_irn_mode(get_irn_n(irn, 0)))),
-                                       get_cfop_target(proj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; cmp(a, b) == TRUE");
+                                       get_cfop_target(proj1, buf));
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* cmp(a, b) == TRUE */");
        }
        else  {
-               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jn%s %s",
-                                       get_cmp_suffix(get_ia32_pncode(irn), !mode_is_signed(get_irn_mode(get_irn_n(irn, 0)))),
-                                       get_cfop_target(proj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; cmp(a, b) == FALSE");
+               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "j%s %s",
+                                       get_cmp_suffix(get_negated_pnc(get_ia32_pncode(irn), mode),
+                                       !mode_is_signed(get_irn_mode(get_irn_n(irn, 0)))),
+                                       get_cfop_target(proj1, buf));
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* cmp(a, b) == FALSE */");
        }
+       IA32_DO_EMIT(irn);
 
-       IA32_DO_EMIT;
-
-       edge = get_irn_out_edge_next(irn, edge);
-       if (edge) {
-               proj = get_edge_src_irn(edge);
-               assert(is_Proj(proj) && "CondJmp with a non-Proj");
-               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(proj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; otherwise");
-
-               IA32_DO_EMIT;
+       /* the second Proj might be a fallthrough */
+       if (proj2) {
+               if (get_cfop_target_block(proj2) != next_bl) {
+                       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(proj2, buf));
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* otherwise */");
+               }
+               else {
+                       cmd_buf[0] = '\0';
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* fallthrogh %s */", get_cfop_target(proj2, buf));
+               }
+               IA32_DO_EMIT(irn);
        }
 }
 
@@ -563,9 +759,9 @@ static void CondJmp_emitter(const ir_node *irn, ia32_emit_env_t *env) {
        char cmnt_buf[SNPRINTF_BUF_LEN];
 
        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "cmp %s", ia32_emit_binop(irn, env));
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F", irn);
-       IA32_DO_EMIT;
-       finish_CondJmp(F, irn);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
+       IA32_DO_EMIT(irn);
+       finish_CondJmp(F, irn, get_ia32_res_mode(irn));
 }
 
 /**
@@ -576,13 +772,58 @@ static void emit_ia32_CondJmp(const ir_node *irn, ia32_emit_env_t *env) {
 }
 
 /**
- * Emits code for conditional jump with immediate.
+ * Emits code for conditional test and jump.
  */
-void emit_ia32_CondJmp_i(const ir_node *irn, ia32_emit_env_t *env) {
-       CondJmp_emitter(irn, env);
+static void TestJmp_emitter(const ir_node *irn, ia32_emit_env_t *env) {
+
+#define IA32_IS_IMMOP (is_ia32_ImmConst(irn) || is_ia32_ImmSymConst(irn))
+
+       FILE       *F   = env->out;
+       const char *op1 = arch_register_get_name(get_in_reg(irn, 0));
+       const char *op2 = IA32_IS_IMMOP ? get_ia32_cnst(irn) : NULL;
+       char        cmd_buf[SNPRINTF_BUF_LEN];
+       char        cmnt_buf[SNPRINTF_BUF_LEN];
+
+       if (! op2)
+               op2 = arch_register_get_name(get_in_reg(irn, 1));
+
+       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "test %%%s,%s%s ", op1, IA32_IS_IMMOP ? " " : " %", op2);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
+
+       IA32_DO_EMIT(irn);
+       finish_CondJmp(F, irn, get_ia32_res_mode(irn));
+
+#undef IA32_IS_IMMOP
 }
 
+/**
+ * Emits code for conditional test and jump with two variables.
+ */
+static void emit_ia32_TestJmp(const ir_node *irn, ia32_emit_env_t *env) {
+       TestJmp_emitter(irn, env);
+}
 
+static void emit_ia32_CJmp(const ir_node *irn, ia32_emit_env_t *env) {
+       FILE *F = env->out;
+       char cmd_buf[SNPRINTF_BUF_LEN];
+       char cmnt_buf[SNPRINTF_BUF_LEN];
+
+       snprintf(cmd_buf, SNPRINTF_BUF_LEN, " ");
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F omitted redundant test */", irn);
+       IA32_DO_EMIT(irn);
+       finish_CondJmp(F, irn, get_ia32_res_mode(irn));
+}
+
+static void emit_ia32_CJmpAM(const ir_node *irn, ia32_emit_env_t *env) {
+       FILE *F = env->out;
+       char cmd_buf[SNPRINTF_BUF_LEN];
+       char cmnt_buf[SNPRINTF_BUF_LEN];
+
+       snprintf(cmd_buf, SNPRINTF_BUF_LEN, " ");
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F omitted redundant test/cmp */", irn);
+       IA32_DO_EMIT(irn);
+       finish_CondJmp(F, irn, get_ia32_res_mode(irn));
+}
 
 /*********************************************************
  *                 _ _       _
@@ -629,7 +870,7 @@ static int ia32_cmp_branch_t(const void *a, const void *b) {
  * possible otherwise a cmp-jmp cascade). Port from
  * cggg ia32 backend
  */
-void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
        unsigned long       interval;
        char                buf[SNPRINTF_BUF_LEN];
        int                 last_value, i, pn, do_jmp_tbl = 1;
@@ -694,27 +935,27 @@ void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
                if (tbl.min_value != 0) {
                        lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "cmpl %lu, -%d(%1S)",
                                interval, tbl.min_value, irn);
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; first switch value is not 0");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* first switch value is not 0 */");
 
-                       IA32_DO_EMIT;
+                       IA32_DO_EMIT(irn);
                }
                else {
                        lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "cmpl %lu, %1S", interval, irn);
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; compare for switch");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* compare for switch */");
 
-                       IA32_DO_EMIT;
+                       IA32_DO_EMIT(irn);
                }
 
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, "ja %s", get_cfop_target(tbl.defProj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; default jump if out of range ");
-               IA32_DO_EMIT;
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default jump if out of range  */");
+               IA32_DO_EMIT(irn);
 
                if (tbl.num_branches > 1) {
                        /* create table */
 
                        lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "jmp [%1S*4+%s]", irn, tbl.label);
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; get jump table entry as target");
-                       IA32_DO_EMIT;
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* get jump table entry as target */");
+                       IA32_DO_EMIT(irn);
 
                        fprintf(F, "\t.section\t.rodata\n");
                        fprintf(F, "\t.align 4\n");
@@ -723,18 +964,18 @@ void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
 
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, ".long %s", get_cfop_target(tbl.branches[0].target, buf));
                        snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* case %d */\n",  tbl.branches[0].value);
-                       IA32_DO_EMIT;
+                       IA32_DO_EMIT(irn);
 
                        last_value = tbl.branches[0].value;
                        for (i = 1; i < tbl.num_branches; ++i) {
                                while (++last_value < tbl.branches[i].value) {
                                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, ".long %s", get_cfop_target(tbl.defProj, buf));
-                                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; default case");
-                                       IA32_DO_EMIT;
+                                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default case */");
+                                       IA32_DO_EMIT(irn);
                                }
-                               snprintf(cmd_buf, SNPRINTF_BUF_LEN, ".long %s", get_cfop_target(tbl.branches[i].target, buf), last_value);
-                               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; case %d", last_value);
-                               IA32_DO_EMIT;
+                               snprintf(cmd_buf, SNPRINTF_BUF_LEN, ".long %s", get_cfop_target(tbl.branches[i].target, buf));
+                               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* case %d */", last_value);
+                               IA32_DO_EMIT(irn);
                        }
 
                        fprintf(F, "\t.text");
@@ -742,21 +983,21 @@ void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
                else {
                        /* one jump is enough */
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(tbl.branches[0].target, buf));
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; only one case given");
-                       IA32_DO_EMIT;
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* only one case given */");
+                       IA32_DO_EMIT(irn);
                }
        }
        else { // no jump table
                for (i = 0; i < tbl.num_branches; ++i) {
                        lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "cmpl %d, %1S", tbl.branches[i].value, irn);
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; case %d", i);
-                       IA32_DO_EMIT;
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* case %d */", i);
+                       IA32_DO_EMIT(irn);
                        fprintf(F, "\tje %s\n", get_cfop_target(tbl.branches[i].target, buf));
                }
 
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(tbl.defProj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; default case");
-               IA32_DO_EMIT;
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default case */");
+               IA32_DO_EMIT(irn);
        }
 
        if (tbl.label)
@@ -768,16 +1009,26 @@ void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
 /**
  * Emits code for a unconditional jump.
  */
-void emit_Jmp(const ir_node *irn, ia32_emit_env_t *env) {
+static void emit_Jmp(const ir_node *irn, ia32_emit_env_t *env) {
+       ir_node *block, *next_bl;
        FILE *F = env->out;
        char buf[SNPRINTF_BUF_LEN], cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
-       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(irn, buf), get_irn_link(irn));
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F(%+F)", irn, get_irn_link(irn));
-       IA32_DO_EMIT;
-}
-
+       /* for now, the code works for scheduled and non-schedules blocks */
+       block = get_nodes_block(irn);
 
+       /* we have a block schedule */
+       next_bl = next_blk_sched(block);
+       if (get_cfop_target_block(irn) != next_bl) {
+               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(irn, buf));
+               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%+F) */", irn, get_cfop_target_block(irn));
+       }
+       else {
+               cmd_buf[0] = '\0';
+               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* fallthrough %s */", get_cfop_target(irn, buf));
+       }
+       IA32_DO_EMIT(irn);
+}
 
 /****************************
  *                  _
@@ -793,7 +1044,7 @@ void emit_Jmp(const ir_node *irn, ia32_emit_env_t *env) {
 /**
  * Emits code for a proj -> node
  */
-void emit_Proj(const ir_node *irn, ia32_emit_env_t *env) {
+static void emit_Proj(const ir_node *irn, ia32_emit_env_t *env) {
        ir_node *pred = get_Proj_pred(irn);
 
        if (get_irn_op(pred) == op_Start) {
@@ -828,33 +1079,33 @@ static void emit_CopyB_prolog(FILE *F, int rem, int size) {
 
        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "cld");
        snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* copy direction forward*/");
-       IA32_DO_EMIT;
+       IA32_DO_EMIT(NULL);
 
        switch(rem) {
                case 1:
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsb");
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy remainder 1");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 1 */");
                        break;
                case 2:
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsw");
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy remainder 2");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 2 */");
                        break;
                case 3:
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsb");
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy remainder 3");
-                       IA32_DO_EMIT;
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 3 */");
+                       IA32_DO_EMIT(NULL);
                        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsw");
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy remainder 3");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 3 */");
                        break;
        }
 
-       IA32_DO_EMIT;
+       IA32_DO_EMIT(NULL);
 }
 
 /**
  * Emit rep movsd instruction for memcopy.
  */
-void emit_ia32_CopyB(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_ia32_CopyB(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE   *F    = emit_env->out;
        tarval *tv   = get_ia32_Immop_tarval(irn);
        int     rem  = get_tarval_long(tv);
@@ -864,14 +1115,14 @@ void emit_ia32_CopyB(const ir_node *irn, ia32_emit_env_t *emit_env) {
        emit_CopyB_prolog(F, rem, size);
 
        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "rep movsd");
-       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy");
-       IA32_DO_EMIT;
+       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy */");
+       IA32_DO_EMIT(irn);
 }
 
 /**
  * Emits unrolled memcopy.
  */
-void emit_ia32_CopyB_i(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_ia32_CopyB_i(const ir_node *irn, ia32_emit_env_t *emit_env) {
        tarval *tv   = get_ia32_Immop_tarval(irn);
        int     size = get_tarval_long(tv);
        FILE   *F    = emit_env->out;
@@ -882,8 +1133,8 @@ void emit_ia32_CopyB_i(const ir_node *irn, ia32_emit_env_t *emit_env) {
        size >>= 2;
        while (size--) {
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsd");
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; memcopy unrolled");
-               IA32_DO_EMIT;
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy unrolled */");
+               IA32_DO_EMIT(irn);
        }
 }
 
@@ -902,7 +1153,7 @@ void emit_ia32_CopyB_i(const ir_node *irn, ia32_emit_env_t *emit_env) {
 /**
  * Emit code for conversions (I, FP), (FP, I) and (FP, FP).
  */
-static void emit_ia32_Conv(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_ia32_Conv_with_FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE               *F    = emit_env->out;
        const lc_arg_env_t *env  = ia32_get_arg_env();
        char               *from, *to, buf[64];
@@ -927,22 +1178,99 @@ static void emit_ia32_Conv(const ir_node *irn, ia32_emit_env_t *emit_env) {
        }
 
        snprintf(cmd_buf, SNPRINTF_BUF_LEN, "cvt%s2%s %s", from, to, buf);
-       lc_esnprintf(env, cmnt_buf, SNPRINTF_BUF_LEN, "; %+F(%+F, %+F)", irn, src_mode, tgt_mode);
-       IA32_DO_EMIT;
+       lc_esnprintf(env, cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%+F, %+F) */", irn, src_mode, tgt_mode);
+       IA32_DO_EMIT(irn);
 }
 
-void emit_ia32_Conv_I2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
-       emit_ia32_Conv(irn, emit_env);
+static void emit_ia32_Conv_I2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
+       emit_ia32_Conv_with_FP(irn, emit_env);
 }
 
-void emit_ia32_Conv_FP2I(const ir_node *irn, ia32_emit_env_t *emit_env) {
-       emit_ia32_Conv(irn, emit_env);
+static void emit_ia32_Conv_FP2I(const ir_node *irn, ia32_emit_env_t *emit_env) {
+       emit_ia32_Conv_with_FP(irn, emit_env);
 }
 
-void emit_ia32_Conv_FP2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
-       emit_ia32_Conv(irn, emit_env);
+static void emit_ia32_Conv_FP2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
+       emit_ia32_Conv_with_FP(irn, emit_env);
 }
 
+/**
+ * Emits code for an Int conversion.
+ */
+static void emit_ia32_Conv_I2I(const ir_node *irn, ia32_emit_env_t *emit_env) {
+       FILE               *F        = emit_env->out;
+       const lc_arg_env_t *env      = ia32_get_arg_env();
+       char               *move_cmd = "movzx";
+       char               *conv_cmd = NULL;
+       ir_mode *src_mode, *tgt_mode;
+       int n, m;
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
+       const arch_register_t *in_reg, *out_reg;
+
+       src_mode = is_ia32_AddrModeS(irn) ? get_ia32_ls_mode(irn) : get_irn_mode(get_irn_n(irn, 2));
+       tgt_mode = get_ia32_res_mode(irn);
+
+       n = get_mode_size_bits(src_mode);
+       m = get_mode_size_bits(tgt_mode);
+
+       if (mode_is_signed(n < m ? src_mode : tgt_mode)) {
+               move_cmd = "movsx";
+               if (n == 8 || m == 8)
+                       conv_cmd = "cbw";
+               else if (n == 16 || m == 16)
+                       conv_cmd = "cwde";
+               else
+                       assert(0 && "unsupported Conv_I2I");
+       }
+
+       switch(get_ia32_op_type(irn)) {
+               case ia32_Normal:
+                       in_reg  = get_in_reg(irn, 2);
+                       out_reg = get_out_reg(irn, 0);
+
+                       if (REGS_ARE_EQUAL(in_reg, &ia32_gp_regs[REG_EAX]) &&
+                               REGS_ARE_EQUAL(out_reg, in_reg)                &&
+                               mode_is_signed(n < m ? src_mode : tgt_mode))
+                       {
+                               /* argument and result are both in EAX and */
+                               /* signedness is ok: -> use converts       */
+                               lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "%s", conv_cmd);
+                       }
+                       else if (REGS_ARE_EQUAL(out_reg, in_reg) &&
+                               ! mode_is_signed(n < m ? src_mode : tgt_mode))
+                       {
+                               /* argument and result are in the same register */
+                               /* and signedness is ok: -> use and with mask   */
+                               int mask = (1 << (n < m ? n : m)) - 1;
+                               lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "and %1D, 0x%x", irn, mask);
+                       }
+                       else {
+                               /* use move w/o sign extension */
+                               lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "%s %1D, %%%s",
+                                       move_cmd, irn, ia32_get_reg_name_for_mode(emit_env, n < m ? src_mode : tgt_mode, in_reg));
+                       }
+
+                       break;
+               case ia32_AddrModeS:
+                       lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "%s %1D, %s",
+                               move_cmd, irn, ia32_emit_am(irn, emit_env));
+                       break;
+               default:
+                       assert(0 && "unsupported op type for Conv");
+       }
+
+       lc_esnprintf(env, cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%d Bit mode_%F -> %d Bit mode_%F) */",
+               irn, n, src_mode, m, tgt_mode);
+
+       IA32_DO_EMIT(irn);
+}
+
+/**
+ * Emits code for an 8Bit Int conversion.
+ */
+void emit_ia32_Conv_I2I8Bit(const ir_node *irn, ia32_emit_env_t *emit_env) {
+       emit_ia32_Conv_I2I(irn, emit_env);
+}
 
 
 /*******************************************
@@ -958,7 +1286,7 @@ void emit_ia32_Conv_FP2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
 /**
  * Emits a backend call
  */
-void emit_be_Call(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_be_Call(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE *F = emit_env->out;
        entity *ent = be_Call_get_entity(irn);
        char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
@@ -970,15 +1298,15 @@ void emit_be_Call(const ir_node *irn, ia32_emit_env_t *emit_env) {
                lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "%1D", get_irn_n(irn, be_pos_Call_ptr));
        }
 
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F (be_Call)", irn);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F (be_Call) */", irn);
 
-       IA32_DO_EMIT;
+       IA32_DO_EMIT(irn);
 }
 
 /**
  * Emits code to increase stack pointer.
  */
-void emit_be_IncSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_be_IncSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE          *F    = emit_env->out;
        unsigned       offs = be_get_IncSP_offset(irn);
        be_stack_dir_t dir  = be_get_IncSP_direction(irn);
@@ -986,53 +1314,58 @@ void emit_be_IncSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
 
        if (offs) {
                lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "add %1S,%s%u", irn,
-                       (dir == be_stack_dir_along) ? " -" : " ", offs);
-               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F (IncSP)", irn);
+                       (dir == be_stack_dir_expand) ? " -" : " ", offs);
+               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F (IncSP) */", irn);
        }
        else {
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, " ");
-               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; omitted %+F (IncSP) with 0", irn);
+               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* omitted %+F (IncSP) with 0 */", irn);
        }
 
-       IA32_DO_EMIT;
+       IA32_DO_EMIT(irn);
 }
 
 /**
  * Emits code to set stack pointer.
  */
-void emit_be_SetSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_be_SetSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE *F = emit_env->out;
        char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "mov %1D, %3S", irn, irn);
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F (restore SP)", irn);
-       IA32_DO_EMIT;
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F (restore SP) */", irn);
+       IA32_DO_EMIT(irn);
 }
 
 /**
  * Emits code for Copy.
  */
-void emit_be_Copy(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_be_Copy(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE *F = emit_env->out;
        char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
-       lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "mov %1D, %1S", irn, irn);
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F", irn);
-       IA32_DO_EMIT;
+       if (mode_is_float(get_irn_mode(irn)))
+               lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "movs%M %1D, %1S", irn, irn, irn);
+       else
+               lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "mov %1D, %1S", irn, irn);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
+       IA32_DO_EMIT(irn);
 }
 
 /**
  * Emits code for exchange.
  */
-void emit_be_Perm(const ir_node *irn, ia32_emit_env_t *emit_env) {
+static void emit_be_Perm(const ir_node *irn, ia32_emit_env_t *emit_env) {
        FILE *F = emit_env->out;
        char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "xchg %1S, %2S", irn, irn);
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F(%1A, %2A)", irn, irn, irn);
-       IA32_DO_EMIT;
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%1A, %2A) */", irn, irn, irn);
+       IA32_DO_EMIT(irn);
 }
 
+
+
 /***********************************************************************************
  *                  _          __                                             _
  *                 (_)        / _|                                           | |
@@ -1061,12 +1394,17 @@ static void ia32_register_emitters(void) {
 
        /* other ia32 emitter functions */
        IA32_EMIT(CondJmp);
+       IA32_EMIT(TestJmp);
+       IA32_EMIT(CJmp);
+       IA32_EMIT(CJmpAM);
        IA32_EMIT(SwitchJmp);
        IA32_EMIT(CopyB);
        IA32_EMIT(CopyB_i);
        IA32_EMIT(Conv_I2FP);
        IA32_EMIT(Conv_FP2I);
        IA32_EMIT(Conv_FP2FP);
+       IA32_EMIT(Conv_I2I);
+       IA32_EMIT(Conv_I2I8Bit);
 
        /* benode emitter */
        BE_EMIT(Call);
@@ -1100,7 +1438,7 @@ static void ia32_emit_node(const ir_node *irn, void *env) {
                (*emit)(irn, env);
        }
        else {
-               ir_fprintf(F, "\t%35s ; %+F \n", " ", irn);
+               ir_fprintf(F, "\t%35s /* %+F (%+G) */\n", " ", irn, irn);
        }
 }
 
@@ -1114,13 +1452,12 @@ static void ia32_gen_block(ir_node *block, void *env) {
        if (! is_Block(block))
                return;
 
-       fprintf(((ia32_emit_env_t *)env)->out, "BLOCK_%ld:\n", get_irn_node_nr(block));
+       fprintf(((ia32_emit_env_t *)env)->out, BLOCK_PREFIX("%ld:\n"), get_irn_node_nr(block));
        sched_foreach(block, irn) {
                ia32_emit_node(irn, env);
        }
 }
 
-
 /**
  * Emits code for function start.
  */
@@ -1128,11 +1465,11 @@ static void ia32_emit_func_prolog(FILE *F, ir_graph *irg) {
        entity     *irg_ent  = get_irg_entity(irg);
        const char *irg_name = get_entity_name(irg_ent);
 
-//     fprintf(F, "\t.text\n");
+       fprintf(F, "\t.section\t.text\n");
        if (get_entity_visibility(irg_ent) == visibility_external_visible) {
-               fprintf(F, "global %s\n", irg_name);
+               fprintf(F, ".globl %s\n", irg_name);
        }
-//     fprintf(F, "\t.type\t%s, @function\n", irg_name);
+       fprintf(F, "\t.type\t%s, @function\n", irg_name);
        fprintf(F, "%s:\n", irg_name);
 }
 
@@ -1143,10 +1480,11 @@ static void ia32_emit_func_epilog(FILE *F, ir_graph *irg) {
        const char *irg_name = get_entity_name(get_irg_entity(irg));
 
        fprintf(F, "\tret\n");
-       //printf(F, "\t.size\t%s, .-%s\n\n", irg_name, irg_name);
+       fprintf(F, "\t.size\t%s, .-%s\n\n", irg_name, irg_name);
 }
 
 /**
+ * Block-walker:
  * Sets labels for control flow nodes (jump target)
  * TODO: Jump optimization
  */
@@ -1165,12 +1503,13 @@ static void ia32_gen_labels(ir_node *block, void *env) {
  */
 void ia32_gen_routine(FILE *F, ir_graph *irg, const ia32_code_gen_t *cg) {
        ia32_emit_env_t emit_env;
+       ir_node *block;
 
-       emit_env.mod      = firm_dbg_register("ir.be.codegen.ia32");
        emit_env.out      = F;
        emit_env.arch_env = cg->arch_env;
        emit_env.cg       = cg;
        emit_env.isa      = (ia32_isa_t *)cg->arch_env->isa;
+       FIRM_DBG_REGISTER(emit_env.mod, "firm.be.ia32.emitter");
 
        /* set the global arch_env (needed by print hooks) */
        arch_env = cg->arch_env;
@@ -1179,6 +1518,28 @@ void ia32_gen_routine(FILE *F, ir_graph *irg, const ia32_code_gen_t *cg) {
 
        ia32_emit_func_prolog(F, irg);
        irg_block_walk_graph(irg, ia32_gen_labels, NULL, &emit_env);
-       irg_walk_blkwise_graph(irg, NULL, ia32_gen_block, &emit_env);
+
+       if (cg->opt.extbb && cg->blk_sched) {
+               int i, n = ARR_LEN(cg->blk_sched);
+
+               for (i = 0; i < n;) {
+                       ir_node *next_bl;
+
+                       block   = cg->blk_sched[i];
+                       ++i;
+                       next_bl = i < n ? cg->blk_sched[i] : NULL;
+
+                       /* set here the link. the emitter expects to find the next block here */
+                       set_irn_link(block, next_bl);
+                       ia32_gen_block(block, &emit_env);
+               }
+       }
+       else {
+               /* "normal" block schedule: Note the get_next_block() returns the NUMBER of the block
+                  in the block schedule. As this number should NEVER be equal the next block,
+                  we does not need a clear block link here. */
+               irg_walk_blkwise_graph(irg, NULL, ia32_gen_block, &emit_env);
+       }
+
        ia32_emit_func_epilog(F, irg);
 }