Loop unrolling implemented. Inversion+unrolling fail 3 tests.
[libfirm] / ir / opt / combo.c
index b54f72b..e1b8abb 100644 (file)
@@ -81,7 +81,7 @@
 #include "array_t.h"
 #include "error.h"
 #include "irnodeset.h"
-
+#include "irpass.h"
 #include "tv_t.h"
 
 #include "irprintf.h"
@@ -196,7 +196,6 @@ typedef struct environment_t {
        partition_t     *touched;       /**< the touched set. */
        partition_t     *initial;       /**< The initial partition. */
        set             *opcode2id_map; /**< The opcodeMode->id map. */
-       pmap            *type2id_map;   /**< The type->id map. */
        ir_node         **kept_memory;  /**< Array of memory nodes that must be kept. */
        int             end_idx;        /**< -1 for local and 0 for global congruences. */
        int             lambda_input;   /**< Captured argument for lambda_partition(). */
@@ -666,7 +665,7 @@ static inline void add_to_worklist(partition_t *X, environment_t *env) {
  * @return a newly allocated partition
  */
 static inline partition_t *new_partition(environment_t *env) {
-       partition_t *part = obstack_alloc(&env->obst, sizeof(*part));
+       partition_t *part = OALLOC(&env->obst, partition_t);
 
        INIT_LIST_HEAD(&part->Leader);
        INIT_LIST_HEAD(&part->Follower);
@@ -725,7 +724,7 @@ static inline lattice_elem_t get_partition_type(const partition_t *X) {
  */
 static node_t *create_partition_node(ir_node *irn, partition_t *part, environment_t *env) {
        /* create a partition node and place it in the partition */
-       node_t *node = obstack_alloc(&env->obst, sizeof(*node));
+       node_t *node = OALLOC(&env->obst, node_t);
 
        INIT_LIST_HEAD(&node->node_list);
        INIT_LIST_HEAD(&node->cprop_list);
@@ -1439,13 +1438,10 @@ static void collect_touched(list_head *list, int idx, environment_t *env) {
 /**
  * Collect commutative nodes to the touched list.
  *
- * @param X     the partition of the list
  * @param list  the list which contains the nodes that must be evaluated
  * @param env   the environment
  */
-static void collect_commutative_touched(partition_t *X, list_head *list, environment_t *env) {
-       int     first      = 1;
-       int     both_input = 0;
+static void collect_commutative_touched(list_head *list, environment_t *env) {
        node_t  *x, *y;
 
        list_for_each_entry(node_t, x, list, node_list) {
@@ -1484,21 +1480,7 @@ static void collect_commutative_touched(partition_t *X, list_head *list, environ
                        /* Partitions of constants should not be split simply because their Nodes have unequal
                           functions or incongruent inputs. */
                        if (type_is_neither_top_nor_const(y->type)) {
-                               int    other_idx = edge->pos ^ 1;
-                               node_t *other    = get_irn_node(get_irn_n(succ, other_idx));
-                               int    equal     = X == other->part;
-
-                               /*
-                                * Note: op(a, a) is NOT congruent to op(a, b).
-                                * So, either all touch nodes must have both inputs congruent,
-                                * or not. We decide this by the first occurred node.
-                                */
-                               if (first) {
-                                       first      = 0;
-                                       both_input = equal;
-                               }
-                               if (both_input == equal)
-                                       add_to_touched(y, env);
+                               add_to_touched(y, env);
                        }
                }
        }
@@ -1526,13 +1508,16 @@ static void cause_splits(environment_t *env) {
                /* empty the touched set: already done, just clear the list */
                env->touched = NULL;
 
-               collect_commutative_touched(X, &X->Leader, env);
-               collect_commutative_touched(X, &X->Follower, env);
+               collect_commutative_touched(&X->Leader, env);
+               collect_commutative_touched(&X->Follower, env);
 
                for (Z = env->touched; Z != NULL; Z = N) {
-                       node_t   *e;
-                       node_t   *touched  = Z->touched;
-                       unsigned n_touched = Z->n_touched;
+                       node_t   *e, *n;
+                       node_t   *touched     = Z->touched;
+                       node_t   *touched_aa  = NULL;
+                       node_t   *touched_ab  = NULL;
+                       unsigned n_touched_aa = 0;
+                       unsigned n_touched_ab = 0;
 
                        assert(Z->touched != NULL);
 
@@ -1543,18 +1528,45 @@ static void cause_splits(environment_t *env) {
                        Z->on_touched = 0;
 
                        /* Empty local Z.touched. */
-                       for (e = touched; e != NULL; e = e->next) {
+                       for (e = touched; e != NULL; e = n) {
+                               node_t *left  = get_irn_node(get_irn_n(e->node, 0));
+                               node_t *right = get_irn_node(get_irn_n(e->node, 1));
+
                                assert(e->is_follower == 0);
                                e->on_touched = 0;
+                               n = e->next;
+
+                               /*
+                                * Note: op(a, a) is NOT congruent to op(a, b).
+                                * So, we must split the touched list.
+                                */
+                               if (left->part == right->part) {
+                                       e->next = touched_aa;
+                                       touched_aa = e;
+                                       ++n_touched_aa;
+                               } else {
+                                       e->next = touched_ab;
+                                       touched_ab = e;
+                                       ++n_touched_ab;
+                               }
                        }
+                       assert(n_touched_aa + n_touched_ab == Z->n_touched);
                        Z->touched   = NULL;
                        Z->n_touched = 0;
 
-                       if (0 < n_touched && n_touched < Z->n_leader) {
-                               DB((dbg, LEVEL_2, "Split part%d by touched\n", Z->nr));
-                               split(&Z, touched, env);
+                       if (0 < n_touched_aa && n_touched_aa < Z->n_leader) {
+                               partition_t *Z_prime = Z;
+                               DB((dbg, LEVEL_2, "Split part%d by touched_aa\n", Z_prime->nr));
+                               split(&Z_prime, touched_aa, env);
                        } else
-                               assert(n_touched <= Z->n_leader);
+                               assert(n_touched_aa <= Z->n_leader);
+
+                       if (0 < n_touched_ab && n_touched_ab < Z->n_leader) {
+                               partition_t *Z_prime = Z;
+                               DB((dbg, LEVEL_2, "Split part%d by touched_ab\n", Z_prime->nr));
+                               split(&Z_prime, touched_ab, env);
+                       } else
+                               assert(n_touched_ab <= Z->n_leader);
                }
        }
 
@@ -1964,7 +1976,7 @@ static void compute_Unknown(node_t *node) {
         * It would be safe to compute Top IF it can be assured, that only Cmp
         * nodes are inputs to Conds. We check that first.
         * This is the way Frontends typically build Firm, but some optimizations
-        * (cond_eval for instance) might replace them by Phib's...
+        * (jump threading for instance) might replace them by Phib's...
         */
        node->type.tv = tarval_UNKNOWN;
 }  /* compute_Unknown */
@@ -3467,7 +3479,6 @@ void combo(ir_graph *irg) {
        env.dbg_list       = NULL;
 #endif
        env.opcode2id_map  = new_set(cmp_opcode, iro_Last * 4);
-       env.type2id_map    = pmap_create();
        env.kept_memory    = NEW_ARR_F(ir_node *, 0);
        env.end_idx        = get_opt_global_cse() ? 0 : -1;
        env.lambda_input   = 0;
@@ -3559,7 +3570,6 @@ void combo(ir_graph *irg) {
        DEBUG_ONLY(set_dump_node_vcgattr_hook(NULL));
 
        DEL_ARR_F(env.kept_memory);
-       pmap_destroy(env.type2id_map);
        del_set(env.opcode2id_map);
        obstack_free(&env.obst, NULL);
 
@@ -3567,3 +3577,9 @@ void combo(ir_graph *irg) {
        set_value_of_func(NULL);
        current_ir_graph = rem;
 }  /* combo */
+
+/* Creates an ir_graph pass for combo. */
+ir_graph_pass_t *combo_pass(const char *name)
+{
+       return def_graph_pass(name ? name : "combo", combo);
+}  /* combo_pass */