X-Git-Url: http://nsz.repo.hu/git/?a=blobdiff_plain;f=ir%2Fopt%2Fboolopt.c;h=eaae5a39e49be89c48f77ce14c6b6adee4c7d11c;hb=c8a000c8803aa2789a92206d888712b4577cb87a;hp=c404e650324e17ef906c4bad1494b42ff18467b7;hpb=d2e404e145ec50a200e9c27fee2f4de93fbc4fa0;p=libfirm

diff --git a/ir/opt/boolopt.c b/ir/opt/boolopt.c
index c404e6503..eaae5a39e 100644
--- a/ir/opt/boolopt.c
+++ b/ir/opt/boolopt.c
@@ -42,15 +42,15 @@
 
 /** Describes a pair of relative conditions lo < hi, lo pnc_lo x, hi pnc_hi x */
 typedef struct cond_pair {
-	ir_node *cmp_lo;  /**< The lo compare node. */
-	ir_node *cmp_hi;  /**< The hi compare node. */
-	pn_Cmp   pnc_lo;  /**< The lo relation node. */
-	pn_Cmp   pnc_hi;  /**< The hi relation node. */
-	ir_node *proj_lo; /**< The mode_b result proj of cmp_lo. */
-	ir_node *proj_hi; /**< The mode_b result proj of cmp_hi. */
-	tarval  *tv_lo;   /**< The tarval of cmp_lo node. */
-	tarval  *tv_hi;   /**< The tarval of cmp_hi node. */
-	ir_mode *lo_mode; /**< The mode of the cmp_lo operands. */
+	ir_node   *cmp_lo;  /**< The lo compare node. */
+	ir_node   *cmp_hi;  /**< The hi compare node. */
+	pn_Cmp     pnc_lo;  /**< The lo relation node. */
+	pn_Cmp     pnc_hi;  /**< The hi relation node. */
+	ir_node   *proj_lo; /**< The mode_b result proj of cmp_lo. */
+	ir_node   *proj_hi; /**< The mode_b result proj of cmp_hi. */
+	ir_tarval *tv_lo;   /**< The tarval of cmp_lo node. */
+	ir_tarval *tv_hi;   /**< The tarval of cmp_hi node. */
+	ir_mode   *lo_mode; /**< The mode of the cmp_lo operands. */
 } cond_pair;
 
 /** Environment for all walker in boolopt. */
