be: get rid of arch_irn_classify callback
[libfirm] / ir / be / ia32 / ia32_x87.c
index bd3a7c9..2be20cf 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 1995-2008 University of Karlsruhe.  All right reserved.
+ * Copyright (C) 1995-2010 University of Karlsruhe.  All right reserved.
  *
  * This file is part of libFirm.
  *
@@ -22,7 +22,6 @@
  * @brief       This file implements the x87 support and virtual to stack
  *              register translation for the ia32 backend.
  * @author      Michael Beck
- * @version     $Id$
  */
 #include "config.h"
 
@@ -43,9 +42,9 @@
 #include "debug.h"
 #include "error.h"
 
-#include "../belive_t.h"
-#include "../besched.h"
-#include "../benode.h"
+#include "belive_t.h"
+#include "besched.h"
+#include "benode.h"
 #include "bearch_ia32_t.h"
 #include "ia32_new_nodes.h"
 #include "gen_ia32_new_nodes.h"
@@ -154,7 +153,7 @@ struct x87_simulator {
 static int x87_get_depth(const x87_state *state)
 {
        return state->depth;
-}  /* x87_get_depth */
+}
 
 /**
  * Return the virtual register index at st(pos).
@@ -168,7 +167,7 @@ static int x87_get_st_reg(const x87_state *state, int pos)
 {
        assert(pos < state->depth);
        return state->st[MASK_TOS(state->tos + pos)].reg_idx;
-}  /* x87_get_st_reg */
+}
 
 #ifdef DEBUG_libfirm
 /**
@@ -183,7 +182,7 @@ static ir_node *x87_get_st_node(const x87_state *state, int pos)
 {
        assert(pos < state->depth);
        return state->st[MASK_TOS(state->tos + pos)].node;
-}  /* x87_get_st_node */
+}
 
 /**
  * Dump the stack for debugging.
@@ -199,7 +198,7 @@ static void x87_dump_stack(const x87_state *state)
                    x87_get_st_node(state, i)));
        }
        DB((dbg, LEVEL_2, "<-- TOS\n"));
-}  /* x87_dump_stack */
+}
 #endif /* DEBUG_libfirm */
 
 /**
@@ -217,8 +216,8 @@ static void x87_set_st(x87_state *state, int reg_idx, ir_node *node, int pos)
        state->st[MASK_TOS(state->tos + pos)].node    = node;
 
        DB((dbg, LEVEL_2, "After SET_REG: "));
-       DEBUG_ONLY(x87_dump_stack(state));
-}  /* x87_set_st */
+       DEBUG_ONLY(x87_dump_stack(state);)
+}
 
 /**
  * Set the tos virtual register.
@@ -230,7 +229,7 @@ static void x87_set_st(x87_state *state, int reg_idx, ir_node *node, int pos)
 static void x87_set_tos(x87_state *state, int reg_idx, ir_node *node)
 {
        x87_set_st(state, reg_idx, node, 0);
-}  /* x87_set_tos */
+}
 
 /**
  * Swap st(0) with st(pos).
@@ -248,8 +247,8 @@ static void x87_fxch(x87_state *state, int pos)
        state->st[MASK_TOS(state->tos)] = entry;
 
        DB((dbg, LEVEL_2, "After FXCH: "));
-       DEBUG_ONLY(x87_dump_stack(state));
-}  /* x87_fxch */
+       DEBUG_ONLY(x87_dump_stack(state);)
+}
 
 /**
  * Convert a virtual register to the stack index.
@@ -268,7 +267,7 @@ static int x87_on_stack(const x87_state *state, int reg_idx)
                if (state->st[MASK_TOS(tos + i)].reg_idx == reg_idx)
                        return i;
        return -1;
-}  /* x87_on_stack */
+}
 
 /**
  * Push a virtual Register onto the stack, double pushed allowed.
@@ -286,8 +285,8 @@ static void x87_push_dbl(x87_state *state, int reg_idx, ir_node *node)
        state->st[state->tos].reg_idx = reg_idx;
        state->st[state->tos].node    = node;
 
-       DB((dbg, LEVEL_2, "After PUSH: ")); DEBUG_ONLY(x87_dump_stack(state));
-}  /* x87_push_dbl */
+       DB((dbg, LEVEL_2, "After PUSH: ")); DEBUG_ONLY(x87_dump_stack(state);)
+}
 
 /**
  * Push a virtual Register onto the stack, double pushes are NOT allowed.
@@ -302,7 +301,7 @@ static void x87_push(x87_state *state, int reg_idx, ir_node *node)
        assert(x87_on_stack(state, reg_idx) == -1 && "double push");
 
        x87_push_dbl(state, reg_idx, node);
-}  /* x87_push */
+}
 
 /**
  * Pop a virtual Register from the stack.
@@ -316,8 +315,8 @@ static void x87_pop(x87_state *state)
        --state->depth;
        state->tos = MASK_TOS(state->tos + 1);
 
-       DB((dbg, LEVEL_2, "After POP: ")); DEBUG_ONLY(x87_dump_stack(state));
-}  /* x87_pop */
+       DB((dbg, LEVEL_2, "After POP: ")); DEBUG_ONLY(x87_dump_stack(state);)
+}
 
 /**
  * Empty the fpu stack
@@ -343,7 +342,7 @@ static blk_state *x87_get_bl_state(x87_simulator *sim, ir_node *block)
        pmap_entry *entry = pmap_find(sim->blk_states, block);
 
        if (! entry) {
-               blk_state *bl_state = obstack_alloc(&sim->obst, sizeof(*bl_state));
+               blk_state *bl_state = OALLOC(&sim->obst, blk_state);
                bl_state->begin = NULL;
                bl_state->end   = NULL;
 
@@ -352,7 +351,7 @@ static blk_state *x87_get_bl_state(x87_simulator *sim, ir_node *block)
        }
 
        return PTR_TO_BLKSTATE(entry->value);
-}  /* x87_get_bl_state */
+}
 
 /**
  * Creates a new x87 state.
@@ -363,11 +362,11 @@ static blk_state *x87_get_bl_state(x87_simulator *sim, ir_node *block)
  */
 static x87_state *x87_alloc_state(x87_simulator *sim)
 {
-       x87_state *res = obstack_alloc(&sim->obst, sizeof(*res));
+       x87_state *res = OALLOC(&sim->obst, x87_state);
 
        res->sim = sim;
        return res;
-}  /* x87_alloc_state */
+}
 
 /**
  * Clone a x87 state.
@@ -383,7 +382,7 @@ static x87_state *x87_clone_state(x87_simulator *sim, const x87_state *src)
 
        *res = *src;
        return res;
-}  /* x87_clone_state */
+}
 
 /**
  * Patch a virtual instruction into a x87 one and return
@@ -416,7 +415,7 @@ static ir_node *x87_patch_insn(ir_node *n, ir_op *op)
        } else if (mode_is_float(mode))
                set_irn_mode(n, ia32_reg_classes[CLASS_ia32_st].mode);
        return res;
-}  /* x87_patch_insn */
+}
 
 /**
  * Returns the first Proj of a mode_T node having a given mode.
@@ -438,7 +437,7 @@ static ir_node *get_irn_Proj_for_mode(ir_node *n, ir_mode *m)
        }
 
        return NULL;
-}  /* get_irn_Proj_for_mode */
+}
 
 /**
  * Wrap the arch_* function here so we can check for errors.
@@ -449,16 +448,16 @@ static inline const arch_register_t *x87_get_irn_register(const ir_node *irn)
 
        assert(res->reg_class == &ia32_reg_classes[CLASS_ia32_vfp]);
        return res;
-}  /* x87_get_irn_register */
+}
 
 static inline const arch_register_t *x87_irn_get_register(const ir_node *irn,
                                                           int pos)
 {
-       const arch_register_t *res = arch_irn_get_register(irn, pos);
+       const arch_register_t *res = arch_get_irn_register_out(irn, pos);
 
        assert(res->reg_class == &ia32_reg_classes[CLASS_ia32_vfp]);
        return res;
-}  /* x87_irn_get_register */
+}
 
 static inline const arch_register_t *get_st_reg(int index)
 {
@@ -493,7 +492,7 @@ static ir_node *x87_fxch_shuffle(x87_state *state, int pos, ir_node *block)
 
        x87_fxch(state, pos);
        return fxch;
-}  /* x87_fxch_shuffle */
+}
 
 /**
  * Calculate the necessary permutations to reach dst_state.
@@ -583,9 +582,9 @@ static x87_state *x87_shuffle(x87_simulator *sim, ir_node *block,
 
        /* Hmm: permutation needed */
        DB((dbg, LEVEL_2, "\n%+F needs permutation: from\n", block));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
        DB((dbg, LEVEL_2, "                  to\n"));
