-/**
- * Author: Daniel Grund
- * Date: 25.05.2005
- * Copyright: (c) Universitaet Karlsruhe
- * Licence: This file protected by GPL - GNU GENERAL PUBLIC LICENSE.
+/*
+ * Copyright (C) 1995-2008 University of Karlsruhe. All right reserved.
+ *
+ * This file is part of libFirm.
+ *
+ * This file may be distributed and/or modified under the terms of the
+ * GNU General Public License version 2 as published by the Free Software
+ * Foundation and appearing in the file LICENSE.GPL included in the
+ * packaging of this file.
*
- * Performs SSA-Destruction.
+ * Licensees holding valid libFirm Professional Edition licenses may use
+ * this file in accordance with the libFirm Commercial License.
+ * Agreement provided with the Software.
+ *
+ * This file is provided AS IS with NO WARRANTY OF ANY KIND, INCLUDING THE
+ * WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE.
+ */
+
+/**
+ * @file
+ * @brief Performs SSA-Destruction.
+ * @author Daniel Grund
+ * @date 25.05.2005
+ * @version $Id$
*/
-#ifdef HAVE_CONFIG_H
#include "config.h"
-#endif
+
+#include "bessadestr.h"
#include "debug.h"
#include "set.h"
#include "irnode_t.h"
#include "ircons_t.h"
#include "iredges_t.h"
+#include "irgwalk.h"
#include "irgmod.h"
#include "irdump.h"
#include "irprintf.h"
#include "bechordal_t.h"
#include "bearch.h"
#include "belive_t.h"
-#include "benode_t.h"
-#include "besched_t.h"
-
-static firm_dbg_module_t *dbg = NULL;
-#define DEBUG_LVL SET_LEVEL_0
-#undef DUMP_GRAPHS
+#include "benode.h"
+#include "besched.h"
+#include "bestatevent.h"
+#include "beirg.h"
+#include "beintlive_t.h"
-#define get_chordal_arch(ce) ((ce)->session_env->main_env->arch_env)
-#define get_reg(irn) arch_get_irn_register(get_chordal_arch(chordal_env), irn, 0)
-#define set_reg(irn, reg) arch_set_irn_register(get_chordal_arch(chordal_env), irn, 0, reg)
-
-#define is_Perm(irn) (arch_irn_classify(arch_env, irn) == arch_irn_class_perm)
-#define get_reg_cls(irn) (arch_get_irn_reg_class(arch_env, irn, arch_pos_make_out(0)))
-#define is_curr_reg_class(irn) (get_reg_cls(p) == chordal_env->cls)
+DEBUG_ONLY(static firm_dbg_module_t *dbg = NULL;)
static void clear_link(ir_node *irn, void *data)
{
- set_irn_link(irn, NULL);
+ (void) data;
+ set_irn_link(irn, NULL);
}
/**
- * Build a list of phis of a block.
+ * For each block build a linked list of phis that
+ * - are in that block
+ * - have the current register class
+ * The list is rooted at get_irn_link(BB).
*/
-static void collect_phis(ir_node *irn, void *data)
+static void collect_phis_walker(ir_node *irn, void *data)
{
- be_chordal_env_t *env = data;
- if(is_Phi(irn) &&
- arch_irn_has_reg_class(env->session_env->main_env->arch_env,
- irn, arch_pos_make_out(0), env->cls)) {
-
- ir_node *bl = get_nodes_block(irn);
- set_irn_link(irn, get_irn_link(bl));
- set_irn_link(bl, irn);
- }
+ be_chordal_env_t *env = (be_chordal_env_t*)data;
+ if (is_Phi(irn) && chordal_has_class(env, irn)) {
+ ir_node *bl = get_nodes_block(irn);
+ set_irn_link(irn, get_irn_link(bl));
+ set_irn_link(bl, irn);
+ }
}
/**
- * Build a ring of phis for each block in the link field.
- * @param env The chordal env.
+ * This struct represents a Proj for a Perm.
+ * It records the argument in the Perm and the corresponding Proj of the
+ * Perm.
*/
-static INLINE void build_phi_rings(be_chordal_env_t *env)
+typedef struct {
+ ir_node *arg; /**< The phi argument to make the Proj for. */
+ int pos; /**< The proj number the Proj will get.
+ This also denotes the position of @p arg
+ in the in array of the Perm. */
+ ir_node *proj; /**< The proj created for @p arg. */
+} perm_proj_t;
+
+static int cmp_perm_proj(const void *a, const void *b, size_t n)
{
- irg_walk_graph(env->session_env->irg, clear_link, collect_phis, env);
-}
+ const perm_proj_t *p = (const perm_proj_t*)a;
+ const perm_proj_t *q = (const perm_proj_t*)b;
+ (void) n;
-static int skip_cf_predicator(const ir_node *irn, void *data) {
- be_chordal_env_t *chordal_env = data;
- arch_env_t *ae = chordal_env->session_env->main_env->arch_env;
- return arch_irn_classify(ae, irn) == arch_irn_class_branch;
+ 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;
- ir_graph *irg = chordal_env->session_env->irg;
- const be_node_factory_t *fact = chordal_env->session_env->main_env->node_factory;
-
- /* Dummy targets for the projs */
- ir_node *dummy = new_rd_Unknown(irg, mode_T);
-
- assert(is_Block(bl));
-
- /* If the link flag is NULL, this block has no phis. */
- if(get_irn_link(bl)) {
- int i, n;
-
- /* Look at all predecessors of the phi block */
- for(i = 0, n = get_irn_arity(bl); i < n; ++i) {
- ir_node *pred_bl = get_Block_cfgpred_block(bl, i);
- ir_node *phi, *perm, *insert_after;
- ir_node **in;
- int n_projs = 0;
- pmap_entry *ent;
- pmap *arg_map = pmap_create();
-
- assert(!pmap_contains(perm_map, pred_bl) && "Already permed that block");
-
- /*
- * Note that all phis in the list are in the same register class
- * by construction.
- */
- for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
- ir_node *arg = get_irn_n(phi, i);
- ir_node *proj = pmap_get(arg_map, arg);
-
- if(!proj && !is_live_in(bl, arg)) {
- proj = new_r_Proj(irg, pred_bl, dummy, get_irn_mode(arg), n_projs++);
- pmap_insert(arg_map, arg, proj);
- }
-
- if (proj) {
- assert(get_irn_mode(phi) == get_irn_mode(proj));
- set_irn_n(phi, i, proj);
+ insert_all_perms_env_t *env = (insert_all_perms_env_t*)data;
+ be_chordal_env_t *chordal_env = env->chordal_env;
+ pmap *perm_map = env->perm_map;
+ be_lv_t *lv = be_get_irg_liveness(chordal_env->irg);
+ int i, n;
+
+ assert(is_Block(bl));
+
+ /* If the link flag is NULL, this block has no phis. */
+ if (!get_irn_link(bl))
+ return;
+
+ /* Look at all predecessors of the phi block */
+ for (i = 0, n = get_irn_arity(bl); i < n; ++i) {
+ ir_node *phi, *perm, *insert_after, **in;
+ perm_proj_t *pp;
+ set *arg_set = new_set(cmp_perm_proj, chordal_env->cls->n_regs);
+ ir_node *pred_bl = get_Block_cfgpred_block(bl, i);
+ int n_projs = 0;
+
+ assert(!pmap_contains(perm_map, pred_bl) && "Already permed that block");
+
+ /*
+ * Note that all phis in the list are in the same
+ * register class by construction.
+ */
+ for (phi = (ir_node*)get_irn_link(bl); phi != NULL;
+ phi = (ir_node*)get_irn_link(phi)) {
+ ir_node *arg = get_irn_n(phi, i);
+ unsigned hash;
+ perm_proj_t templ;
+
+ hash = hash_irn(arg);
+ templ.arg = arg;
+ pp = (perm_proj_t*)set_find(arg_set, &templ, sizeof(templ), hash);
+
+ /*
+ * If a proj_perm_t entry has not been made in the argument set,
+ * create one. The only restriction is, that the phi argument
+ * may not be live in at the current block, since this argument
+ * interferes with the phi and must thus not be member of a
+ * Perm. A copy will be inserted for this argument later on.
+ */
+ if (!pp && !be_is_live_in(lv, bl, arg)) {
+ templ.pos = n_projs++;
+ set_insert(arg_set, &templ, sizeof(templ), hash);
+ }
}
- }
-
- in = malloc(n_projs * sizeof(in[0]));
- pmap_foreach(arg_map, ent) {
- int proj_nr = get_Proj_proj(ent->value);
- in[proj_nr] = ent->key;
- }
-
- perm = new_Perm(fact, chordal_env->cls, irg, pred_bl, n_projs, in);
- insert_after = sched_skip(sched_last(pred_bl), 0, skip_cf_predicator, chordal_env);
- sched_add_after(insert_after, perm);
- exchange(dummy, perm);
-
- /* register allocation is copied form former arguments to the projs (new arguments) */
- pmap_foreach(arg_map, ent) {
- DBG((dbg, LEVEL_2, "Copy register assignment %s from %+F to %+F\n", get_reg(ent->key)->name, ent->key, ent->value));
- set_reg(ent->value, get_reg(ent->key));
- }
-
- free(in);
- pmap_destroy(arg_map);
-
- /* register in perm map */
- pmap_insert(perm_map, pred_bl, perm);
- }
- }
-}
-static void insert_all_perms(be_chordal_env_t *chordal_env) {
- DBG((dbg, LEVEL_1, "Placing perms...\n"));
- irg_block_walk_graph(chordal_env->session_env->irg, insert_all_perms_walker, NULL, chordal_env);
+
+ if (n_projs) {
+ /*
+ * Create a new Perm with the arguments just collected
+ * above in the arg_set and insert it into the schedule.
+ */
+ in = XMALLOCN(ir_node*, n_projs);
+ foreach_set(arg_set, perm_proj_t*, pp) {
+ in[pp->pos] = pp->arg;
+ }
+
+ perm = be_new_Perm(chordal_env->cls, pred_bl, n_projs, in);
+ be_stat_ev("phi_perm", n_projs);
+
+ insert_after = pred_bl;
+ do {
+ insert_after = sched_prev(insert_after);
+ } while (is_cfop(insert_after));
+ sched_add_after(insert_after, perm);
+
+ /*
+ * Make the Projs for the Perm and insert into schedule.
+ * Register allocation is copied from the former phi
+ * arguments to the projs (new phi arguments).
+ */
+ insert_after = perm;
+ foreach_set(arg_set, perm_proj_t*, pp) {
+ ir_node *proj = new_r_Proj(perm, get_irn_mode(pp->arg), pp->pos);
+ pp->proj = proj;
+ assert(arch_get_irn_register(pp->arg));
+ arch_set_irn_register(proj, arch_get_irn_register(pp->arg));
+ insert_after = proj;
+ DBG((dbg, LEVEL_2, "Copy register assignment %s from %+F to %+F\n", arch_get_irn_register(pp->arg)->name, pp->arg, pp->proj));
+ }
+
+ /*
+ * Set the phi nodes to their new arguments: The Projs of the Perm
+ */
+ for (phi = (ir_node*)get_irn_link(bl); phi != NULL;
+ phi = (ir_node*)get_irn_link(phi)) {
+ perm_proj_t templ;
+
+ templ.arg = get_irn_n(phi, i);
+ pp = (perm_proj_t*)set_find(arg_set, &templ, sizeof(templ), hash_irn(templ.arg));
+
+ /* If not found, it was an interfering argument */
+ if (pp) {
+ set_irn_n(phi, i, pp->proj);
+ be_liveness_introduce(lv, pp->proj);
+ }
+ }
+
+ /* update the liveness of the Perm's operands. It might be changed. */
+ {
+ int i;
+ for (i = 0; i < n_projs; ++i)
+ be_liveness_update(lv, in[i]);
+ }
+ free(in);
+
+ /* register in perm map */
+ pmap_insert(perm_map, pred_bl, perm);
+ }
+
+ del_set(arg_set);
+ }
}
#define is_pinned(irn) (get_irn_link(irn))
#define get_pinning_block(irn) ((ir_node *)get_irn_link(irn))
#define pin_irn(irn, lock) (set_irn_link(irn, lock))
-
/**
- * Adjusts the register allocation for the phi-operands
- * by inserting perm nodes, if necessary.
- * @param phi The phi node to adjust operands for
+ * Adjusts the register allocation for the (new) phi-operands
+ * and insert duplicates iff necessary.
*/
-static void adjust_phi_arguments(be_chordal_env_t *chordal_env, ir_node *phi) {
- int i, max;
- ir_node *arg, *phi_block, *arg_block;
- const be_main_session_env_t *session = chordal_env->session_env;
- const arch_register_t *phi_reg, *arg_reg;
- const arch_register_class_t *cls;
-
- 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, arch_pos_make_out(0));
-
- /* process all arguments of the phi */
- 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);
- 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 = new_Copy(session->main_env->node_factory, cls, session->irg, arg_block, arg);
- assert(get_irn_mode(phi) == get_irn_mode(dupl));
- set_irn_n(phi, i, dupl);
- set_reg(dupl, phi_reg);
- sched_add_after(sched_skip(sched_last(arg_block), 0, skip_cf_predicator, chordal_env), dupl);
- pin_irn(dupl, phi_block);
- DBG((dbg, LEVEL_1, " they do interfere: insert %+F(%s)\n", dupl, get_reg(dupl)->name));
- } else {
- /*
- * First check if there is a 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
- */
+static void set_regs_or_place_dupls_walker(ir_node *bl, void *data)
+{
+ be_chordal_env_t *chordal_env = (be_chordal_env_t*)data;
+ be_lv_t *lv = be_get_irg_liveness(chordal_env->irg);
+ ir_node *phi;
+
+ /* Consider all phis of this block */
+ for (phi = (ir_node*)get_irn_link(bl); phi != NULL;
+ phi = (ir_node*)get_irn_link(phi)) {
+ ir_node *phi_block = get_nodes_block(phi);
+ const arch_register_t *phi_reg = arch_get_irn_register(phi);
+ const arch_register_class_t *cls = phi_reg->reg_class;
+ int max;
+ int i;
+
+ assert(is_Phi(phi) && "Can only handle phi-destruction :)");
+
+ /* process all arguments of the phi */
+ for (i = 0, max = get_irn_arity(phi); i < max; ++i) {
+ ir_node *arg = get_irn_n(phi, i);
+ const arch_register_t *arg_reg;
+ ir_node *arg_block;
+
+ arg_block = get_Block_cfgpred_block(phi_block, i);
+ arg_reg = arch_get_irn_register(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 (phi_reg == arg_reg
+ || (arg_reg->type & arch_register_type_joker)
+ || (arg_reg->type & arch_register_type_virtual)) {
+ /* 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, arch_get_irn_register(arg)->name));
+ continue;
+ }
+
+ if (be_values_interfere(lv, phi, arg)) {
+ ir_node *schedpoint;
+
+ /*
+ 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, arg_block, arg);
+
+ set_irn_n(phi, i, dupl);
+ arch_set_irn_register(dupl, phi_reg);
+ schedpoint = arg_block;
+ do {
+ schedpoint = sched_prev(schedpoint);
+ } while (is_cfop(schedpoint));
+ sched_add_after(schedpoint, dupl);
+ pin_irn(dupl, phi_block);
+ be_liveness_introduce(lv, dupl);
+ be_liveness_update(lv, arg);
+ DBG((dbg, LEVEL_1, " they do interfere: insert %+F(%s)\n", dupl, arch_get_irn_register(dupl)->name));
+ continue; /* with next argument */
+ }
+
DBG((dbg, LEVEL_1, " they do not interfere\n"));
assert(is_Proj(arg));
- 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");
- 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));
+
+ for (other_phi = (ir_node*)get_irn_link(phi_block);
+ other_phi != NULL;
+ other_phi = (ir_node*)get_irn_link(other_phi)) {
+
+ 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 && arch_get_irn_register(other_phi) == arg_reg) {
+ DBG((dbg, LEVEL_1, " found %+F(%s)\n", other_phi, arch_get_irn_register(other_phi)->name));
pin_irn(arg, phi_block);
+ break;
}
}
}
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
- */
+ /*
+ 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 = new_Copy(session->main_env->node_factory, cls, session->irg, arg_block, orig_val);
- assert(get_irn_mode(phi) == get_irn_mode(dupl));
+ ir_node *dupl = be_new_Copy(cls, arg_block, arg);
+ ir_node *ins;
+
set_irn_n(phi, i, dupl);
- set_reg(dupl, phi_reg);
- sched_add_before(perm, dupl);
+ arch_set_irn_register(dupl, phi_reg);
+ /* 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));
+ be_liveness_introduce(lv, dupl);
+ be_liveness_update(lv, arg);
+ DBG((dbg, LEVEL_1, " arg is pinned: insert %+F(%s)\n", dupl, arch_get_irn_register(dupl)->name));
} else {
- /* No other phi has the same color (else arg would be pinned),
- * so just set the register and pin
- */
- set_reg(arg, phi_reg);
+ /*
+ No other phi has the same color (else arg would have been pinned),
+ so just set the register and pin
+ */
+ arch_set_irn_register(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));
+ DBG((dbg, LEVEL_1, " arg is not pinned: so pin %+F(%s)\n", arg, arch_get_irn_register(arg)->name));
}
}
}
}
-static void set_regs_or_place_dupls_walker(ir_node *bl, void *data) {
- be_chordal_env_t *chordal_env = data;
- ir_node *phi;
+void be_ssa_destruction(be_chordal_env_t *chordal_env)
+{
+ insert_all_perms_env_t insert_perms_env;
+ pmap *perm_map = pmap_create();
+ ir_graph *irg = chordal_env->irg;
+ be_lv_t *lv = be_assure_liveness(irg);
- for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi))
- adjust_phi_arguments(chordal_env, phi);
-}
+ FIRM_DBG_REGISTER(dbg, "ir.be.ssadestr");
-static void set_regs_or_place_dupls(be_chordal_env_t *chordal_env)
-{
- DBG((dbg, LEVEL_1, "Setting regs and placing dupls...\n"));
- irg_block_walk_graph(chordal_env->session_env->irg,
- set_regs_or_place_dupls_walker, NULL, chordal_env);
-}
+ be_liveness_invalidate(lv);
+ /* create a map for fast lookup of perms: block --> perm */
+ irg_walk_graph(irg, clear_link, collect_phis_walker, chordal_env);
-void be_ssa_destruction(be_chordal_env_t *chordal_env) {
- pmap *perm_map = pmap_create();
- ir_graph *irg = chordal_env->session_env->irg;
+ DBG((dbg, LEVEL_1, "Placing perms...\n"));
+ 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);
- dbg = firm_dbg_register("ir.be.ssadestr");
- firm_dbg_set_mask(dbg, DEBUG_LVL);
+ // Matze: really needed here?
+ // Sebastian: Yes. the walker function uses interference.
+ be_liveness_invalidate(lv);
- /* create a map for fast lookup of perms: block --> perm */
- chordal_env->data = perm_map;
-
- build_phi_rings(chordal_env);
- insert_all_perms(chordal_env);
-#ifdef DUMP_GRAPHS
- dump_ir_block_graph_sched(irg, "-ssa_destr_perms_placed");
-#endif
- set_regs_or_place_dupls(chordal_env);
-#ifdef DUMP_GRAPHS
- dump_ir_block_graph_sched(irg, "-ssa_destr_regs_set");
-#endif
+ if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR)
+ dump_ir_graph(irg, "ssa_destr_perms_placed");
+
+ be_liveness_assure_chk(lv);
+
+ DBG((dbg, LEVEL_1, "Setting regs and placing dupls...\n"));
+ irg_block_walk_graph(irg, set_regs_or_place_dupls_walker, NULL, chordal_env);
+
+ /* TODO: unfortunately updating doesn't work yet. */
+ be_liveness_invalidate(lv);
+
+ if (chordal_env->opts->dump_flags & BE_CH_DUMP_SSADESTR)
+ dump_ir_graph(irg, "ssa_destr_regs_set");
pmap_destroy(perm_map);
}
static void ssa_destruction_check_walker(ir_node *bl, void *data)
{
- be_chordal_env_t *chordal_env = data;
ir_node *phi;
int i, max;
+ (void)data;
- for(phi = get_irn_link(bl); phi; phi = get_irn_link(phi)) {
+ for (phi = (ir_node*)get_irn_link(bl); phi != NULL;
+ phi = (ir_node*)get_irn_link(phi)) {
const arch_register_t *phi_reg, *arg_reg;
- phi_reg = get_reg(phi);
+ phi_reg = arch_get_irn_register(phi);
/* iterate over all args of phi */
- 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) {
+ for (i = 0, max = get_irn_arity(phi); i < max; ++i) {
+ ir_node *arg = get_irn_n(phi, i);
+ const arch_register_req_t *req = arch_get_register_req_out(arg);
+
+ if (req->type & arch_register_req_type_ignore)
+ continue;
+
+ arg_reg = arch_get_irn_register(arg);
+
+ 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);
}
}
}
-void be_ssa_destruction_check(be_chordal_env_t *chordal_env) {
- irg_block_walk_graph(chordal_env->session_env->irg, ssa_destruction_check_walker, NULL, chordal_env);
+void be_ssa_destruction_check(be_chordal_env_t *chordal_env)
+{
+ irg_block_walk_graph(chordal_env->irg, ssa_destruction_check_walker, NULL, NULL);
}