Added phi handler
[libfirm] / ir / be / ia32 / ia32_emitter.c
index f1099da..d391370 100644 (file)
@@ -21,7 +21,7 @@
 #include "irprog_t.h"
 #include "iredges_t.h"
 
-#include "../besched.h"
+#include "../besched_t.h"
 #include "../benode_t.h"
 
 #include "ia32_emitter.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;
 
+/* indicates whether blocks are scheduled or not
+   (this variable is set automatically) */
+static int have_block_sched       = 0;
+
 /*************************************************************
  *             _       _    __   _          _
  *            (_)     | |  / _| | |        | |
@@ -125,6 +132,9 @@ enum io_direction {
  */
 static const char *get_ia32_reg_name(ir_node *irn, int pos, enum io_direction in_out) {
        const arch_register_t *reg;
+       const char            *name;
+       static char           *buf = NULL;
+       int                    len;
 
        if (in_out == IN_REG) {
                reg = get_in_reg(irn, pos);
@@ -138,7 +148,18 @@ static const char *get_ia32_reg_name(ir_node *irn, int pos, enum io_direction in
                reg = get_out_reg(irn, pos);
        }
 
-       return arch_register_get_name(reg);
+       name = arch_register_get_name(reg);
+
+       if (buf) {
+               free(buf);
+       }
+
+       len = strlen(name) + 2;
+       buf = xcalloc(1, len);
+
+       snprintf(buf, len, "%%%s", name);
+
+       return buf;
 }
 
 /**
@@ -152,11 +173,11 @@ 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));
+       return lc_appendable_snadd(app, buf, strlen(buf));
 }
 
 /**
@@ -178,7 +199,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 +222,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');
        }
 }
@@ -234,11 +254,18 @@ const lc_arg_env_t *ia32_get_arg_env(void) {
 /**
  * Emits registers and/or address mode of a binary operation.
  */
-char *ia32_emit_binop(const ir_node *n) {
+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_Store8Bit(n)  || \
+       is_ia32_CondJmp(n)    || \
+       is_ia32_fCondJmp(n)   || \
+       is_ia32_SwitchJmp(n)))
 
        if (! buf) {
                buf = xcalloc(1, SNPRINTF_BUF_LEN);
@@ -255,38 +282,64 @@ char *ia32_emit_binop(const ir_node *n) {
                        else {
                                const arch_register_t *in1 = get_in_reg(n, 2);
                                const arch_register_t *in2 = get_in_reg(n, 3);
-                               const arch_register_t *out = get_ia32_n_res(n) > 0 ? get_out_reg(n, 0) : NULL;
+                               const arch_register_t *out = PRODUCES_RESULT(n) ? get_out_reg(n, 0) : NULL;
                                const arch_register_t *in;
 
                                in  = out ? (REGS_ARE_EQUAL(out, in2) ? in1 : in2) : in2;
                                out = out ? out : in1;
 
-                               snprintf(buf, SNPRINTF_BUF_LEN, "%s, %s", \
+                               snprintf(buf, SNPRINTF_BUF_LEN, "%%%s, %%%s", \
                                        arch_register_get_name(out), arch_register_get_name(in));
                        }
                        break;
                case ia32_AddrModeS:
-                       lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%4S, %s", n, ia32_emit_am(n));
+                       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), get_ia32_cnst(n));
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s,%s%s",
+                                       ia32_emit_am(n, env),
+                                       get_ia32_sc(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 {
-                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s, %3S", ia32_emit_am(n), n);
+                               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;
+                               }
+
+                               lc_esnprintf(ia32_get_arg_env(), buf, SNPRINTF_BUF_LEN, "%s, %%%s", ia32_emit_am(n, env), reg_name);
                        }
                        break;
                default:
                        assert(0 && "unsupported op type");
        }
 
+#undef PRODUCES_RESULT
+
        return buf;
 }
 
 /**
  * Emits registers and/or address mode of a unary operation.
  */
