* @date 6.2005
* @version $Id$
*/
-#ifdef HAVE_CONFIG_H
-# include "config.h"
-#endif
+#include "config.h"
#include "irgraph_t.h"
#include "irnode_t.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. */
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;
}
* 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 {
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);
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)) {
} 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
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(current_ir_graph, 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 */
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));
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);
} /* insert_Confirm_in_block */
/**
- * The given will be dereferenced, add non-null confirms.
+ * Checks if a node is a non-null Confirm.
+ */
+static int is_non_null_Confirm(const ir_node *ptr) {
+ for (;;) {
+ if (! is_Confirm(ptr))
+ break;
+ if (get_Confirm_cmp(ptr) == pn_Cmp_Lg) {
+ ir_node *bound = get_Confirm_bound(ptr);
+
+ if (is_Const(bound) && is_Const_null(bound))
+ return 1;
+ }
+ 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 */
+
+/**
+ * The given pointer will be dereferenced, add non-null Confirms.
*
* @param ptr a node representing an address
* @param block the block of the dereferencing instruction
ir_node *blk;
- 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. */
+ /* for now, we place a Confirm only in front of a Cmp */
+ if (! is_Cmp(succ))
continue;
- }
pos = get_edge_src_pos(edge);
blk = get_effective_use_block(succ, pos);
*/
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);
}
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;
}
}
} /* insert_non_null */
-/**
- * Checks if a node is a non-null Confirm.
- */
-static int is_non_null_Confirm(const ir_node *ptr) {
- for (;;) {
- if (! is_Confirm(ptr))
- return 0;
- if (get_Confirm_cmp(ptr) == pn_Cmp_Lg) {
- ir_node *bound = get_Confirm_bound(ptr);
-
- if (is_Const(bound) && is_Const_null(bound))
- return 1;
- }
- ptr = get_Confirm_value(ptr);
- }
-} /* is_non_null_Confirm */
-
/**
* Pre-walker: Called for every node to insert Confirm nodes
*/
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 */
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)
exchange(n, value);
else {
/*
- * Strange: a Confirm is it's own bound. This can happen
+ * Strange: a Confirm is its own bound. This can happen
* in dead blocks when Phi nodes are already removed.
*/
exchange(n, new_Bad());