-       DEBUG_ONLY(x87_dump_stack(dst_state));
+       DEBUG_ONLY(x87_dump_stack(dst_state);)
 
 
 #ifdef DEBUG_libfirm
@@ -634,7 +633,7 @@ static x87_state *x87_shuffle(x87_simulator *sim, ir_node *block,
                }
        }
        return state;
-}  /* x87_shuffle */
+}
 
 /**
  * Create a fxch node before another node.
@@ -663,7 +662,7 @@ static ir_node *x87_create_fxch(x87_state *state, ir_node *n, int pos)
        sched_add_before(n, fxch);
        DB((dbg, LEVEL_1, "<<< %s %s, %s\n", get_irn_opname(fxch), attr->x87[0]->name, attr->x87[2]->name));
        return fxch;
-}  /* x87_create_fxch */
+}
 
 /**
  * Create a fpush before node n.
@@ -690,7 +689,7 @@ static void x87_create_fpush(x87_state *state, ir_node *n, int pos, int op_idx)
        sched_add_before(n, fpush);
 
        DB((dbg, LEVEL_1, "<<< %s %s, %s\n", get_irn_opname(fpush), attr->x87[0]->name, attr->x87[2]->name));
-}  /* x87_create_fpush */
+}
 
 /**
  * Create a fpop before node n.
@@ -707,7 +706,7 @@ static ir_node *x87_create_fpop(x87_state *state, ir_node *n, int num)
        ia32_x87_attr_t *attr;
 
        assert(num > 0);
-       while (num > 0) {
+       do {
                x87_pop(state);
                if (ia32_cg_config.use_ffreep)
                        fpop = new_bd_ia32_ffreep(NULL, get_nodes_block(n));
@@ -721,11 +720,9 @@ static ir_node *x87_create_fpop(x87_state *state, ir_node *n, int num)
                keep_alive(fpop);
                sched_add_before(n, fpop);
                DB((dbg, LEVEL_1, "<<< %s %s\n", get_irn_opname(fpop), attr->x87[0]->name));
-
-               --num;
-       }
+       } while (--num > 0);
        return fpop;
-}  /* x87_create_fpop */
+}
 
 /* --------------------------------- liveness ------------------------------------------ */
 
