Added phi handler
[libfirm] / ir / be / ia32 / ia32_emitter.c
index daa4981..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');
        }
 }
@@ -268,7 +288,7 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *env) {
                                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;
@@ -277,7 +297,10 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *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));
+                               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 {
                                const arch_register_t *in1 = get_in_reg(n, 2);
@@ -301,7 +324,7 @@ char *ia32_emit_binop(const ir_node *n, ia32_emit_env_t *env) {
                                                break;
                                }
 
-                               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), reg_name);
                        }
                        break;
                default:
@@ -367,10 +390,13 @@ 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;
                        default:
                                break;
@@ -480,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 */
 };
 
 /*
@@ -509,52 +535,90 @@ 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");
+
+       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;
 
-       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");
-
+       /* 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;
        }
 }
@@ -568,9 +632,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);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
        IA32_DO_EMIT;
-       finish_CondJmp(F, irn);
+       finish_CondJmp(F, irn, get_irn_mode(get_irn_n(irn, 2)));
 }
 
 /**
@@ -591,16 +655,19 @@ void emit_ia32_CondJmp_i(const ir_node *irn, ia32_emit_env_t *env) {
  * Emits code for conditional test and jump.
  */
 static void TestJmp_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];
-       const arch_register_t *in1 = get_in_reg(irn, 0);
-       const arch_register_t *in2 = get_in_reg(irn, 1);
+       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 ", arch_register_get_name(in1), arch_register_get_name(in2));
-       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "; %+F", irn);
+       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);
+       finish_CondJmp(F, irn, get_irn_mode(get_irn_n(irn, 0)));
 }
 
 /**
@@ -610,12 +677,7 @@ static void emit_ia32_TestJmp(const ir_node *irn, ia32_emit_env_t *env) {
        TestJmp_emitter(irn, env);
 }
 
-/**
- * Emits code for conditional test and jump with immediate.
- */
-static void emit_ia32_TestJmp_i(const ir_node *irn, ia32_emit_env_t *env) {
-       TestJmp_emitter(irn, env);
-}
+
 
 /*********************************************************
  *                 _ _       _
@@ -727,26 +789,26 @@ 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;
                }
                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;
                }
 
                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 ");
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default jump if out of range  */");
                IA32_DO_EMIT;
 
                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");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* get jump table entry as target */");
                        IA32_DO_EMIT;
 
                        fprintf(F, "\t.section\t.rodata\n");
@@ -762,11 +824,11 @@ void emit_ia32_SwitchJmp(const ir_node *irn, ia32_emit_env_t *emit_env) {
                        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");
+                                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default case */");
                                        IA32_DO_EMIT;
                                }
-                               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);
+                               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;
                        }
 
@@ -775,20 +837,20 @@ 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");
+                       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_esnprintf(env, cmd_buf, SNPRINTF_BUF_LEN, "cmpl %d, %1S", tbl.branches[i].value, irn);
-                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; case %d", i);
+                       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));
                }
 
                snprintf(cmd_buf, SNPRINTF_BUF_LEN, "jmp %s", get_cfop_target(tbl.defProj, buf));
-               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "; default case");
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* default case */");
                IA32_DO_EMIT;
        }
 
@@ -802,16 +864,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) {
+       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));
+       /* 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;
 }
 
-
-
 /****************************
  *                  _
  *                 (_)
@@ -866,18 +938,18 @@ static void emit_CopyB_prolog(FILE *F, int rem, int size) {
        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");
+                       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");
+                       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy remainder 3 */");
                        break;
        }
 
@@ -897,7 +969,7 @@ 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");
+       snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy */");
        IA32_DO_EMIT;
 }
 
@@ -915,7 +987,7 @@ 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");
+               snprintf(cmnt_buf, SNPRINTF_BUF_LEN, "/* memcopy unrolled */");
                IA32_DO_EMIT;
        }
 }
@@ -960,7 +1032,7 @@ 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);
+       lc_esnprintf(env, cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%+F, %+F) */", irn, src_mode, tgt_mode);
        IA32_DO_EMIT;
 }
 
@@ -1003,7 +1075,7 @@ 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;
 }
@@ -1020,11 +1092,11 @@ 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);
+               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;
@@ -1038,7 +1110,7 @@ void emit_be_SetSP(const ir_node *irn, ia32_emit_env_t *emit_env) {
        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);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F (restore SP) */", irn);
        IA32_DO_EMIT;
 }
 
@@ -1050,7 +1122,7 @@ void emit_be_Copy(const ir_node *irn, ia32_emit_env_t *emit_env) {
        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);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F */", irn);
        IA32_DO_EMIT;
 }
 
@@ -1062,7 +1134,7 @@ void emit_be_Perm(const ir_node *irn, ia32_emit_env_t *emit_env) {
        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);
+       lc_esnprintf(ia32_get_arg_env(), cmnt_buf, SNPRINTF_BUF_LEN, "/* %+F(%1A, %2A) */", irn, irn, irn);
        IA32_DO_EMIT;
 }
 
@@ -1134,7 +1206,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 */\n", " ", irn);
        }
 }
 
@@ -1148,13 +1220,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.
  */
@@ -1162,11 +1233,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, "\tsection .text\n");
+       fprintf(F, "\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);
 }
 
@@ -1176,11 +1247,12 @@ static void ia32_emit_func_prolog(FILE *F, ir_graph *irg) {
 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\n");
-       //printf(F, "\t.size\t%s, .-%s\n\n", irg_name, irg_name);
+       fprintf(F, "\tret\n");
+       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
  */
@@ -1194,13 +1266,40 @@ 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) {
        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;
@@ -1213,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);
 }