-char *ia32_emit_unop(const ir_node *n) {
+char *ia32_emit_unop(const ir_node *n, ia32_emit_env_t *env) {
        static char *buf = NULL;
 
        if (! buf) {
@@ -301,7 +354,7 @@ char *ia32_emit_unop(const ir_node *n) {
                        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));
+                       snprintf(buf, SNPRINTF_BUF_LEN, ia32_emit_am(n, env));
                        break;
                default:
                        assert(0 && "unsupported op type");
@@ -311,9 +364,9 @@ char *ia32_emit_unop(const ir_node *n) {
 }
 
 /**
- * Emits adress mode.
+ * Emits address mode.
  */
-char *ia32_emit_am(const ir_node *n) {
+char *ia32_emit_am(const ir_node *n, ia32_emit_env_t *env) {
        ia32_am_flavour_t am_flav    = get_ia32_am_flavour(n);
        int               had_output = 0;
        char             *s;
@@ -342,6 +395,9 @@ char *ia32_emit_am(const ir_node *n) {
                        case 16:
                                obstack_printf(obst, "WORD PTR ");
                                break;
+                       case 32:
+                               obstack_printf(obst, "DWORD PTR ");
+                               break;
                        default:
                                break;
                }
@@ -383,6 +439,15 @@ char *ia32_emit_am(const ir_node *n) {
 
 
 
+/**
+ * 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);
+}
+
+
+
 /**
  * Add a number to a prefix. This number will not be used a second time.
  */
@@ -404,6 +469,9 @@ 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)
+
 /*
  * coding of conditions
  */
@@ -438,22 +506,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 */
 };
 
 /*
@@ -467,67 +535,146 @@ 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 have_block_sched ? get_irn_link(block) : NULL;
+}
+
 /**
  * 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");
 
-       if (get_Proj_proj(proj) == 1) {
-               fprintf(F, "\tj%s %s\t\t\t/* cmp(a, b) == TRUE */\n",
+       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;
+               }
+       }
+
+       /* 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));
+                                       get_cfop_target(proj1, buf));
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* cmp(a, b) == TRUE */");
        }
        else  {
-               fprintf(F, "\tjn%s %s\t\t\t/* cmp(a, b) == FALSE */\n",
-                                       get_cmp_suffix(get_ia32_pncode(irn), !mode_is_signed(get_irn_mode(get_irn_n(irn, 0)))),
-                                       get_cfop_target(proj, buf));
+               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;
 
-       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");
-               fprintf(F, "\tjmp %s\t\t\t/* otherwise */\n", get_cfop_target(proj, buf));
+       /* 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;
        }
 }
 
 /**
- * Emits code for conditional jump with two variables.
+ * Emits code for conditional jump.
  */
-static void emit_ia32_CondJmp(const ir_node *irn, emit_env_t *env) {
+static void CondJmp_emitter(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];
 
-       lc_efprintf(ia32_get_arg_env(), F, "\tcmp %s\t\t\t/* CondJmp(%+F, %+F) */\n",
-               ia32_emit_binop(irn), get_irn_n(irn, 0), get_irn_n(irn, 1));
-       finish_CondJmp(F, irn);
+       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, get_irn_mode(get_irn_n(irn, 2)));
+}
+
+/**
+ * Emits code for conditional jump with two variables.
+ */
+static void emit_ia32_CondJmp(const ir_node *irn, ia32_emit_env_t *env) {
+       CondJmp_emitter(irn, env);
 }
 
 /**
  * Emits code for conditional jump with immediate.
  */