@@ -771,7 +768,7 @@ static vfp_liveness vfp_liveness_transfer(ir_node *irn, vfp_liveness live)
                }
        }
        return live;
-}  /* vfp_liveness_transfer */
+}
 
 /**
  * Put all live virtual registers at the end of a block into a bitset.
@@ -800,7 +797,7 @@ static vfp_liveness vfp_liveness_end_of_block(x87_simulator *sim, const ir_node
        }
 
        return live;
-}  /* vfp_liveness_end_of_block */
+}
 
 /** get the register mask from an arch_register */
 #define REGMASK(reg)    (1 << (arch_register_get_index(reg)))
@@ -820,7 +817,7 @@ static unsigned vfp_live_args_after(x87_simulator *sim, const ir_node *pos, unsi
 
        assert(idx < sim->n_idx);
        return sim->live[idx] & ~kill;
-}  /* vfp_live_args_after */
+}
 
 /**
  * Calculate the liveness for a whole block and cache it.
@@ -848,7 +845,7 @@ static void update_liveness(x87_simulator *sim, ir_node *block)
        }
        idx = get_irn_idx(block);
        sim->live[idx] = live;
-}  /* update_liveness */
+}
 
 /**
  * Returns true if a register is live in a set.
@@ -875,7 +872,7 @@ static void vfp_dump_live(vfp_liveness live)
                }
        }
        DB((dbg, LEVEL_2, "\n"));
-}  /* vfp_dump_live */
+}
 #endif /* DEBUG_libfirm */
 
 /* --------------------------------- simulators ---------------------------------------- */
