set *spill_ctxs;
set *spills; /**< all spill_info_t's, which must be placed */
pset *mem_phis; /**< set of all special spilled phis. allocated and freed separately */
- decide_irn_t is_mem_phi; /**< callback func to decide if a phi needs special spilling */
+ decide_irn_t is_spilled_phi;/**< callback func to decide if a phi needs special spilling */
void *data; /**< data passed to all callbacks */
DEBUG_ONLY(firm_dbg_module_t *dbg;)
};
)
/* Creates a new spill environment. */
-spill_env_t *be_new_spill_env(const be_chordal_env_t *chordal_env, decide_irn_t is_mem_phi, void *data) {
- spill_env_t *env = xmalloc(sizeof(env[0]));
- env->spill_ctxs = new_set(cmp_spillctx, 1024);
- env->spills = new_set(cmp_spillinfo, 1024);
- env->cls = chordal_env->cls;
- env->is_mem_phi = is_mem_phi;
- env->data = data;
- env->chordal_env = chordal_env;
+spill_env_t *be_new_spill_env(const be_chordal_env_t *chordal_env, decide_irn_t is_spilled_phi, void *data) {
+ spill_env_t *env = xmalloc(sizeof(env[0]));
+ env->spill_ctxs = new_set(cmp_spillctx, 1024);
+ env->spills = new_set(cmp_spillinfo, 1024);
+ env->cls = chordal_env->cls;
+ env->is_spilled_phi = is_spilled_phi;
+ env->data = data;
+ env->chordal_env = chordal_env;
obstack_init(&env->obst);
return env;
}
+void be_set_is_spilled_phi(spill_env_t *env, decide_irn_t is_spilled_phi, void *data) {
+ env->is_spilled_phi = is_spilled_phi;
+ env->data = data;
+}
+
/* Deletes a spill environment. */
void be_delete_spill_env(spill_env_t *senv) {
del_set(senv->spill_ctxs);
*
* @return a be_Spill node
*/
-static ir_node *be_spill_phi(spill_env_t *senv, ir_node *phi, ir_node *ctx_irn, unsigned visited_nr, set *already_visited_phis) {
+static ir_node *be_spill_phi(spill_env_t *senv, ir_node *phi, ir_node *ctx_irn, set *already_visited_phis, bitset_t *bs) {
int i, n = get_irn_arity(phi);
ir_graph *irg = senv->chordal_env->irg;
ir_node *bl = get_nodes_block(phi);
/* build a new PhiM */
NEW_ARR_A(ir_node *, ins, n);
for (i = 0; i < n; ++i) {
- ins[i] = new_r_Bad(irg);
+ ins[i] = new_r_Bad(irg);
}
phi_spill = new_r_Phi(senv->chordal_env->irg, bl, n, ins, mode_M);
key.phi = phi;
key.spill = phi_spill;
set_insert(already_visited_phis, &key, sizeof(key), HASH_PTR(phi));
+ bitset_set(bs, get_irn_idx(phi));
/* search an existing spill for this context */
ctx = be_get_spill_ctx(senv->spill_ctxs, phi, ctx_irn);
/* if not found spill the phi */
if (! ctx->spill) {
- set_irn_visited(phi, visited_nr);
-
/* collect all arguments of the phi */
for (i = 0; i < n; ++i) {
ir_node *arg = get_irn_n(phi, i);
phi_spill_assoc_t *entry;
if(is_Phi(arg) && pset_find_ptr(senv->mem_phis, arg)) {
- if (get_irn_visited(arg) < visited_nr)
- sub_res = be_spill_phi(senv, arg, ctx_irn, visited_nr, already_visited_phis);
+ if (! bitset_is_set(bs, get_irn_idx(arg)))
+ sub_res = be_spill_phi(senv, arg, ctx_irn, already_visited_phis, bs);
else {
/* we already visited the argument phi: get it's spill */
key.phi = arg;
*
* @return a be_Spill node
*/
-static ir_node *be_spill_node(spill_env_t *senv, ir_node *to_spill, unsigned visited_nr) {
+static ir_node *be_spill_node(spill_env_t *senv, ir_node *to_spill) {
ir_graph *irg = get_irn_irg(to_spill);
- int save_optimize = get_optimize();
- int save_normalize = get_opt_normalize();
set *already_visited_phis = new_set(cmp_phi_spill_assoc, 10);
- ir_node *res;
-
- /*
- * Disable optimization so that the phi functions do not
- * disappear.
- */
- set_optimize(0);
- set_opt_normalize(0);
+ ir_node *res;
+ bitset_t *bs = bitset_alloca(get_irg_last_idx(irg));
if (pset_find_ptr(senv->mem_phis, to_spill))
- res = be_spill_phi(senv, to_spill, to_spill, visited_nr, already_visited_phis);
+ res = be_spill_phi(senv, to_spill, to_spill, already_visited_phis, bs);
else
res = be_spill_irn(senv, to_spill, to_spill);
- del_set(already_visited_phis);
-
- /* reset the optimizations */
- set_optimize(save_optimize);
- set_opt_normalize(save_normalize);
-
return res;
}
/**
* Walker: fills the mem_phis set by evaluating Phi nodes
- * using the is_mem_phi() callback.
+ * using the is_spilled_phi() callback.
*/
static void phi_walker(ir_node *irn, void *env) {
spill_env_t *senv = env;
if (is_Phi(irn)) {
const arch_env_t *arch = senv->chordal_env->birg->main_env->arch_env;
if (arch_irn_has_reg_class(arch, irn, 0, senv->cls) &&
- senv->is_mem_phi(irn, senv->data)) {
+ senv->is_spilled_phi(irn, senv->data)) {
DBG((senv->dbg, LEVEL_1, " %+F\n", irn));
pset_insert_ptr(senv->mem_phis, irn);
}
}
}
-void be_insert_spills_reloads(spill_env_t *senv, pset *reload_set) {
+void be_insert_spills_reloads(spill_env_t *senv) {
const arch_env_t *aenv = senv->chordal_env->birg->main_env->arch_env;
ir_graph *irg = senv->chordal_env->irg;
- unsigned visited_nr;
ir_node *irn;
spill_info_t *si;
- pdeq *possibly_dead;
/* get all special spilled phis */
DBG((senv->dbg, LEVEL_1, "Mem-phis:\n"));
}
}
- visited_nr = get_irg_visited(irg) + 1;
- set_irg_visited(irg, visited_nr);
-
/* process each spilled node */
DBG((senv->dbg, LEVEL_1, "Insert spills and reloads:\n"));
- possibly_dead = new_pdeq();
for(si = set_first(senv->spills); si; si = set_next(senv->spills)) {
- int i;
reloader_t *rld;
ir_mode *mode = get_irn_mode(si->spilled_node);
- ir_node *value;
+ //ir_node *value;
pset *values = pset_new_ptr(16);
/* go through all reloads for this spill */
ir_node *new_val;
/* the spill for this reloader */
- ir_node *spill = be_spill_node(senv, si->spilled_node, visited_nr);
+ ir_node *spill = be_spill_node(senv, si->spilled_node);
#ifdef REMAT
if (check_remat_conditions(senv, spill, si->spilled_node, rld->reloader)) {
new_val = do_remat(senv, si->spilled_node, rld->reloader);
- pdeq_putl(possibly_dead, spill);
+ //pdeq_putl(possibly_dead, spill);
}
else
#endif
DBG((senv->dbg, LEVEL_1, " %+F of %+F before %+F\n", new_val, si->spilled_node, rld->reloader));
pset_insert_ptr(values, new_val);
- if(reload_set)
- pset_insert_ptr(reload_set, new_val);
}
/* introduce copies, rewire the uses */
pset_insert_ptr(values, si->spilled_node);
be_ssa_constr_set_ignore(senv->chordal_env->dom_front, values, senv->mem_phis);
- /* Remove reloads which are not used by anyone */
- /* TODO: Better call a general garbage collection routine here... this here gets clunky
- * and doesn't handle all cases (like memphis)
- */
- foreach_pset(values, value) {
- if(get_irn_n_edges(value) == 0) {
- sched_remove(value);
- // remove the node from preds
- if(be_is_Reload(value)) {
- ir_node* spill = get_irn_n(value, be_pos_Reload_mem);
- if(be_is_Spill(spill)) {
- assert(be_is_Spill(spill));
-
- set_irn_n(value, be_pos_Reload_mem, new_r_Bad(irg));
- set_irn_n(value, be_pos_Reload_frame, new_r_Bad(irg));
-
- // maybe the spill is not used anymoe too now?
- if(get_irn_n_edges(spill) == 0) {
- sched_remove(spill);
- set_irn_n(spill, be_pos_Spill_val, new_r_Bad(irg));
- set_irn_n(spill, be_pos_Spill_frame, new_r_Bad(irg));
- }
- } else if(is_Phi(spill)) {
- // TODO memphi
- } else {
- assert(0 && "Only spill or mem-phi expected here");
- }
- } else if(is_Phi(value)) {
- for(i = 0; i < get_Phi_n_preds(value); ++i)
- set_irn_n(value, i, new_r_Bad(irg));
- } else {
- assert(0);
- }
- }
- }
-
del_pset(values);
}
- foreach_pset(senv->mem_phis, irn) {
- int i, n;
- for (i = 0, n = get_irn_arity(irn); i < n; ++i) {
- pdeq_putl(possibly_dead, get_irn_n(irn, i));
- set_irn_n(irn, i, new_r_Bad(senv->chordal_env->irg));
- }
- sched_remove(irn);
- }
-
- /* check if possibly dead nodes are really dead yet */
- while (! pdeq_empty(possibly_dead)) {
- ir_node *irn = pdeq_getr(possibly_dead);
- const ir_edge_t *edge = get_irn_out_edge_first(irn);
-
- if (! edge) {
- int i;
- for (i = get_irn_arity(irn) - 1; i >= 0; --i) {
- pdeq_putl(possibly_dead, get_irn_n(irn, i));
- set_irn_n(irn, i, new_r_Bad(senv->chordal_env->irg));
- }
- sched_remove(irn);
- }
- }
- del_pdeq(possibly_dead);
del_pset(senv->mem_phis);
// reloads are placed now, but we might reuse the spill environment for further spilling decisions
struct _arch_env_t *arch_env = ssenv->cenv->birg->main_env->arch_env;
const arch_register_class_t *cls = arch_get_irn_reg_class(arch_env, spill, be_pos_Spill_val);
int size = get_mode_size_bytes(arch_register_class_mode(cls));
- assert(ss->size == size && "Different sizes for the same spill slot are not allowed.");
+ assert((int) ss->size == size && "Different sizes for the same spill slot are not allowed.");
for (irn = pset_first(ss->members); irn; irn = pset_next(ss->members)) {
/* use values_interfere here, because it uses the dominance check,
which does work for values in memory */