Added support for SymConst(ofs_ent)
[libfirm] / ir / be / belower.c
index e093688..0ff5504 100644 (file)
@@ -23,6 +23,7 @@
 #include "bechordal_t.h"
 #include "besched_t.h"
 #include "bestat.h"
+#include "benodesets.h"
 
 #include "irgmod.h"
 #include "iredges_t.h"
 #endif
 
 #undef is_Perm
-#define is_Perm(arch_env, irn) (arch_irn_classify(arch_env, irn) == arch_irn_class_perm)
+#define is_Perm(arch_env, irn) (arch_irn_class_is(arch_env, irn, perm))
+
+/* associates op with it's copy and CopyKeep */
+typedef struct {
+       ir_node *op;         /* an irn which must be different */
+       pset    *copies;     /* all non-spillable copies of this irn */
+} op_copy_assoc_t;
+
+/* environment for constraints */
+typedef struct {
+       be_irg_t       *birg;
+       pset           *op_set;
+       struct obstack obst;
+} constraint_env_t;
 
 /* lowering walker environment */
 typedef struct _lower_env_t {
@@ -70,6 +84,14 @@ typedef struct _perm_cycle_t {
        perm_type_t             type;        /**< type (CHAIN or CYCLE) */
 } perm_cycle_t;
 
+/* Compare the two operands */
+static int cmp_op_copy_assoc(const void *a, const void *b) {
+       const op_copy_assoc_t *op1 = a;
+       const op_copy_assoc_t *op2 = b;
+
+       return op1->op != op2->op;
+}
+
 /* Compare the in registers of two register pairs */
 static int compare_reg_pair(const void *a, const void *b) {
        const reg_pair_t *pair_a = a;
@@ -248,8 +270,9 @@ static void lower_perm_node(ir_node *irn, void *walk_env) {
        const arch_register_class_t *reg_class;
        const arch_env_t            *arch_env;
        lower_env_t     *env         = walk_env;
-       int              real_size   = 0;
-       int              n, i, pn, do_copy, j, n_ops;
+       int             real_size    = 0;
+       int             keep_perm    = 0;
+       int             n, i, pn, do_copy, j, n_ops;
        reg_pair_t      *pairs;
        const ir_edge_t *edge;
        perm_cycle_t    *cycle;
@@ -315,6 +338,7 @@ static void lower_perm_node(ir_node *irn, void *walk_env) {
 
                        /* reroute the edges from the proj to the argument */
                        edges_reroute(pairs[i].out_node, pairs[i].in_node, env->chord_env->irg);
+                       set_irn_n(pairs[i].out_node, 0, new_Bad());
 
                        pairs[i].checked = 1;
                }
@@ -344,11 +368,14 @@ static void lower_perm_node(ir_node *irn, void *walk_env) {
                }
                DB((mod, LEVEL_1, "\n"));
 
-               /* We don't need to do anything if we have a Perm with two
+               /*
+                       We don't need to do anything if we have a Perm with two
                        elements which represents a cycle, because those nodes
-                       already represent exchange nodes */
+                       already represent exchange nodes
+               */
                if (n == 2 && cycle->type == PERM_CYCLE) {
                        free(cycle);
+                       keep_perm = 1;
                        continue;
                }
 
@@ -472,10 +499,14 @@ static void lower_perm_node(ir_node *irn, void *walk_env) {
                free(cycle);
        }
 
-
-
        /* remove the perm from schedule */
-       sched_remove(irn);
+       if (! keep_perm) {
+               int arity = get_irn_arity(irn);
+               for(i = 0; i < arity; ++i) {
+                       set_irn_n(irn, i, new_Bad());
+               }
+               sched_remove(irn);
+       }
 }
 
 
@@ -497,25 +528,14 @@ static ir_node *belower_skip_proj(ir_node *irn) {
        return irn;
 }
 
-static void fix_in(ir_node *irn, ir_node *old, ir_node *nw) {
-       int i, n;
-
-       irn = belower_skip_proj(irn);
-       n   = get_irn_arity(irn);
-
-       for (i = 0; i < n; i++) {
-               if (get_irn_n(irn, i) == old) {
-                       set_irn_n(irn, i, nw);
-                       break;
-               }
-       }
-}
-
-static void gen_assure_different_pattern(ir_node *irn, be_irg_t *birg, ir_node *other_different) {
-       const arch_env_t          *arch_env = birg->main_env->arch_env;
-       ir_node                   *in[2], *keep, *cpy, *temp;
-       ir_node                   *block = get_nodes_block(irn);
-       const arch_register_class_t *cls = arch_get_irn_reg_class(arch_env, other_different, -1);
+static void gen_assure_different_pattern(ir_node *irn, ir_node *other_different, constraint_env_t *env) {
+       be_irg_t                    *birg     = env->birg;
+       pset                        *op_set   = env->op_set;
+       const arch_env_t            *arch_env = birg->main_env->arch_env;
+       ir_node                     *block    = get_nodes_block(irn);
+       const arch_register_class_t *cls      = arch_get_irn_reg_class(arch_env, other_different, -1);
+       ir_node                     *in[2], *keep, *cpy;
+       op_copy_assoc_t             key, *entry;
        FIRM_DBG_REGISTER(firm_dbg_module_t *mod, "firm.be.lower");
 
        if (arch_irn_is(arch_env, other_different, ignore) || ! mode_is_datab(get_irn_mode(other_different))) {
@@ -528,16 +548,12 @@ static void gen_assure_different_pattern(ir_node *irn, be_irg_t *birg, ir_node *
        /* in block far far away                             */
        /* The copy is optimized later if not needed         */
 
-       temp = new_rd_Unknown(birg->irg, get_irn_mode(other_different));
-       cpy = be_new_Copy(cls, birg->irg, block, temp);
+       cpy = be_new_Copy(cls, birg->irg, block, other_different);
        be_node_set_flags(cpy, BE_OUT_POS(0), arch_irn_flags_dont_spill);
 
        in[0] = irn;
        in[1] = cpy;
 
-       /* Let the irn use the copy instead of the old other_different */
-       fix_in(irn, other_different, cpy);
-
        /* Add the Keep resp. CopyKeep and reroute the users */
        /* of the other_different irn in case of CopyKeep.   */
        if (get_n_out_edges(other_different) == 0) {
@@ -546,12 +562,36 @@ static void gen_assure_different_pattern(ir_node *irn, be_irg_t *birg, ir_node *
        else {
                keep = be_new_CopyKeep_single(cls, birg->irg, block, cpy, irn, get_irn_mode(other_different));
                be_node_set_reg_class(keep, 1, cls);
-               edges_reroute(other_different, keep, birg->irg);
        }
 
-       /* after rerouting: let the copy point to the other_different irn */
+       /* let the copy point to the other_different irn */
        set_irn_n(cpy, 0, other_different);
 
+       /* insert copy and keep into schedule */
+       assert(sched_is_scheduled(irn) && "need schedule to assure constraints");
+       sched_add_before(belower_skip_proj(irn), cpy);
+       sched_add_after(irn, keep);
+
+       /* insert the other different and it's copies into the set */
+       key.op         = other_different;
+       key.copies     = NULL;
+       entry          = pset_find(op_set, &key, nodeset_hash(other_different));
+
+       if (! entry) {
+               entry         = obstack_alloc(&env->obst, sizeof(*entry));
+               entry->copies = pset_new_ptr_default();
+               entry->op     = other_different;
+       }
+
+       /* insert copy */
+       pset_insert_ptr(entry->copies, cpy);
+
+       /* insert keep in case of CopyKeep */
+       if (be_is_CopyKeep(keep))
+               pset_insert_ptr(entry->copies, keep);
+
+       pset_insert(op_set, entry, nodeset_hash(other_different));
+
        DBG((mod, LEVEL_1, "created %+F for %+F to assure should_be_different\n", keep, irn));
 }
 
@@ -559,22 +599,20 @@ static void gen_assure_different_pattern(ir_node *irn, be_irg_t *birg, ir_node *
  * Checks if node has a should_be_different constraint in output
  * and adds a Keep then to assure the constraint.
  */
-static void assure_different_constraints(ir_node *irn, be_irg_t *birg) {
-       const arch_env_t          *arch_env = birg->main_env->arch_env;
+static void assure_different_constraints(ir_node *irn, constraint_env_t *env) {
        const arch_register_req_t *req;
-       arch_register_req_t        req_temp;
-       int i, n;
+       arch_register_req_t       req_temp;
 
-       req = arch_get_register_req(arch_env, &req_temp, irn, -1);
+       req = arch_get_register_req(env->birg->main_env->arch_env, &req_temp, irn, -1);
 
        if (req) {
                if (arch_register_req_is(req, should_be_different)) {
-                       gen_assure_different_pattern(irn, birg, req->other_different);
+                       gen_assure_different_pattern(irn, req->other_different, env);
                }
                else if (arch_register_req_is(req, should_be_different_from_all)) {
-                       n = get_irn_arity(belower_skip_proj(irn));
+                       int i, n = get_irn_arity(belower_skip_proj(irn));
                        for (i = 0; i < n; i++) {
-                               gen_assure_different_pattern(irn, birg, get_irn_n(belower_skip_proj(irn), i));
+                               gen_assure_different_pattern(irn, get_irn_n(belower_skip_proj(irn), i), env);
                        }
                }
        }
@@ -606,7 +644,71 @@ static void assure_constraints_walker(ir_node *irn, void *walk_env) {
  * @param birg  The birg structure containing the irg
  */
 void assure_constraints(be_irg_t *birg) {
-       irg_walk_blkwise_graph(birg->irg, NULL, assure_constraints_walker, birg);
+       constraint_env_t cenv;
+       op_copy_assoc_t  *entry;
+       dom_front_info_t *dom;
+       ir_node          **nodes;
+       FIRM_DBG_REGISTER(firm_dbg_module_t *mod, "firm.be.lower");
+
+       cenv.birg   = birg;
+       cenv.op_set = new_pset(cmp_op_copy_assoc, 16);
+       obstack_init(&cenv.obst);
+
+       irg_walk_blkwise_graph(birg->irg, NULL, assure_constraints_walker, &cenv);
+
+       /* introduce copies needs dominance information */
+       dom = be_compute_dominance_frontiers(birg->irg);
+
+       /* for all */
+       foreach_pset(cenv.op_set, entry) {
+               int     n;
+               ir_node *cp;
+
+               n     = pset_count(entry->copies);
+               nodes = alloca((n + 1) * sizeof(nodes[0]));
+
+               /* put the node in an array */
+               n          = 0;
+               nodes[n++] = entry->op;
+               DBG((mod, LEVEL_1, "introduce copies for %+F ", entry->op));
+
+               /* collect all copies */
+               foreach_pset(entry->copies, cp) {
+                       nodes[n++] = cp;
+                       DB((mod, LEVEL_1, ", %+F ", cp));
+               }
+
+               DB((mod, LEVEL_1, "\n"));
+
+               /* introduce the copies for the operand and it's copies */
+               be_ssa_constr(dom, NULL, n, nodes);
+
+
+               /* Could be that not all CopyKeeps are really needed, */
+               /* so we transform unnecessary ones into Keeps.       */
+               foreach_pset(entry->copies, cp) {
+                       if (be_is_CopyKeep(cp) && get_irn_n_edges(cp) < 1) {
+                               ir_node *keep;
+                               int     n = get_irn_arity(cp);
+
+                               keep = be_new_Keep(arch_get_irn_reg_class(birg->main_env->arch_env, cp, -1),
+                                       birg->irg, get_nodes_block(cp), n, (ir_node **)&get_irn_in(cp)[1]);
+                               sched_add_before(cp, keep);
+
+                               /* Set all ins (including the block) of the CopyKeep BAD to keep the verifier happy. */
+                               while (--n >= -1)
+                                       set_irn_n(cp, n, new_Bad());
+                               sched_remove(cp);
+                       }
+               }
+
+               del_pset(entry->copies);
+       }
+
+       be_free_dominance_frontiers(dom);
+
+       del_pset(cenv.op_set);
+       obstack_free(&cenv.obst, NULL);
 }