@@ -912,9 +909,9 @@ static int sim_binop(x87_state *state, ir_node *n, const exchange_tmpl *tmpl)
        DB((dbg, LEVEL_1, ">>> %+F %s, %s -> %s\n", n,
                arch_register_get_name(op1_reg), arch_register_get_name(op2_reg),
                arch_register_get_name(out)));
-       DEBUG_ONLY(vfp_dump_live(live));
+       DEBUG_ONLY(vfp_dump_live(live);)
        DB((dbg, LEVEL_1, "Stack before: "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
 
        op1_idx = x87_on_stack(state, reg_index_1);
        assert(op1_idx >= 0);
@@ -1052,7 +1049,7 @@ static int sim_binop(x87_state *state, ir_node *n, const exchange_tmpl *tmpl)
        }
 
        return NO_NODE_ADDED;
-}  /* sim_binop */
+}
 
 /**
  * Simulate a virtual Unop.
@@ -1073,7 +1070,7 @@ static int sim_unop(x87_state *state, ir_node *n, ir_op *op)
        unsigned live = vfp_live_args_after(sim, n, REGMASK(out));
 
        DB((dbg, LEVEL_1, ">>> %+F -> %s\n", n, out->name));
-       DEBUG_ONLY(vfp_dump_live(live));
+       DEBUG_ONLY(vfp_dump_live(live);)
 
        op1_idx = x87_on_stack(state, arch_register_get_index(op1));
 
@@ -1097,7 +1094,7 @@ static int sim_unop(x87_state *state, ir_node *n, ir_op *op)
        DB((dbg, LEVEL_1, "<<< %s -> %s\n", get_irn_opname(n), out->name));
 
        return NO_NODE_ADDED;
-}  /* sim_unop */
+}
 
 /**
  * Simulate a virtual Load instruction.
@@ -1121,7 +1118,7 @@ static int sim_load(x87_state *state, ir_node *n, ir_op *op, int res_pos)
        DB((dbg, LEVEL_1, "<<< %s -> %s\n", get_irn_opname(n), arch_register_get_name(out)));
 
        return NO_NODE_ADDED;
-}  /* sim_load */
+}
 
 /**
  * Rewire all users of @p old_val to @new_val iff they are scheduled after @p store.
@@ -1150,7 +1147,7 @@ static void collect_and_rewire_users(ir_node *store, ir_node *old_val, ir_node *
                        }
                }
        }
-}  /* collect_and_rewire_users */
+}
 
 /**
  * Simulate a virtual Store.
@@ -1197,14 +1194,15 @@ static int sim_store(x87_state *state, ir_node *n, ir_op *op, ir_op *op_p)
                                x87_patch_insn(n, op_p);
                        } else {
                                ir_node  *vfld, *mem, *block, *rproj, *mproj;
-                               ir_graph *irg = get_irn_irg(n);
+                               ir_graph *irg   = get_irn_irg(n);
+                               ir_node  *nomem = get_irg_no_mem(irg);
 
                                /* stack full here: need fstp + load */
                                x87_pop(state);
                                x87_patch_insn(n, op_p);
 
                                block = get_nodes_block(n);
-                               vfld  = new_bd_ia32_vfld(NULL, block, get_irn_n(n, 0), get_irn_n(n, 1), new_r_NoMem(irg), get_ia32_ls_mode(n));
+                               vfld  = new_bd_ia32_vfld(NULL, block, get_irn_n(n, 0), get_irn_n(n, 1), nomem, get_ia32_ls_mode(n));
 
                                /* copy all attributes */
                                set_ia32_frame_ent(vfld, get_ia32_frame_ent(n));
