X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fbe%2Fbessadestr.c;h=16efca4f777d336512c16dbb38a4b0dba5cd9b71;hb=80a6158fdd766f42ee6c508a773bc114ff1b61f3;hp=6d59ecac5a1871334cc4f8bff5ff92b9d3f92f25;hpb=f7f70d5202205131aa74a227dcc0831a419432e0;p=libfirm diff --git a/ir/be/bessadestr.c b/ir/be/bessadestr.c index 6d59ecac5..16efca4f7 100644 --- a/ir/be/bessadestr.c +++ b/ir/be/bessadestr.c @@ -28,6 +28,8 @@ #include "belive_t.h" #include "benode_t.h" #include "besched_t.h" +#include "benodesets.h" +#include "bestatevent.h" DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;) @@ -35,7 +37,7 @@ DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;) #define get_reg(irn) arch_get_irn_register(get_chordal_arch(chordal_env), irn) #define set_reg(irn, reg) arch_set_irn_register(get_chordal_arch(chordal_env), irn, reg) -#define is_Perm(irn) (arch_irn_classify(arch_env, irn) == arch_irn_class_perm) +#define is_Perm(irn) (arch_irn_class_is(arch_env, irn, perm)) #define get_reg_cls(irn) (arch_get_irn_reg_class(arch_env, irn, -1)) #define is_curr_reg_class(irn) (get_reg_cls(p) == chordal_env->cls) @@ -84,6 +86,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { be_chordal_env_t *chordal_env = data; pmap *perm_map = chordal_env->data; ir_graph *irg = chordal_env->irg; + be_lv_t *lv = chordal_env->birg->lv; int i, n; assert(is_Block(bl)); @@ -109,7 +112,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) { perm_proj_t templ; ir_node *arg = get_irn_n(phi, i); - unsigned hash = HASH_PTR(arg); + unsigned hash = nodeset_hash(arg); templ.arg = arg; pp = set_find(arg_set, &templ, sizeof(templ), hash); @@ -121,7 +124,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { * interferes with the phi and must thus not be member of a * Perm. A copy will be inserted for this argument alter on. */ - if(!pp && !is_live_in(bl, arg)) { + if(!pp && !be_is_live_in(lv, bl, arg)) { templ.pos = n_projs++; set_insert(arg_set, &templ, sizeof(templ), hash); } @@ -138,6 +141,8 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { in[pp->pos] = pp->arg; perm = be_new_Perm(chordal_env->cls, irg, pred_bl, n_projs, in); + be_stat_ev("phi_perm", n_projs); + free(in); insert_after = sched_skip(sched_last(pred_bl), 0, sched_skip_cf_predicator, chordal_env->birg->main_env->arch_env); sched_add_after(insert_after, perm); @@ -165,7 +170,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { perm_proj_t templ; templ.arg = get_irn_n(phi, i); - pp = set_find(arg_set, &templ, sizeof(templ), HASH_PTR(templ.arg)); + pp = set_find(arg_set, &templ, sizeof(templ), nodeset_hash(templ.arg)); /* If not found, it was an interfering argument */ if (pp) @@ -190,6 +195,7 @@ static void insert_all_perms_walker(ir_node *bl, void *data) { */ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { be_chordal_env_t *chordal_env = data; + be_lv_t *lv = chordal_env->birg->lv; ir_node *phi; /* Consider all phis of this block */ @@ -215,7 +221,7 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { DBG((dbg, LEVEL_1, " for %+F(%s) -- %+F(%s)\n", phi, phi_reg->name, arg, arg_reg->name)); - if(values_interfere(phi, arg)) { + if(values_interfere(lv, phi, arg)) { /* Insert a duplicate in arguments block, make it the new phi arg, @@ -224,6 +230,9 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { pin it */ ir_node *dupl = be_new_Copy(cls, chordal_env->irg, arg_block, arg); + + /* this is commented out because it will fail in case of unknown float */ +#if 0 ir_mode *m_phi = get_irn_mode(phi), *m_dupl = get_irn_mode(dupl); /* @@ -233,6 +242,7 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { assert(((mode_is_int(m_phi) && mode_is_int(m_dupl)) || (mode_is_float(m_phi) && mode_is_float(m_dupl))) && (get_mode_size_bits(m_phi) == get_mode_size_bits(m_dupl))); +#endif /* if 0 */ set_irn_n(phi, i, dupl); set_reg(dupl, phi_reg); @@ -287,8 +297,11 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { pin it */ ir_node *perm = get_Proj_pred(arg); - ir_node *orig_val = get_irn_n(perm, get_Proj_proj(arg)); - ir_node *dupl = be_new_Copy(cls, chordal_env->irg, arg_block, orig_val); + ir_node *dupl = be_new_Copy(cls, chordal_env->irg, arg_block, arg); + ir_node *ins; + + /* this is commented out because it will fail in case of unknown float */ +#if 0 ir_mode *m_phi = get_irn_mode(phi); ir_mode *m_dupl = get_irn_mode(dupl); @@ -299,10 +312,13 @@ static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) { assert(((mode_is_int(m_phi) && mode_is_int(m_dupl)) || (mode_is_float(m_phi) && mode_is_float(m_dupl))) && (get_mode_size_bits(m_phi) == get_mode_size_bits(m_dupl))); +#endif /* if 0 */ set_irn_n(phi, i, dupl); set_reg(dupl, phi_reg); - sched_add_before(perm, dupl); + /* skip the Perm's Projs and insert the copies behind. */ + for(ins = sched_next(perm); is_Proj(ins); ins = sched_next(ins)); + sched_add_before(ins, dupl); pin_irn(dupl, phi_block); DBG((dbg, LEVEL_1, " arg is pinned: insert %+F(%s)\n", dupl, get_reg(dupl)->name)); } else { @@ -324,6 +340,8 @@ void be_ssa_destruction(be_chordal_env_t *chordal_env) { FIRM_DBG_REGISTER(dbg, "ir.be.ssadestr"); + be_assure_liveness(chordal_env->birg); + /* create a map for fast lookup of perms: block --> perm */ chordal_env->data = perm_map; irg_walk_graph(irg, clear_link, collect_phis_walker, chordal_env); @@ -334,7 +352,9 @@ void be_ssa_destruction(be_chordal_env_t *chordal_env) { if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR) be_dump(irg, "-ssa_destr_perms_placed", dump_ir_block_graph_sched); - be_liveness(irg); + // Matze: really needed here? + be_invalidate_liveness(chordal_env->birg); + be_assure_liveness(chordal_env->birg); DBG((dbg, LEVEL_1, "Setting regs and placing dupls...\n")); irg_block_walk_graph(irg, set_regs_or_place_dupls_walker, NULL, chordal_env);