} /* 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) {
+ /*
+ * 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;
+ 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 */
+
+/**
+ * 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_Confirm(succ)) {
+ /* beware of loops */
+ continue;
+ }
+
if ((is_Load(succ) || is_Store(succ)) &&
get_nodes_block(succ) == block) {
/* Ignore Loads and Store in the same block for now,
pos = get_edge_src_pos(edge);
blk = get_effective_use_block(succ, pos);
- if (block_dominates(block, blk)) {
+ if (block_dominates(block, blk) && !is_non_null_Confirm(get_irn_n(succ, pos))) {
/*
* Ok, we found a usage of ptr in a block
* dominated by the Load/Store block.
}
} /* 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
*/
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());