@@ -1224,7 +1222,7 @@ static int sim_store(x87_state *state, ir_node *n, ir_op *op, ir_op *op_p)
                                arch_set_irn_register(rproj, op2);
 
                                /* reroute all former users of the store memory to the load memory */
-                               edges_reroute(mem, mproj, irg);
+                               edges_reroute(mem, mproj);
                                /* set the memory input of the load to the store memory */
                                set_irn_n(vfld, n_ia32_vfld_mem, mem);
 
@@ -1258,7 +1256,7 @@ static int sim_store(x87_state *state, ir_node *n, ir_op *op, ir_op *op_p)
        DB((dbg, LEVEL_1, "<<< %s %s ->\n", get_irn_opname(n), arch_register_get_name(op2)));
 
        return insn;
-}  /* sim_store */
+}
 
 #define _GEN_BINOP(op, rev) \
 static int sim_##op(x87_state *state, ir_node *n) { \
@@ -1338,7 +1336,7 @@ static int sim_fisttp(x87_state *state, ir_node *n)
        DB((dbg, LEVEL_1, "<<< %s %s ->\n", get_irn_opname(n), arch_register_get_name(op2)));
 
        return NO_NODE_ADDED;
-}  /* sim_fisttp */
+}
 
 /**
  * Simulate a virtual FtstFnstsw.
@@ -1359,9 +1357,9 @@ static int sim_FtstFnstsw(x87_state *state, ir_node *n)
        unsigned               live        = vfp_live_args_after(sim, n, 0);
 
        DB((dbg, LEVEL_1, ">>> %+F %s\n", n, arch_register_get_name(reg1)));
-       DEBUG_ONLY(vfp_dump_live(live));
+       DEBUG_ONLY(vfp_dump_live(live);)
        DB((dbg, LEVEL_1, "Stack before: "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
        assert(op1_idx >= 0);
 
        if (op1_idx != 0) {
@@ -1381,7 +1379,7 @@ static int sim_FtstFnstsw(x87_state *state, ir_node *n)
                x87_create_fpop(state, sched_next(n), 1);
 
        return NO_NODE_ADDED;
-}  /* sim_FtstFnstsw */
+}
 
 /**
  * Simulate a Fucom
@@ -1411,9 +1409,9 @@ static int sim_Fucom(x87_state *state, ir_node *n)
 
        DB((dbg, LEVEL_1, ">>> %+F %s, %s\n", n,
                arch_register_get_name(op1), arch_register_get_name(op2)));
-       DEBUG_ONLY(vfp_dump_live(live));
+       DEBUG_ONLY(vfp_dump_live(live);)
        DB((dbg, LEVEL_1, "Stack before: "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
 
        op1_idx = x87_on_stack(state, reg_index_1);
        assert(op1_idx >= 0);
@@ -1620,7 +1618,7 @@ static int sim_Fucom(x87_state *state, ir_node *n)
        }
 
        return NO_NODE_ADDED;
-}  /* sim_Fucom */
+}
 
 /**
  * Simulate a Keep.
@@ -1657,10 +1655,10 @@ static int sim_Keep(x87_state *state, ir_node *node)
        }
 
        DB((dbg, LEVEL_1, "Stack after: "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
 
        return NO_NODE_ADDED;
-}  /* sim_Keep */
+}
 
 /**
  * Keep the given node alive by adding a be_Keep.
@@ -1748,7 +1746,7 @@ static ir_node *create_Copy(x87_state *state, ir_node *n)
        arch_set_irn_register(res, out);
 
        return res;
-}  /* create_Copy */
+}
 
 /**
  * Simulate a be_Copy.
@@ -1768,7 +1766,7 @@ static int sim_Copy(x87_state *state, ir_node *n)
        int                         op1_idx, out_idx;
        unsigned                    live;
 
-       cls = arch_get_irn_reg_class_out(n);
+       cls = arch_get_irn_reg_class(n);
        if (cls != &ia32_reg_classes[CLASS_ia32_vfp])
                return 0;
 
@@ -1779,13 +1777,11 @@ static int sim_Copy(x87_state *state, ir_node *n)
 
        DB((dbg, LEVEL_1, ">>> %+F %s -> %s\n", n,
                arch_register_get_name(op1), arch_register_get_name(out)));
-       DEBUG_ONLY(vfp_dump_live(live));
+       DEBUG_ONLY(vfp_dump_live(live);)
 
        op1_idx = x87_on_stack(state, arch_register_get_index(op1));
 
        if (is_vfp_live(arch_register_get_index(op1), live)) {
-               ir_node *pred = get_irn_n(n, 0);
-
                /* Operand is still live, a real copy. We need here an fpush that can
                   hold a a register, so use the fpushCopy or recreate constants */
                node = create_Copy(state, n);