-void emit_ia32_CondJmp_i(const ir_node *irn, emit_env_t *env) {
-       FILE *F = env->out;
+void emit_ia32_CondJmp_i(const ir_node *irn, ia32_emit_env_t *env) {
+       CondJmp_emitter(irn, env);
+}
 
-       lc_efprintf(ia32_get_arg_env(), F, "\tcmp %s\t\t\t/* CondJmp_i(%+F) */\n",
-               ia32_emit_binop(irn), get_irn_n(irn, 0));
-       finish_CondJmp(F, irn);
+/**
+ * Emits code for conditional test and jump.
+ */
+static void TestJmp_emitter(const ir_node *irn, ia32_emit_env_t *env) {
+       FILE       *F   = env->out;
+       const char *op1 = arch_register_get_name(get_in_reg(irn, 0));
+       const char *op2 = get_ia32_cnst(irn);
+       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, get_ia32_cnst(irn) ? " " : " %", op2);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
+       IA32_DO_EMIT;
+       finish_CondJmp(F, irn, get_irn_mode(get_irn_n(irn, 0)));
+}
+
+/**
+ * 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);
 }
 
 
@@ -577,7 +724,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, emit_env_t *emit_env) {
+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;
@@ -586,6 +733,7 @@ void emit_ia32_SwitchJmp(const ir_node *irn, emit_env_t *emit_env) {
        const ir_edge_t    *edge;
        const lc_arg_env_t *env = ia32_get_arg_env();
        FILE               *F   = emit_env->out;
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        /* fill the table structure */
        tbl.label        = xmalloc(SNPRINTF_BUF_LEN);
@@ -639,48 +787,71 @@ void emit_ia32_SwitchJmp(const ir_node *irn, emit_env_t *emit_env) {
        if (do_jmp_tbl) {
                /* emit the table */
                if (tbl.min_value != 0) {
-                       lc_efprintf(env, F, "\tcmpl %lu, -%d(%1S)\t\t/* first switch value is not 0 */\n",
+                       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 */");
+
+                       IA32_DO_EMIT;
                }
                else {
-                       lc_efprintf(env, F, "\tcmpl %lu, %1S\t\t\t/* compare for switch */\n", interval, irn);
+                       lc_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "cmpl %lu, %1S", interval, irn);
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* compare for switch */");
+
+                       IA32_DO_EMIT;
                }
 
-               fprintf(F, "\tja %s\t\t\t/* default jump if out of range  */\n", get_cfop_target(tbl.defProj, buf));
+               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;
 
                if (tbl.num_branches > 1) {
                        /* create table */
 
-                       lc_efprintf(env, F, "\tjmp [%1S*4+%s]\t\t/* get jump table entry as target */\n", irn, tbl.label);
+                       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;
 
-                       fprintf(F, "\t.section\t.rodata\t\t/* start jump table */\n");
+                       fprintf(F, "\t.section\t.rodata\n");
                        fprintf(F, "\t.align 4\n");
 
                        fprintf(F, "%s:\n", tbl.label);
-                       fprintf(F, "\t.long %s\t\t\t/* case %d */\n", get_cfop_target(tbl.branches[0].target, buf), tbl.branches[0].value);
+
+                       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;
 
                        last_value = tbl.branches[0].value;
                        for (i = 1; i < tbl.num_branches; ++i) {
                                while (++last_value < tbl.branches[i].value) {
-                                       fprintf(F, "\t.long %s\t\t/* default case */\n", get_cfop_target(tbl.defProj, buf));
+                                       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;
                                }
-                               fprintf(F, "\t.long %s\t\t\t/* case %d */\n", get_cfop_target(tbl.branches[i].target, buf), last_value);
+                               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;
                        }
 
-                       fprintf(F, "\t.text\t\t\t\t/* end of jump table */\n");
+                       fprintf(F, "\t.text");
                }
                else {
                        /* one jump is enough */
-                       fprintf(F, "\tjmp %s\t\t/* only one case given */\n", get_cfop_target(tbl.branches[0].target, buf));
+                       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;
                }
        }
        else { // no jump table
                for (i = 0; i < tbl.num_branches; ++i) {
-                       lc_efprintf(env, F, "\tcmpl %d, %1S\t\t\t/* case %d */\n", tbl.branches[i].value, irn, 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;
                        fprintf(F, "\tje %s\n", get_cfop_target(tbl.branches[i].target, buf));
                }
 
-               fprintf(F, "\tjmp %s\t\t\t/* default case */\n", get_cfop_target(tbl.defProj, 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;
        }
 
        if (tbl.label)
@@ -692,14 +863,26 @@ void emit_ia32_SwitchJmp(const ir_node *irn, emit_env_t *emit_env) {
 /**
  * Emits code for a unconditional jump.
  */
-void emit_Jmp(const ir_node *irn, emit_env_t *env) {
+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];
 
-       char buf[SNPRINTF_BUF_LEN];
-       ir_fprintf(F, "\tjmp %s\t\t\t/* Jmp(%+F) */\n", get_cfop_target(irn, buf), get_irn_link(irn));
-}
-
+       /* 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;
+}
 
 /****************************
  *                  _
@@ -715,7 +898,7 @@ void emit_Jmp(const ir_node *irn, emit_env_t *env) {
 /**
  * Emits code for a proj -> node
  */
-void emit_Proj(const ir_node *irn, emit_env_t *env) {
+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) {
@@ -744,50 +927,68 @@ void emit_Proj(const ir_node *irn, emit_env_t *env) {
  * Emit movsb/w instructions to make mov count divideable by 4
  */
 static void emit_CopyB_prolog(FILE *F, int rem, int size) {
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
+
        fprintf(F, "\t/* memcopy %d bytes*/\n", size);
-       fprintf(F, "\tcld\t\t\t\t/* copy direction forward*/\n");
+
+       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "cld");
+       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* copy direction forward*/");
+       IA32_DO_EMIT;
 
        switch(rem) {
                case 1:
-                       fprintf(F, "\tmovsb\t\t\t\t/* memcopy remainder 1 */\n");
+                       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsb");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 1 */");
                        break;
                case 2:
-                       fprintf(F, "\tmovsw\t\t\t\t/* memcopy remainder 2 */\n");
+                       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsw");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 2 */");
                        break;
                case 3:
-                       fprintf(F, "\tmovsb\t\t\t\t/* memcopy remainder 3 */\n");
-                       fprintf(F, "\tmovsw\t\t\t\t/* memcopy remainder 3 */\n");
+                       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsb");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 3 */");
+                       IA32_DO_EMIT;
+                       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsw");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 3 */");
                        break;
        }
+
+       IA32_DO_EMIT;
 }
 
 /**
  * Emit rep movsd instruction for memcopy.
  */
-void emit_ia32_CopyB(const ir_node *irn, emit_env_t *emit_env) {
+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);
        int     size = get_tarval_long(get_ia32_Immop_tarval(get_irn_n(irn, 2)));
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        emit_CopyB_prolog(F, rem, size);
 
-       fprintf(F, "\trep movsd\t\t\t\t/* memcopy */\n");
+       snprintf(cmd_buf, SNPRINTF_BUF_LEN, "rep movsd");
+       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy */");
+       IA32_DO_EMIT;
 }
 
 /**
  * Emits unrolled memcopy.
  */
