Use @version tag
[libfirm] / ir / ana / irconsconfirm.c
index 57b8f3c..1198034 100644 (file)
@@ -24,9 +24,7 @@
  * @date     6.2005
  * @version  $Id$
  */
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
+#include "config.h"
 
 #include "irgraph_t.h"
 #include "irnode_t.h"
@@ -37,6 +35,7 @@
 #include "irgwalk.h"
 #include "irprintf.h"
 #include "irgopt.h"
+#include "irpass.h"
 #include "irtools.h"
 #include "array_t.h"
 #include "debug.h"
  * Walker environment.
  */
 typedef struct _env_t {
-       unsigned num_confirms;  /**< number of inserted Confirm nodes */
-       unsigned num_consts;    /**< number of constants placed */
-       unsigned num_eq;        /**< number of equalities placed */
+       unsigned num_confirms;  /**< Number of inserted Confirm nodes. */
+       unsigned num_consts;    /**< Number of constants placed. */
+       unsigned num_eq;        /**< Number of equalities placed. */
+       unsigned num_non_null;  /**< Number of non-null Confirms. */
 } env_t;
 
 /** The debug handle. */
@@ -109,12 +109,12 @@ static void handle_case(ir_node *block, ir_node *irn, long nr, env_t *env) {
                                ir_mode *mode = get_irn_mode(irn);
                                ir_type *tp   = get_irn_type(irn);
                                tarval *tv    = new_tarval_from_long(nr, mode);
-                               c = new_r_Const_type(current_ir_graph, block, mode, tv, tp);
+                               c = new_r_Const_type(current_ir_graph, tv, tp);
                        }
 
                        set_irn_n(succ, pos, c);
                        DBG_OPT_CONFIRM_C(irn, c);
-//                     ir_printf("1 Replacing input %d of node %n with %n\n", pos, succ, c);
+                       DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, succ, c));
 
                        env->num_consts += 1;
                }