@@ -1853,7 +1849,7 @@ static int sim_Copy(x87_state *state, ir_node *n)
                }
        }
        return NO_NODE_ADDED;
-}  /* sim_Copy */
+}
 
 /**
  * Returns the vf0 result Proj of a Call.
@@ -1874,7 +1870,7 @@ static ir_node *get_call_result_proj(ir_node *call)
        }
 
        return NULL;
-}  /* get_call_result_proj */
+}
 
 /**
  * Simulate a ia32_Call.
@@ -1919,10 +1915,10 @@ static int sim_Call(x87_state *state, ir_node *n)
 
 end_call:
        DB((dbg, LEVEL_1, "Stack after: "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
 
        return NO_NODE_ADDED;
-}  /* sim_Call */
+}
 
 /**
  * Simulate a be_Return.
@@ -1939,7 +1935,7 @@ static int sim_Return(x87_state *state, ir_node *n)
 
        /* only floating point return values must reside on stack */
        for (i = 0; i < n_res; ++i) {
-               ir_node *res = get_irn_n(n, be_pos_Return_val + i);
+               ir_node *res = get_irn_n(n, n_be_Return_val + i);
 
                if (mode_is_float(get_irn_mode(res)))
                        ++n_float_res;
@@ -1951,7 +1947,7 @@ static int sim_Return(x87_state *state, ir_node *n)
                x87_pop(state);
 
        return NO_NODE_ADDED;
-}  /* sim_Return */
+}
 
 typedef struct perm_data_t {
        const arch_register_t *in;
@@ -2007,7 +2003,7 @@ static int sim_Perm(x87_state *state, ir_node *irn)
        DB((dbg, LEVEL_1, "<<< %+F\n", irn));
 
        return NO_NODE_ADDED;
-}  /* sim_Perm */
+}
 
 /**
  * Kill any dead registers at block start by popping them from the stack.
@@ -2041,8 +2037,8 @@ static x87_state *x87_kill_deads(x87_simulator *sim, ir_node *block, x87_state *
                state = x87_clone_state(sim, state);
 
                DB((dbg, LEVEL_1, "Killing deads:\n"));
-               DEBUG_ONLY(vfp_dump_live(live));
-               DEBUG_ONLY(x87_dump_stack(state));
+               DEBUG_ONLY(vfp_dump_live(live);)
+               DEBUG_ONLY(x87_dump_stack(state);)
 
                if (kill_mask != 0 && live == 0) {
                        /* special case: kill all registers */
@@ -2094,7 +2090,7 @@ static x87_state *x87_kill_deads(x87_simulator *sim, ir_node *block, x87_state *
                keep_alive(keep);
        }
        return state;
-}  /* x87_kill_deads */
+}
 
 /**
  * Run a simulation and fix all virtual instructions for a block.
@@ -2117,7 +2113,7 @@ static void x87_simulate_block(x87_simulator *sim, ir_node *block)
 
        DB((dbg, LEVEL_1, "Simulate %+F\n", block));
        DB((dbg, LEVEL_2, "State at Block begin:\n "));
-       DEBUG_ONLY(x87_dump_stack(state));
+       DEBUG_ONLY(x87_dump_stack(state);)
 
        /* at block begin, kill all dead registers */
        state = x87_kill_deads(sim, block, state);
@@ -2154,7 +2150,7 @@ static void x87_simulate_block(x87_simulator *sim, ir_node *block)
 
        start_block = get_irg_start_block(get_irn_irg(block));
 