-void emit_ia32_CopyB_i(const ir_node *irn, emit_env_t *emit_env) {
+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;
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        emit_CopyB_prolog(F, size & 0x3, size);
 
        size >>= 2;
        while (size--) {
-               fprintf(F, "\tmovsd\t\t\t\t/* memcopy unrolled */\n");
+               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "movsd");
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy unrolled */");
+               IA32_DO_EMIT;
        }
 }
 
@@ -806,11 +1007,12 @@ void emit_ia32_CopyB_i(const ir_node *irn, 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, emit_env_t *emit_env) {
+static void emit_ia32_Conv(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];
        ir_mode *src_mode, *tgt_mode;
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        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);
@@ -823,24 +1025,26 @@ static void emit_ia32_Conv(const ir_node *irn, emit_env_t *emit_env) {
                        lc_esnprintf(env, buf, sizeof(buf), "%1D, %3S", irn, irn);
                        break;
                case ia32_AddrModeS:
-                       lc_esnprintf(env, buf, sizeof(buf), "%1D, %s", irn, ia32_emit_am(irn));
+                       lc_esnprintf(env, buf, sizeof(buf), "%1D, %s", irn, ia32_emit_am(irn, emit_env));
                        break;
                default:
                        assert(0 && "unsupported op type for Conv");
        }
 
-       lc_efprintf(env, F, "\tcvt%s2%s %s\t\t\t/* %+F(%+F, %+F) */\n", from, to, buf, irn, src_mode, tgt_mode);
+       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;
 }
 
