#include "belive_t.h"
#include "benode_t.h"
#include "besched_t.h"
+#include "benodesets.h"
+#include "bestatevent.h"
-static firm_dbg_module_t *dbg = NULL;
-#define DUMP_GRAPHS
+DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;)
#define get_chordal_arch(ce) ((ce)->birg->main_env->arch_env)
#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)
return !(p->arg == q->arg);
}
+typedef struct insert_all_perms_env_t {
+ be_chordal_env_t *chordal_env;
+ pmap *perm_map;
+} insert_all_perms_env_t;
+
/**
* Insert Perms in all predecessors of a block containing a phi
*/
static void insert_all_perms_walker(ir_node *bl, void *data) {
- be_chordal_env_t *chordal_env = data;
- pmap *perm_map = chordal_env->data;
+ insert_all_perms_env_t *env = data;
+ be_chordal_env_t *chordal_env = env->chordal_env;
+ pmap *perm_map = env->perm_map;
ir_graph *irg = chordal_env->irg;
+ be_lv_t *lv = chordal_env->birg->lv;
int i, n;
assert(is_Block(bl));
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);
* 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);
}
* Create a new Perm with the arguments just collected
* above in the arg_set and insert it into the schedule.
*/
- in = malloc(n_projs * sizeof(in[0]));
+ in = xmalloc(n_projs * sizeof(in[0]));
for(pp = set_first(arg_set); pp; pp = set_next(arg_set))
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);
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)
*/
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 */
- for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
+ for (phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
int i, max;
ir_node *arg, *phi_block, *arg_block;
const arch_register_t *phi_reg, *arg_reg;
assert(is_Phi(phi) && "Can only handle phi-destruction :)");
phi_block = get_nodes_block(phi);
- phi_reg = get_reg(phi);
- cls = arch_get_irn_reg_class(get_chordal_arch(chordal_env), phi, -1);
+ phi_reg = get_reg(phi);
+ cls = arch_get_irn_reg_class(get_chordal_arch(chordal_env), phi, -1);
/* process all arguments of the phi */
- for(i=0, max=get_irn_arity(phi); i<max; ++i) {
- arg = get_irn_n(phi, i);
+ for (i = 0, max = get_irn_arity(phi); i < max; ++i) {
+ arg = get_irn_n(phi, i);
arg_block = get_Block_cfgpred_block(phi_block, i);
- arg_reg = get_reg(arg);
+ arg_reg = get_reg(arg);
+
assert(arg_reg && "Register must be set while placing perms");
DBG((dbg, LEVEL_1, " for %+F(%s) -- %+F(%s)\n", phi, phi_reg->name, arg, arg_reg->name));
- if(nodes_interfere(chordal_env, phi, arg)) {
- /* Insert a duplicate in arguments block,
- * make it the new phi arg,
- * set its register,
- * insert it into schedule,
- * pin it
- */
- ir_node *dupl = be_new_Copy(cls, chordal_env->irg, arg_block, arg);
- assert(get_irn_mode(phi) == get_irn_mode(dupl));
+ if(values_interfere(lv, phi, arg)) {
+ /*
+ Insert a duplicate in arguments block,
+ make it the new phi arg,
+ set its register,
+ insert it into schedule,
+ 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);
+
+ /*
+ Conv signed <-> unsigned is killed on ia32
+ check for: (both int OR both float) AND equal mode sizes
+ */
+ 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_after(sched_skip(sched_last(arg_block), 0, sched_skip_cf_predicator, chordal_env->birg->main_env->arch_env), dupl);
}
if (phi_reg == arg_reg) {
- /* Phi and arg have the same register,
- * so pin and continue
- */
+ /* Phi and arg have the same register, so pin and continue */
pin_irn(arg, phi_block);
DBG((dbg, LEVEL_1, " arg has same reg: pin %+F(%s)\n", arg, get_reg(arg)->name));
continue;
DBG((dbg, LEVEL_1, " they do not interfere\n"));
assert(is_Proj(arg));
/*
- * First check if there is an other phi
- * - in the same block
- * - having arg at the current pos in its arg-list
- * - having the same color as arg
- *
- * If found, then pin the arg (for that phi)
- */
- if (!is_pinned(arg)) {
+ First check if there is an other phi
+ - in the same block
+ - having arg at the current pos in its arg-list
+ - having the same color as arg
+
+ If found, then pin the arg (for that phi)
+ */
+ if (! is_pinned(arg)) {
ir_node *other_phi;
+
DBG((dbg, LEVEL_1, " searching for phi with same arg having args register\n"));
+
for(other_phi = get_irn_link(phi_block); other_phi; other_phi = get_irn_link(other_phi)) {
- assert(is_Phi(other_phi) && get_nodes_block(phi) == get_nodes_block(other_phi) && "link fields are screwed up");
+
+ assert(is_Phi(other_phi) &&
+ get_nodes_block(phi) == get_nodes_block(other_phi) &&
+ "link fields are screwed up");
+
if (get_irn_n(other_phi, i) == arg && get_reg(other_phi) == arg_reg) {
DBG((dbg, LEVEL_1, " found %+F(%s)\n", other_phi, get_reg(other_phi)->name));
pin_irn(arg, phi_block);
}
if (is_pinned(arg)) {
- /* Insert a duplicate of the original value in arguments block,
- * make it the new phi arg,
- * set its register,
- * insert it into schedule,
- * 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);
- assert(get_irn_mode(phi) == get_irn_mode(dupl));
+ /*
+ Insert a duplicate of the original value in arguments block,
+ make it the new phi arg,
+ set its register,
+ insert it into schedule,
+ pin it
+ */
+ ir_node *perm = get_Proj_pred(arg);
+ 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);
+
+ /*
+ Conv signed <-> unsigned is killed on ia32
+ check for: (both int OR both float) AND equal mode sizes
+ */
+ 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 {
- /* No other phi has the same color (else arg would have been pinned),
- * so just set the register and pin
- */
+ /*
+ No other phi has the same color (else arg would have been pinned),
+ so just set the register and pin
+ */
set_reg(arg, phi_reg);
pin_irn(arg, phi_block);
DBG((dbg, LEVEL_1, " arg is not pinned: so pin %+F(%s)\n", arg, get_reg(arg)->name));
void be_ssa_destruction(be_chordal_env_t *chordal_env) {
pmap *perm_map = pmap_create();
- ir_graph *irg = chordal_env->irg;
+ ir_graph *irg = chordal_env->irg;
+ insert_all_perms_env_t insert_perms_env;
- dbg = firm_dbg_register("ir.be.ssadestr");
+ 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);
DBG((dbg, LEVEL_1, "Placing perms...\n"));
- irg_block_walk_graph(irg, insert_all_perms_walker, NULL, chordal_env);
-#ifdef DUMP_GRAPHS
- be_dump(irg, "-ssa_destr_perms_placed", dump_ir_block_graph_sched);
-#endif
+ insert_perms_env.chordal_env = chordal_env;
+ insert_perms_env.perm_map = perm_map;
+ irg_block_walk_graph(irg, insert_all_perms_walker, NULL, &insert_perms_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);
-#ifdef DUMP_GRAPHS
- be_dump(irg, "-ssa_destr_regs_set", dump_ir_block_graph_sched);
-#endif
+
+ if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR)
+ be_dump(irg, "-ssa_destr_regs_set", dump_ir_block_graph_sched);
pmap_destroy(perm_map);
}
ir_node *phi;
int i, max;
- for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
+ for (phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
const arch_register_t *phi_reg, *arg_reg;
phi_reg = get_reg(phi);
/* iterate over all args of phi */
- for(i=0, max=get_irn_arity(phi); i<max; ++i) {
+ for (i = 0, max = get_irn_arity(phi); i < max; ++i) {
ir_node *arg = get_irn_n(phi, i);
+
arg_reg = get_reg(arg);
- if(phi_reg != arg_reg) {
+
+ if (phi_reg != arg_reg) {
DBG((dbg, 0, "Error: Registers of %+F and %+F differ: %s %s\n", phi, arg, phi_reg->name, arg_reg->name));
assert(0);
}
- if(!is_pinned(arg)) {
+
+ if (! is_pinned(arg)) {
DBG((dbg, 0, "Warning: Phi argument %+F is not pinned.\n", arg));
assert(0);
}