@@ -75,16 +75,14 @@ static int find_cond_pair(ir_node *const l, ir_node *const r, cond_pair *const r
 			ir_node *const lor   = get_Cmp_right(lo);
 			ir_node *const rol   = get_Cmp_left(ro);
 			ir_node *const ror   = get_Cmp_right(ro);
-			pn_Cmp   const pnc_l = get_Proj_proj(l);
-			pn_Cmp   const pnc_r = get_Proj_proj(r);
+			pn_Cmp   const pnc_l = get_Proj_pn_cmp(l);
+			pn_Cmp   const pnc_r = get_Proj_pn_cmp(r);
 
 			if (is_Const(lor) && is_Const_null(lor) &&
 			    is_Const(ror) && is_Const_null(ror) &&
 			    pnc_l == pnc_r &&
 			    (pnc_l == pn_Cmp_Lg || pnc_l == pn_Cmp_Eq)) {
 				/* lo == (lol !=|== NULL) && ro == (rol !=|== NULL) */
-				DB((dbg, LEVEL_1, "found <null null>\n"));
-
 				res->cmp_lo  = lo;
 				res->cmp_hi  = ro;
 				res->pnc_lo  = pnc_l;
@@ -100,9 +98,9 @@ static int find_cond_pair(ir_node *const l, ir_node *const r, cond_pair *const r
 
 			if (lol == rol && lor != ror && is_Const(lor) && is_Const(ror)) {
 				/* lo == (x CMP c_l), ro == (x cmp c_r) */
-				tarval *const tv_l  = get_Const_tarval(lor);
-				tarval *const tv_r  = get_Const_tarval(ror);
-				pn_Cmp  const rel   = tarval_cmp(tv_l, tv_r);
+				ir_tarval *const tv_l  = get_Const_tarval(lor);
+				ir_tarval *const tv_r  = get_Const_tarval(ror);
+				pn_Cmp     const rel   = tarval_cmp(tv_l, tv_r);
 
 				res->lo_mode = get_irn_mode(lol);
 
@@ -141,23 +139,24 @@ static int find_cond_pair(ir_node *const l, ir_node *const r, cond_pair *const r
 /**
  * Handle (lo pnc_lo x) AND (hi pnc_hi x)
  */
-static ir_node *bool_and(cond_pair* const cpair)
+static ir_node *bool_and(cond_pair* const cpair, ir_node *dst_block)
 {
-	ir_node *const cmp_lo  = cpair->cmp_lo;
-	ir_node *const cmp_hi  = cpair->cmp_hi;
-	pn_Cmp         pnc_lo  = cpair->pnc_lo;
-	pn_Cmp   const pnc_hi  = cpair->pnc_hi;
-	ir_node *const proj_lo = cpair->proj_lo;
-	ir_node *const proj_hi = cpair->proj_hi;
-	tarval  *      tv_lo   = cpair->tv_lo;
-	tarval  *      tv_hi   = cpair->tv_hi;
-	ir_mode *      mode    = cpair->lo_mode;
+	ir_node    *const cmp_lo  = cpair->cmp_lo;
+	ir_node    *const cmp_hi  = cpair->cmp_hi;
+	pn_Cmp            pnc_lo  = cpair->pnc_lo;
+	pn_Cmp      const pnc_hi  = cpair->pnc_hi;
+	ir_node    *const proj_lo = cpair->proj_lo;
+	ir_node    *const proj_hi = cpair->proj_hi;
+	ir_tarval  *      tv_lo   = cpair->tv_lo;
+	ir_tarval  *      tv_hi   = cpair->tv_hi;
+	ir_mode    *      mode    = cpair->lo_mode;
+	ir_graph   *      irg     = get_irn_irg(cmp_lo);
 
 	if (pnc_lo == pn_Cmp_Eq && pnc_hi == pn_Cmp_Eq &&
 	    tarval_is_null(tv_lo) && tarval_is_null(tv_hi) &&
 	    mode == get_tarval_mode(tv_hi)) {
 		/* p == NULL && q == NULL ==> (p&q) == NULL) */
-		ir_node *block, *lol, *hil, *cmp, *c, *p;
+		ir_node *lol, *hil, *cmp, *c, *p;
 
 		if (mode_is_reference(mode)) {
 			mode = find_unsigned_mode(mode);
@@ -168,19 +167,22 @@ static ir_node *bool_and(cond_pair* const cpair)
 				return NULL;
 		}
 		if (mode_is_int(mode)) {
-			block = get_nodes_block(cmp_lo);
 			lol   = get_Cmp_left(cmp_lo);
-			lol   = new_r_Conv(block, lol, mode);
+			lol   = new_r_Conv(dst_block, lol, mode);
 			hil   = get_Cmp_left(cmp_hi);
-			hil   = new_r_Conv(block, hil, mode);
-			p     = new_r_And(block, lol, hil, mode);
-			c     = new_Const(tv_lo);
-			cmp   = new_r_Cmp(block, p, c);
-			p     = new_r_Proj(block, cmp, mode_b, pn_Cmp_Eq);
+			hil   = new_r_Conv(dst_block, hil, mode);
+			p     = new_r_And(dst_block, lol, hil, mode);
+			c     = new_r_Const(irg, tv_lo);
+			cmp   = new_r_Cmp(dst_block, p, c);
+			p     = new_r_Proj(cmp, mode_b, pn_Cmp_Eq);
 			return p;
 		}
 	}
 
+	/* the following tests expect one common operand */
+	if (get_Cmp_left(cmp_lo) !=  get_Cmp_left(cmp_hi))
+		return 0;
+
 	/* TODO: for now reject float modes */
 	if (! mode_is_int(mode))
 		return 0;
@@ -189,7 +191,7 @@ static ir_node *bool_and(cond_pair* const cpair)
 	if ((pnc_lo == pn_Cmp_Lt || pnc_lo == pn_Cmp_Le || pnc_lo == pn_Cmp_Eq) &&
 	    (pnc_hi == pn_Cmp_Eq || pnc_hi == pn_Cmp_Ge || pnc_hi == pn_Cmp_Gt)) {
 		/* x <|<=|== lo && x ==|>=|> hi ==> false */
-		ir_node *const t = new_Const(tarval_b_false);
+		ir_node *const t = new_r_Const(irg, tarval_b_false);
 		return t;
 	} else if ((pnc_lo == pn_Cmp_Lt || pnc_lo == pn_Cmp_Le || pnc_lo == pn_Cmp_Eq) &&
 	           (pnc_hi == pn_Cmp_Lt || pnc_hi == pn_Cmp_Le || pnc_hi == pn_Cmp_Lg)) {
@@ -202,29 +204,25 @@ static ir_node *bool_and(cond_pair* const cpair)
 	} else if (tarval_is_one(tarval_sub(tv_hi, tv_lo, NULL))) { /* lo + 1 == hi */
 		if (pnc_lo == pn_Cmp_Ge && pnc_hi == pn_Cmp_Lt) {
 			/* x >= c && x < c + 1 ==> x == c */
-			ir_node  *const block = get_nodes_block(cmp_lo);
-			ir_node  *const p = new_r_Proj(block, cmp_lo, mode_b, pn_Cmp_Eq);
+			ir_node  *const p = new_r_Proj(cmp_lo, mode_b, pn_Cmp_Eq);
 			return p;
 		} else if (pnc_lo == pn_Cmp_Gt) {
 			if (pnc_hi == pn_Cmp_Lg) {
 				/* x > c && x != c + 1 ==> x > c + 1 */
-				ir_node  *const block = get_nodes_block(cmp_hi);
-				ir_node  *const p = new_r_Proj(block, cmp_hi, mode_b, pn_Cmp_Gt);
+				ir_node  *const p = new_r_Proj(cmp_hi, mode_b, pn_Cmp_Gt);
 				return p;
 			} else if (pnc_hi == pn_Cmp_Lt) {
 				/* x > c && x < c + 1 ==> false */
-				ir_node *const t = new_Const(tarval_b_false);
+				ir_node *const t = new_r_Const(irg, tarval_b_false);
 				return t;
 			} else if (pnc_hi == pn_Cmp_Le) {
 				/* x > c && x <= c + 1 ==> x != c + 1 */
-				ir_node  *const block = get_nodes_block(cmp_hi);
-				ir_node  *const p = new_r_Proj(block, cmp_hi, mode_b, pn_Cmp_Eq);
+				ir_node  *const p = new_r_Proj(cmp_hi, mode_b, pn_Cmp_Eq);
 				return p;
 			}
 		} else if (pnc_lo == pn_Cmp_Lg && pnc_hi == pn_Cmp_Lt) {
 			/* x != c && c < c + 1 ==> x < c */
-			ir_node  *const block = get_nodes_block(cmp_lo);
-			ir_node  *const p     = new_r_Proj(block, cmp_lo, mode_b, pn_Cmp_Lt);
+			ir_node  *const p     = new_r_Proj(cmp_lo, mode_b, pn_Cmp_Lt);
 			return p;
 		}
 	} else if ((pnc_lo == pn_Cmp_Gt || pnc_lo == pn_Cmp_Ge) &&
@@ -234,8 +232,8 @@ static ir_node *bool_and(cond_pair* const cpair)
 		/* x >|\= lo && x <|<= hi ==> (x - lo) <u|<=u (hi-lo) */
 		if (pnc_lo == pn_Cmp_Gt) {
 			/* must convert to >= */
-			ir_mode *mode = get_tarval_mode(tv_lo);
-			tarval *n = tarval_add(tv_lo, get_mode_one(mode));
+			ir_mode   *mode = get_tarval_mode(tv_lo);
+			ir_tarval *n    = tarval_add(tv_lo, get_mode_one(mode));
 			if (n != tarval_bad && tarval_cmp(n, tv_lo) == pn_Cmp_Gt) {
 				/* no overflow */
 				tv_lo = n;
@@ -260,11 +258,11 @@ static ir_node *bool_and(cond_pair* const cpair)
 				if (tv_lo == tarval_bad || tv_hi == tarval_bad)
 					return NULL;
 			}
-			c    = new_Const(tv_lo);
+			c    = new_r_Const(irg, tv_lo);
 			sub  = new_r_Sub(block, x, c, mode);
-			subc = new_r_Sub(block, new_Const(tv_hi), c, mode);
+			subc = new_r_Sub(block, new_r_Const(irg, tv_hi), c, mode);
 			cmp  = new_r_Cmp(block, sub, subc);
-			p    = new_r_Proj(block, cmp, mode_b, pnc_hi);
+			p    = new_r_Proj(cmp, mode_b, pnc_hi);
 			return p;
 		}
 	}
@@ -274,23 +272,24 @@ static ir_node *bool_and(cond_pair* const cpair)
 /**
  * Handle (lo pnc_lo x) OR (hi pnc_hi x)
  */
-static ir_node *bool_or(cond_pair *const cpair)
+static ir_node *bool_or(cond_pair *const cpair, ir_node *dst_block)
 {
-	ir_node *const cmp_lo  = cpair->cmp_lo;
-	ir_node *const cmp_hi  = cpair->cmp_hi;
-	pn_Cmp         pnc_lo  = cpair->pnc_lo;
-	pn_Cmp   const pnc_hi  = cpair->pnc_hi;
-	ir_node *const proj_lo = cpair->proj_lo;
-	ir_node *const proj_hi = cpair->proj_hi;
-	tarval  *      tv_lo   = cpair->tv_lo;
-	tarval  *      tv_hi   = cpair->tv_hi;
-	ir_mode *      mode    = cpair->lo_mode;
+	ir_node   *const cmp_lo  = cpair->cmp_lo;
+	ir_node   *const cmp_hi  = cpair->cmp_hi;
+	pn_Cmp           pnc_lo  = cpair->pnc_lo;
+	pn_Cmp     const pnc_hi  = cpair->pnc_hi;
+	ir_node   *const proj_lo = cpair->proj_lo;
+	ir_node   *const proj_hi = cpair->proj_hi;
+	ir_tarval *      tv_lo   = cpair->tv_lo;
+	ir_tarval *      tv_hi   = cpair->tv_hi;
+	ir_mode   *      mode    = cpair->lo_mode;
+	ir_graph  *      irg     = get_irn_irg(cmp_lo);
 
 	if (pnc_lo == pn_Cmp_Lg && pnc_hi == pn_Cmp_Lg &&
 		tarval_is_null(tv_lo) && tarval_is_null(tv_hi) &&
 		mode == get_tarval_mode(tv_hi)) {
 		/* p != NULL || q != NULL ==> (p|q) != NULL) */
-		ir_node *block, *lol, *hil, *cmp, *c, *p;
+		ir_node *lol, *hil, *cmp, *c, *p;
 
 		if (mode_is_reference(mode)) {
 			mode = find_unsigned_mode(mode);
@@ -301,19 +300,22 @@ static ir_node *bool_or(cond_pair *const cpair)
 				return NULL;
 		}
 		if (mode_is_int(mode)) {
-			block = get_nodes_block(cmp_lo);
 			lol   = get_Cmp_left(cmp_lo);
-			lol   = new_r_Conv(block, lol, mode);
+			lol   = new_r_Conv(dst_block, lol, mode);
 			hil   = get_Cmp_left(cmp_hi);
-			hil   = new_r_Conv(block, hil, mode);
-			p     = new_r_Or(block, lol, hil, mode);
-			c     = new_Const(tv_lo);
-			cmp   = new_r_Cmp(block, p, c);
-			p     = new_r_Proj(block, cmp, mode_b, pn_Cmp_Lg);
+			hil   = new_r_Conv(dst_block, hil, mode);
+			p     = new_r_Or(dst_block, lol, hil, mode);
+			c     = new_r_Const(irg, tv_lo);
+			cmp   = new_r_Cmp(dst_block, p, c);
+			p     = new_r_Proj(cmp, mode_b, pn_Cmp_Lg);
 			return p;
 		}
 	}
 
+	/* the following tests expect one common operand */
+	if (get_Cmp_left(cmp_lo) !=  get_Cmp_left(cmp_hi))
+		return 0;
+
 	/* TODO: for now reject float modes */
 	if (! mode_is_int(mode))
 		return 0;
@@ -322,7 +324,7 @@ static ir_node *bool_or(cond_pair *const cpair)
 	if ((pnc_lo == pn_Cmp_Ge || pnc_lo == pn_Cmp_Gt || pnc_lo == pn_Cmp_Lg) &&
 	    (pnc_hi == pn_Cmp_Lt || pnc_hi == pn_Cmp_Le || pnc_hi == pn_Cmp_Lg)) {
 		/* x >=|>|!= lo | x <|<=|!= hi ==> true */
-		ir_node *const t = new_Const(tarval_b_true);
+		ir_node *const t = new_r_Const(irg, tarval_b_true);
 		return t;
 	} else if ((pnc_lo == pn_Cmp_Lt || pnc_lo == pn_Cmp_Le || pnc_lo == pn_Cmp_Eq) &&
 	           (pnc_hi == pn_Cmp_Lt || pnc_hi == pn_Cmp_Le || pnc_hi == pn_Cmp_Lg)) {
@@ -335,47 +337,43 @@ static ir_node *bool_or(cond_pair *const cpair)
 	} else if (tarval_is_one(tarval_sub(tv_hi, tv_lo, NULL))) { /* lo + 1 == hi */
 		if (pnc_lo == pn_Cmp_Lt && pnc_hi == pn_Cmp_Ge) {
 			/* x < c || x >= c + 1 ==> x != c */
-			ir_node  *const block = get_nodes_block(cmp_lo);
-			ir_node  *const p = new_r_Proj(block, cmp_lo, mode_b, pn_Cmp_Lg);
+			ir_node  *const p = new_r_Proj(cmp_lo, mode_b, pn_Cmp_Lg);
 			return p;
 		} else if (pnc_lo == pn_Cmp_Le) {
 			if (pnc_hi == pn_Cmp_Eq) {
 				/* x <= c || x == c + 1 ==> x <= c + 1 */
-				ir_node  *const block = get_nodes_block(cmp_hi);
-				ir_node  *const p = new_r_Proj(block, cmp_hi, mode_b, pn_Cmp_Le);
+				ir_node  *const p = new_r_Proj(cmp_hi, mode_b, pn_Cmp_Le);
 				return p;
 			} else if (pnc_hi == pn_Cmp_Ge) {
 				/* x <= c || x >= c + 1 ==> true */
-				ir_node *const t = new_Const(tarval_b_true);
+				ir_node *const t = new_r_Const(irg, tarval_b_true);
 				return t;
 			} else if (pnc_hi == pn_Cmp_Gt) {
 				/* x <= c || x > c + 1 ==> x != c + 1 */
-				ir_node  *const block = get_nodes_block(cmp_hi);
-				ir_node  *const p = new_r_Proj(block, cmp_hi, mode_b, pn_Cmp_Lg);
+				ir_node  *const p = new_r_Proj(cmp_hi, mode_b, pn_Cmp_Lg);
 				return p;
 			}
 		} else if (pnc_lo == pn_Cmp_Eq && pnc_hi == pn_Cmp_Ge) {
 			/* x == c || x >= c + 1 ==> x >= c */
-			ir_node  *const block = get_nodes_block(cmp_lo);
-			ir_node  *const p     = new_r_Proj(block, cmp_lo, mode_b, pn_Cmp_Ge);
+			ir_node  *const p     = new_r_Proj(cmp_lo, mode_b, pn_Cmp_Ge);
 			return p;
 		}
 	} else if ((pnc_lo == pn_Cmp_Lt || pnc_lo == pn_Cmp_Le) &&
 	           (pnc_hi == pn_Cmp_Gt || pnc_lo == pn_Cmp_Ge) &&
 	           get_mode_arithmetic(mode) == irma_twos_complement) {
 		/* works for two-complements only */
-		/* x <|\= lo  || x >|>= hi ==> (x - lo) >u|>=u (hi-lo) */
-		if (pnc_lo == pn_Cmp_Lt) {
-			/* must convert to <= */
-			ir_mode *mode = get_tarval_mode(tv_lo);
-			tarval *n = tarval_sub(tv_lo, get_mode_one(mode), NULL);
-			if (n != tarval_bad && tarval_cmp(n, tv_lo) == pn_Cmp_Lt) {
+		/* x <|<= lo  || x >|>= hi ==> (x - lo) >u|>=u (hi-lo) */
+		if (pnc_lo == pn_Cmp_Le) {
+			/* must convert to < */
+			ir_mode   *mode = get_tarval_mode(tv_lo);
+			ir_tarval *n    = tarval_add(tv_lo, get_mode_one(mode));
+			if (n != tarval_bad && tarval_cmp(n, tv_lo) == pn_Cmp_Gt) {
 				/* no overflow */
 				tv_lo = n;
-				pnc_lo = pn_Cmp_Le;
+				pnc_lo = pn_Cmp_Lt;
 			}
 		}
-		if (pnc_lo == pn_Cmp_Le) {
+		if (pnc_lo == pn_Cmp_Lt) {
 			/* all fine */
 			ir_node *const block = get_nodes_block(cmp_hi);
 			ir_node *      x     = get_Cmp_left(cmp_hi);
@@ -393,11 +391,11 @@ static ir_node *bool_or(cond_pair *const cpair)
 				if (tv_lo == tarval_bad || tv_hi == tarval_bad)
 					return NULL;
 			}
-			c    = new_Const(tv_lo);
+			c    = new_r_Const(irg, tv_lo);
 			sub  = new_r_Sub(block, x, c, mode);
-			subc = new_r_Sub(block, new_Const(tv_hi), c, mode);
+			subc = new_r_Sub(block, new_r_Const(irg, tv_hi), c, mode);
 			cmp  = new_r_Cmp(block, sub, subc);
-			p    = new_r_Proj(block, cmp, mode_b, pnc_hi);
+			p    = new_r_Proj(cmp, mode_b, pnc_hi);
 			return p;
 		}
 	}
@@ -409,7 +407,7 @@ static ir_node *bool_or(cond_pair *const cpair)
  */
 static void bool_walk(ir_node *n, void *ctx)
 {
-	bool_opt_env_t *env = ctx;
+	bool_opt_env_t *env = (bool_opt_env_t*)ctx;
 
 	if (get_irn_mode(n) != mode_b)
 		return;
@@ -421,7 +419,7 @@ static void bool_walk(ir_node *n, void *ctx)
 		cond_pair      cpair;
 		if (!find_cond_pair(l, r, &cpair))
 			return;
-		replacement = bool_and(&cpair);
+		replacement = bool_and(&cpair, get_nodes_block(n));
 		if (replacement) {
 			exchange(n, replacement);
 			env->changed = 1;
@@ -433,7 +431,7 @@ static void bool_walk(ir_node *n, void *ctx)
 		cond_pair      cpair;
 		if (!find_cond_pair(l, r, &cpair))
 			return;
-		replacement = bool_or(&cpair);
+		replacement = bool_or(&cpair, get_nodes_block(n));
 		if (replacement) {
 			exchange(n, replacement);
 			env->changed = 1;
@@ -505,7 +503,8 @@ static ir_node *skip_empty_blocks(ir_node *node)
  * This can be done, if block contains no Phi node that depends on
  * different inputs idx_i and idx_j.
  */
-static int can_fuse_block_inputs(const ir_node *block, int idx_i, int idx_j) {
+static int can_fuse_block_inputs(const ir_node *block, int idx_i, int idx_j)
+{
 	const ir_node *phi;
 
 	for (phi = get_Block_phis(block); phi != NULL; phi = get_Phi_next(phi)) {
@@ -554,7 +553,8 @@ static void remove_block_input(ir_node *block, int idx)
  * Under the preposition that we have a chain of blocks from
  * from_block to to_block, collapse them all into to_block.
  */
-static void move_nodes_to_block(ir_node *jmp, ir_node *to_block) {
+static void move_nodes_to_block(ir_node *jmp, ir_node *to_block)
+{
 	ir_node *new_jmp = NULL;
 	ir_node *block, *next_block;
 
@@ -586,10 +586,17 @@ static void move_nodes_to_block(ir_node *jmp, ir_node *to_block) {
  */
 static void find_cf_and_or_walker(ir_node *block, void *ctx)
 {
+	bool_opt_env_t *env = (bool_opt_env_t*)ctx;
 	int low_idx, up_idx;
-	int n_cfgpreds = get_Block_n_cfgpreds(block);
-	bool_opt_env_t *env = ctx;
+	int n_cfgpreds;
 
+	/* because we modify the graph in regions we might not visited yet,
+	 * Id nodes might arise here. Ignore them.
+	 */
+	if (is_Id(block))
+		return;
+
+	n_cfgpreds = get_Block_n_cfgpreds(block);
 restart:
 	if (n_cfgpreds < 2)
 		return;
@@ -639,6 +646,8 @@ restart:
 			upper_block = get_nodes_block(upper_cf);
 			if (upper_block != lower_pred)
 				continue;
+			if (!block_dominates(upper_block, block))
+				continue;
 
 			assert(is_Proj(upper_cf));
 			upper_cond = get_Proj_pred(upper_cf);
@@ -663,42 +672,41 @@ restart:
 				if (cpair.proj_lo == cond_selector) {
 					ir_mode *mode  = get_tarval_mode(cpair.tv_lo);
 					ir_node *cmp   = get_Proj_pred(cpair.proj_lo);
-					ir_node *block = get_nodes_block(cmp);
 					cpair.pnc_lo   = get_negated_pnc(cpair.pnc_lo, mode);
-					cpair.proj_lo  = new_r_Proj(block, cmp, mode_b, cpair.pnc_lo);
+					cpair.proj_lo  = new_r_Proj(cmp, mode_b, cpair.pnc_lo);
 				} else {
 					ir_mode *mode  = get_tarval_mode(cpair.tv_hi);
 					ir_node *cmp   = get_Proj_pred(cpair.proj_hi);
-					ir_node *block = get_nodes_block(cmp);
 					assert(cpair.proj_hi == cond_selector);
 					cpair.pnc_hi   = get_negated_pnc(cpair.pnc_hi, mode);
-					cpair.proj_hi  = new_r_Proj(block, cmp, mode_b, cpair.pnc_hi);
+					cpair.proj_hi  = new_r_Proj(cmp, mode_b, cpair.pnc_hi);
 				}
 			}
 			if (get_Proj_proj(upper_cf) == pn_Cond_false) {
 				if (cpair.proj_lo == upper_cond_selector) {
 					ir_mode *mode  = get_tarval_mode(cpair.tv_lo);
 					ir_node *cmp   = get_Proj_pred(cpair.proj_lo);
-					ir_node *block = get_nodes_block(cmp);
 					cpair.pnc_lo   = get_negated_pnc(cpair.pnc_lo, mode);
-					cpair.proj_lo  = new_r_Proj(block, cmp, mode_b, cpair.pnc_lo);
+					cpair.proj_lo  = new_r_Proj(cmp, mode_b, cpair.pnc_lo);
 				} else {
 					ir_mode *mode  = get_tarval_mode(cpair.tv_hi);
 					ir_node *cmp   = get_Proj_pred(cpair.proj_hi);
-					ir_node *block = get_nodes_block(cmp);
 					assert(cpair.proj_hi == upper_cond_selector);
 					cpair.pnc_hi   = get_negated_pnc(cpair.pnc_hi, mode);
-					cpair.proj_hi  = new_r_Proj(block, cmp, mode_b, cpair.pnc_hi);
+					cpair.proj_hi  = new_r_Proj(cmp, mode_b, cpair.pnc_hi);
 				}
 			}
 
 			/* can we optimize the case? */
-			replacement = bool_or(&cpair);
+			replacement = bool_or(&cpair, upper_block);
 			if (replacement == NULL)
 				continue;
 
 			env->changed = 1;
 
+			DB((dbg, LEVEL_1, "boolopt: %+F: fusing (ub %+F lb %+F)\n",
+				get_irn_irg(upper_block), upper_block, lower_block));
+
 			/* move all expressions on the path to lower/upper block */
 			move_nodes_to_block(get_Block_cfgpred(block, up_idx), upper_block);
 			move_nodes_to_block(get_Block_cfgpred(block, low_idx), lower_block);
@@ -716,7 +724,6 @@ restart:
 			}
 			set_Cond_selector(cond, replacement);
 
-			DB((dbg, LEVEL_1, "%+F: replaced (ub %+F)\n", current_ir_graph, upper_block));
 			goto restart;
 		}
 	}