-void emit_ia32_Conv_I2FP(const ir_node *irn, emit_env_t *emit_env) {
+void emit_ia32_Conv_I2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        emit_ia32_Conv(irn, emit_env);
 }
 
-void emit_ia32_Conv_FP2I(const ir_node *irn, emit_env_t *emit_env) {
+void emit_ia32_Conv_FP2I(const ir_node *irn, ia32_emit_env_t *emit_env) {
        emit_ia32_Conv(irn, emit_env);
 }
 
-void emit_ia32_Conv_FP2FP(const ir_node *irn, emit_env_t *emit_env) {
+void emit_ia32_Conv_FP2FP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        emit_ia32_Conv(irn, emit_env);
 }
 
@@ -859,64 +1063,79 @@ void emit_ia32_Conv_FP2FP(const ir_node *irn, emit_env_t *emit_env) {
 /**
  * Emits a backend call
  */
-void emit_be_Call(const ir_node *irn, emit_env_t *emit_env) {
+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);
-
-       fprintf(F, "\tcall ");
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        if (ent) {
-               fprintf(F, "%s", get_entity_name(ent));
+               snprintf(cmd_buf, SNPRINTF_BUF_LEN, "call %s", get_entity_name(ent));
        }
        else {
-               lc_efprintf(ia32_get_arg_env(), F, "%1D", get_irn_n(irn, be_pos_Call_ptr));
+               lc_esnprintf(ia32_get_arg_env(), cmd_buf, SNPRINTF_BUF_LEN, "%1D", get_irn_n(irn, be_pos_Call_ptr));
        }
 
-       ir_fprintf(F, "\t\t\t/* %+F (be_Call) */\n", irn);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F (be_Call) */", irn);
+
+       IA32_DO_EMIT;
 }
 
 /**
  * Emits code to increase stack pointer.
  */
-void emit_be_IncSP(const ir_node *irn, emit_env_t *emit_env) {
+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);
+       char cmd_buf[SNPRINTF_BUF_LEN], cmnt_buf[SNPRINTF_BUF_LEN];
 
        if (offs) {
-               lc_efprintf(ia32_get_arg_env(), F, "\tadd %1S,%s%u\t\t\t/* %+F (IncSP) */\n", irn,
-                       (dir == be_stack_dir_along) ? " -" : " ", offs, irn);
+               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);
        }
        else {
-               fprintf(F, "\t\t\t\t\t/* omitted IncSP with 0 */\n");
+               snprintf(cmd_buf, SNPRINTF_BUF_LEN, " ");
+               lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* omitted %+F (IncSP) with 0 */", irn);
        }
+
+       IA32_DO_EMIT;
 }
 
 /**
  * Emits code to set stack pointer.
  */
-void emit_be_SetSP(const ir_node *irn, emit_env_t *emit_env) {
+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_efprintf(ia32_get_arg_env(), F, "\tmov %1D, %3S\t\t\t/* restore SP */\n", irn, irn);
+       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;
 }
 
 /**
  * Emits code for Copy.
  */
-void emit_be_Copy(const ir_node *irn, emit_env_t *emit_env) {
+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_efprintf(ia32_get_arg_env(), F, "\tmov %1D, %1S\t\t\t/* %+F */\n", irn, irn, irn);
+       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;
 }
 
 /**
  * Emits code for exchange.
  */
-void emit_be_Perm(const ir_node *irn, emit_env_t *emit_env) {
+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_efprintf(ia32_get_arg_env(), F, "\txchg %1S, %2S\t\t\t/* %+F(%1A, %2A) */\n", irn, irn, irn);
+       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;
 }
 
 /***********************************************************************************
@@ -947,6 +1166,7 @@ static void ia32_register_emitters(void) {
 
        /* other ia32 emitter functions */
        IA32_EMIT(CondJmp);
+       IA32_EMIT(TestJmp);
        IA32_EMIT(SwitchJmp);
        IA32_EMIT(CopyB);
        IA32_EMIT(CopyB_i);