@@ -147,13 +147,13 @@ static void handle_modeb(ir_node *block, ir_node *selector, pn_Cond pnc, env_t *
                         * We can replace the input with true/false.
                         */
                        if (con == NULL) {
-                               con = new_Const(mode_b, pnc == pn_Cond_true ? tarval_b_true : tarval_b_false);
+                               con = new_Const(pnc == pn_Cond_true ? tarval_b_true : tarval_b_false);
                        }
                        old = get_irn_n(user, pos);
                        set_irn_n(user, pos, con);
                        DBG_OPT_CONFIRM_C(old, con);
 
-                       DB((dbg, LEVEL_2, "Replacing input %d of node %n with %n\n", pos, user, con));
+                       DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, user, con));
 
                        env->num_consts += 1;
                } else {
@@ -208,12 +208,15 @@ static void handle_modeb(ir_node *block, ir_node *selector, pn_Cond pnc, env_t *
                                NEW_ARR_A(ir_node *, in, n);
                                /* ok, ALL predecessors are either dominated by block OR other block */
                                if (c_b == NULL) {
-                                       ir_node *c_true  = new_Const(mode_b, tarval_b_true);
-                                       ir_node *c_false = new_Const(mode_b, tarval_b_false);
-                                       c_b = new_r_Confirm(current_ir_graph, cond_block, selector,
-                                               pnc == pn_Cond_true ? c_true : c_false, pn_Cmp_Eq);
-                                       c_o = new_r_Confirm(current_ir_graph, cond_block, selector,
-                                               pnc == pn_Cond_false ? c_true : c_false, pn_Cmp_Eq);
+                                       ir_node *c_true  = new_Const(tarval_b_true);
+                                       ir_node *c_false = new_Const(tarval_b_false);
+                                       if (pnc == pn_Cond_true) {
+                                               c_b = c_true;
+                                               c_o = c_false;
+                                       } else {
+                                               c_b = c_false;
+                                               c_o = c_true;
+                                       }
                                }
                                for (i = n - 1; i >= 0; --i) {
                                        ir_node *pred_blk = get_Block_cfgpred_block(user_blk, i);
@@ -223,7 +226,7 @@ static void handle_modeb(ir_node *block, ir_node *selector, pn_Cond pnc, env_t *
                                        else
                                                in[i] = c_o;
                                }
-                               phi = new_r_Phi(current_ir_graph, user_blk, n, in, mode_b);
+                               phi = new_r_Phi(user_blk, n, in, mode_b);
                                set_irn_n(user, pos, phi);
                        }
                }
@@ -287,7 +290,7 @@ static void handle_if(ir_node *block, ir_node *cmp, pn_Cmp pnc, env_t *env) {
                                set_irn_n(user, pos, right);
                                DBG_OPT_CONFIRM(left, right);
 
-                               DB((dbg, LEVEL_2, "Replacing input %d of node %n with %n\n", pos, user, right));
+                               DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, user, right));
 
                                env->num_eq += 1;
                        } else if (block_dominates(blk, cond_block)) {
@@ -330,12 +333,11 @@ static void handle_if(ir_node *block, ir_node *cmp, pn_Cmp pnc, env_t *env) {
        } else { /* not pn_Cmp_Eq cases */
                ir_node *c = NULL;
 
-               for (edge = get_irn_out_edge_first(left); edge; edge = next) {
+               foreach_out_edge_safe(left, edge, next) {
                        ir_node *succ = get_edge_src_irn(edge);
                        int     pos   = get_edge_src_pos(edge);
                        ir_node *blk  = get_effective_use_block(succ, pos);
 
-                       next = get_irn_out_edge_next(left, edge);
                        if (block_dominates(block, blk)) {
                                /*
                                 * Ok, we found a usage of left in a block
@@ -343,15 +345,49 @@ static void handle_if(ir_node *block, ir_node *cmp, pn_Cmp pnc, env_t *env) {
                                 * We can replace the input with a Confirm(left, pnc, right).
                                 */
                                if (! c)
-                                       c = new_r_Confirm(current_ir_graph, block, left, right, pnc);
+                                       c = new_r_Confirm(block, left, right, pnc);
 
                                pos = get_edge_src_pos(edge);
                                set_irn_n(succ, pos, c);
-                               DB((dbg, LEVEL_2, "Replacing input %d of node %n with %n\n", pos, succ, c));
+                               DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, succ, c));
 
                                env->num_confirms += 1;
                        }
                }
+
+               if (! is_Const(right)) {
+                       /* also construct inverse Confirms */
+                       ir_node *rc = NULL;
+
+                       pnc = get_inversed_pnc(pnc);
+                       foreach_out_edge_safe(right, edge, next) {
+                               ir_node *succ = get_edge_src_irn(edge);
+                               int     pos;
+                               ir_node *blk;
+
+                               if (succ == c)
+                                       continue;
+
+                               pos  = get_edge_src_pos(edge);
+                               blk  = get_effective_use_block(succ, pos);
+
+                               if (block_dominates(block, blk)) {
+                                       /*
+                                        * Ok, we found a usage of right in a block
+                                        * dominated by the branch block.
+                                        * We can replace the input with a Confirm(right, pnc^-1, left).
+                                        */
+                                       if (! rc)
+                                               rc = new_r_Confirm(block, right, left, pnc);
+
+                                       pos = get_edge_src_pos(edge);
+                                       set_irn_n(succ, pos, rc);
+                                       DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, succ, rc));
+
+                                       env->num_confirms += 1;
+                               }
+                       }
+               }
        }
 }  /* handle_if */
 