-       DB((dbg, LEVEL_2, "State at Block end:\n ")); DEBUG_ONLY(x87_dump_stack(state));
+       DB((dbg, LEVEL_2, "State at Block end:\n ")); DEBUG_ONLY(x87_dump_stack(state);)
 
        /* check if the state must be shuffled */
        foreach_block_succ(block, edge) {
@@ -2168,7 +2164,7 @@ static void x87_simulate_block(x87_simulator *sim, ir_node *block)
 
                if (succ_state->begin == NULL) {
                        DB((dbg, LEVEL_2, "Set begin state for succ %+F:\n", succ));
-                       DEBUG_ONLY(x87_dump_stack(state));
+                       DEBUG_ONLY(x87_dump_stack(state);)
                        succ_state->begin = state;
 
                        waitq_put(sim->worklist, succ);
@@ -2184,7 +2180,7 @@ static void x87_simulate_block(x87_simulator *sim, ir_node *block)
                }
        }
        bl_state->end = state;
-}  /* x87_simulate_block */
+}
 
 /**
  * Register a simulator function.
@@ -2196,7 +2192,7 @@ static void register_sim(ir_op *op, sim_func func)
 {
        assert(op->ops.generic == NULL);
        op->ops.generic = (op_func) func;
-}  /* register_sim */
+}
 
 /**
  * Create a new x87 simulator.
@@ -2209,7 +2205,7 @@ static void x87_init_simulator(x87_simulator *sim, ir_graph *irg)
        obstack_init(&sim->obst);
        sim->blk_states = pmap_create();
        sim->n_idx      = get_irg_last_idx(irg);
-       sim->live       = obstack_alloc(&sim->obst, sizeof(*sim->live) * sim->n_idx);
+       sim->live       = OALLOCN(&sim->obst, vfp_liveness, sim->n_idx);
 
        DB((dbg, LEVEL_1, "--------------------------------\n"
                "x87 Simulator started for %+F\n", irg));
@@ -2239,7 +2235,7 @@ static void x87_init_simulator(x87_simulator *sim, ir_graph *irg)
        register_sim(op_be_Return,         sim_Return);
        register_sim(op_be_Perm,           sim_Perm);
        register_sim(op_be_Keep,           sim_Keep);
-}  /* x87_init_simulator */
+}
 
 /**
  * Destroy a x87 simulator.
@@ -2251,7 +2247,7 @@ static void x87_destroy_simulator(x87_simulator *sim)
        pmap_destroy(sim->blk_states);
        obstack_free(&sim->obst, NULL);
        DB((dbg, LEVEL_1, "x87 Simulator stopped\n\n"));
-}  /* x87_destroy_simulator */
+}
 
 /**
  * Pre-block walker: calculate the liveness information for the block
@@ -2259,16 +2255,16 @@ static void x87_destroy_simulator(x87_simulator *sim)
  */
 static void update_liveness_walker(ir_node *block, void *data)
 {
-       x87_simulator *sim = data;
+       x87_simulator *sim = (x87_simulator*)data;
        update_liveness(sim, block);
-}  /* update_liveness_walker */
+}
 
 /*
  * Run a simulation and fix all virtual instructions for a graph.
  * Replaces all virtual floating point instructions and registers
  * by real ones.
  */
-void x87_simulate_graph(ir_graph *irg)
+void ia32_x87_simulate_graph(ir_graph *irg)
 {
        /* TODO improve code quality (less executed fxch) by using execfreqs */
 
@@ -2303,17 +2299,17 @@ void x87_simulate_graph(ir_graph *irg)
 
        /* iterate */
        do {
-               block = waitq_get(sim.worklist);
+               block = (ir_node*)waitq_get(sim.worklist);
                x87_simulate_block(&sim, block);
        } while (! waitq_empty(sim.worklist));
 
        /* kill it */
        del_waitq(sim.worklist);
        x87_destroy_simulator(&sim);
-}  /* x87_simulate_graph */
+}
 
 /* Initializes the x87 simulator. */
 void ia32_init_x87(void)
 {
        FIRM_DBG_REGISTER(dbg, "firm.be.ia32.x87");
-}  /* ia32_init_x87 */
+}