@@ -974,7 +1194,7 @@ static void ia32_register_emitters(void) {
  * Emits code for a node.
  */
 static void ia32_emit_node(const ir_node *irn, void *env) {
-       emit_env_t        *emit_env = env;
+       ia32_emit_env_t        *emit_env = env;
        firm_dbg_module_t *mod      = emit_env->mod;
        FILE              *F        = emit_env->out;
        ir_op             *op       = get_irn_op(irn);
@@ -985,8 +1205,9 @@ static void ia32_emit_node(const ir_node *irn, void *env) {
                void (*emit)(const ir_node *, void *) = (void (*)(const ir_node *, void *))op->ops.generic;
                (*emit)(irn, env);
        }
-
-       ir_fprintf(F, "\t\t\t\t\t/* %+F */\n", irn);
+       else {
+               ir_fprintf(F, "\t%35s /* %+F */\n", " ", irn);
+       }
 }
 
 /**
@@ -999,21 +1220,23 @@ static void ia32_gen_block(ir_node *block, void *env) {
        if (! is_Block(block))
                return;
 
-       fprintf(((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.
  */
 static void ia32_emit_func_prolog(FILE *F, ir_graph *irg) {
-       const char *irg_name = get_entity_name(get_irg_entity(irg));
+       entity     *irg_ent  = get_irg_entity(irg);
+       const char *irg_name = get_entity_name(irg_ent);
 
        fprintf(F, "\t.text\n");
-       fprintf(F, ".globl %s\n", irg_name);
+       if (get_entity_visibility(irg_ent) == visibility_external_visible) {
+               fprintf(F, ".globl %s\n", irg_name);
+       }
        fprintf(F, "\t.type\t%s, @function\n", irg_name);
        fprintf(F, "%s:\n", irg_name);
 }
@@ -1029,6 +1252,7 @@ static void ia32_emit_func_epilog(FILE *F, ir_graph *irg) {
 }
 
 /**
+ * Block-walker:
  * Sets labels for control flow nodes (jump target)
  * TODO: Jump optimization
  */
@@ -1042,16 +1266,44 @@ static void ia32_gen_labels(ir_node *block, void *env) {
        }
 }
 
+typedef struct {
+       ir_node *start;
+       ir_node *end;
+} anchor;
+
+/**
+ * Ext-Block walker: create a block schedule
+ */
+static void create_block_list(ir_extblk *blk, void *env) {
+       anchor *list = env;
+       int i, n;
+
+       for (i = 0, n = get_extbb_n_blocks(blk); i < n; ++i) {
+               ir_node *block = get_extbb_block(blk, i);
+
+               set_irn_link(block, NULL);
+               if (list->start)
+                       set_irn_link(list->end, block);
+               else
+                       list->start = block;
+
+               list->end = block;
+       }
+}
+
 /**
  * Main driver. Emits the code for one routine.
  */
 void ia32_gen_routine(FILE *F, ir_graph *irg, const ia32_code_gen_t *cg) {
-       emit_env_t emit_env;
+       ia32_emit_env_t emit_env;
+       anchor list;
+       ir_node *block;
 
-       emit_env.mod      = firm_dbg_register("ir.be.codegen.ia32");
+       emit_env.mod      = firm_dbg_register("firm.be.ia32.emitter");
        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;
 
        /* set the global arch_env (needed by print hooks) */
        arch_env = cg->arch_env;
@@ -1060,6 +1312,26 @@ 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) {
+               /* schedule extended basic blocks */
+
+               compute_extbb(irg);
+
+               list.start = NULL;
+               list.end   = NULL;
+               irg_extblock_walk_graph(irg, NULL, create_block_list, &list);
+
+               have_block_sched = 1;
+               for (block = list.start; block; block = get_irn_link(block))
+                       ia32_gen_block(block, &emit_env);
+       }
+       else {
+               /* "normal" block schedule */
+
+               have_block_sched = 0;
+               irg_walk_blkwise_graph(irg, NULL, ia32_gen_block, &emit_env);
+       }
+
        ia32_emit_func_epilog(F, irg);
 }