@@ -398,6 +434,7 @@ static void insert_Confirm_in_block(ir_node *block, void *env) {
 
                if (get_Proj_proj(proj) != pn_Cond_true) {
                        /* it's the false branch */
+                       mode = get_irn_mode(get_Cmp_left(cmp));
                        pnc = get_negated_pnc(pnc, mode);
                }
                DB((dbg, LEVEL_2, "At %+F using %+F Confirm %=\n", block, cmp, pnc));
@@ -407,7 +444,7 @@ static void insert_Confirm_in_block(ir_node *block, void *env) {
                long proj_nr = get_Proj_proj(proj);
 
                /* this is a CASE, but we cannot handle the default case */
-               if (proj_nr == get_Cond_defaultProj(cond))
+               if (proj_nr == get_Cond_default_proj(cond))
                        return;
 
                handle_case(block, get_Cond_selector(cond), proj_nr, env);
@@ -418,16 +455,9 @@ static void insert_Confirm_in_block(ir_node *block, void *env) {
  * Checks if a node is a non-null Confirm.
  */
 static int is_non_null_Confirm(const ir_node *ptr) {
-       /*
-        * While a SymConst is not a Confirm, it is non-null
-        * anyway. This helps to reduce the number of
-        * constructed Confirms.
-        */
-       if (is_SymConst_addr_ent(ptr))
-               return 1;
        for (;;) {
                if (! is_Confirm(ptr))
-                       return 0;
+                       break;
                if (get_Confirm_cmp(ptr) == pn_Cmp_Lg) {
                        ir_node *bound = get_Confirm_bound(ptr);
 
@@ -436,6 +466,14 @@ static int is_non_null_Confirm(const ir_node *ptr) {
                }
                ptr = get_Confirm_value(ptr);
        }
+       /*
+        * While a SymConst is not a Confirm, it is non-null
+        * anyway. This helps to reduce the number of
+        * constructed Confirms.
+        */
+       if (is_SymConst_addr_ent(ptr))
+               return 1;
+       return 0;
 }  /* is_non_null_Confirm */
 
 /**
@@ -455,24 +493,14 @@ static void insert_non_null(ir_node *ptr, ir_node *block, env_t *env) {
                ir_node *blk;
 
 
-               if (is_Confirm(succ)) {
-                       /* beware of loops */
+               /* for now, we place a Confirm only in front of a Cmp */
+               if (! is_Cmp(succ))
                        continue;
-               }
-
-               if ((is_Load(succ) || is_Store(succ)) &&
-                       get_nodes_block(succ) == block) {
-                       /* Ignore Loads and Store in the same block for now,
-                          because we are not sure if they are dominated.
-                          This is not a bad restriction: if exception flow is
-                          present, they are in other blocks either. */
-                       continue;
-               }
 
                pos = get_edge_src_pos(edge);
                blk = get_effective_use_block(succ, pos);
 
-               if (block_dominates(block, blk) && !is_non_null_Confirm(get_irn_n(succ, pos))) {
+               if (block_dominates(block, blk)) {
                        /*
                         * Ok, we found a usage of ptr in a block
                         * dominated by the Load/Store block.
@@ -480,15 +508,15 @@ static void insert_non_null(ir_node *ptr, ir_node *block, env_t *env) {
                         */
                        if (c == NULL) {
                                ir_mode *mode = get_irn_mode(ptr);
-                               c = new_Const(mode, get_mode_null(mode));
+                               c = new_Const(get_mode_null(mode));
 
-                               c = new_r_Confirm(current_ir_graph, block, ptr, c, pn_Cmp_Lg);
+                               c = new_r_Confirm(block, ptr, c, pn_Cmp_Lg);
                        }
 
                        set_irn_n(succ, pos, c);
-                       DB((dbg, LEVEL_2, "Replacing input %d of node %n with %n\n", pos, succ, c));
-
+                       DB((dbg, LEVEL_2, "Replacing input %d of node %+F with %+F\n", pos, succ, c));
 
+                       env->num_non_null += 1;
                        env->num_confirms += 1;
                }
        }
@@ -545,6 +573,7 @@ void construct_confirms(ir_graph *irg) {
        env.num_confirms = 0;
        env.num_consts   = 0;
        env.num_eq       = 0;
+       env.num_non_null = 0;
 
        if (get_opt_global_null_ptr_elimination()) {
                /* do global NULL test elimination */
@@ -565,12 +594,18 @@ void construct_confirms(ir_graph *irg) {
        DB((dbg, LEVEL_1, "# Confirms inserted : %u\n", env.num_confirms));
        DB((dbg, LEVEL_1, "# Const replacements: %u\n", env.num_consts));
        DB((dbg, LEVEL_1, "# node equalities   : %u\n", env.num_eq));
+       DB((dbg, LEVEL_1, "# non-null Confirms : %u\n", env.num_non_null));
 
        /* deactivate edges if they where off */
        if (! edges_active)
                edges_deactivate(irg);
 }  /* construct_confirms */
 
+/* Construct a pass. */
+ir_graph_pass_t *construct_confirms_pass(const char *name) {
+       return def_graph_pass(name ? name : "confirm", construct_confirms);
+}  /* construct_confirms_pass */
+
 #if 0
 /**
  * Post-walker: Remove Confirm nodes
@@ -602,3 +637,8 @@ void remove_confirms(ir_graph *irg) {
        optimize_graph_df(irg);
        set_opt_remove_confirm(rem);
 }  /* remove_confirms */
+
+/* Construct a pass. */
+ir_graph_pass_t *remove_confirms_pass(const char *name) {
+       return def_graph_pass(name ? name : "rem_confirm", remove_confirms);
+}  /* remove_confirms_pass */