typo fixed
[libfirm] / ir / ir / iropt.c
index 8241810..3bcf548 100644 (file)
@@ -119,7 +119,7 @@ static tarval *computed_value_Add(ir_node *n) {
        tarval *ta = value_of(a);
        tarval *tb = value_of(b);
 
-       if ((ta != tarval_bad) && (tb != tarval_bad) && (get_irn_mode(a) == get_irn_mode(b)))
+       if ((ta != tarval_bad) && (tb != tarval_bad))
                return tarval_add(ta, tb);
 
        return tarval_bad;
@@ -130,20 +130,21 @@ static tarval *computed_value_Add(ir_node *n) {
  * Special case: a - a
  */
 static tarval *computed_value_Sub(ir_node *n) {
-       ir_node *a = get_Sub_left(n);
-       ir_node *b = get_Sub_right(n);
-       tarval *ta;
-       tarval *tb;
+       ir_mode *mode = get_irn_mode(n);
+       ir_node *a    = get_Sub_left(n);
+       ir_node *b    = get_Sub_right(n);
+       tarval  *ta;
+       tarval  *tb;
 
        /* a - a */
        if (a == b && !is_Bad(a))
-               return get_mode_null(get_irn_mode(n));
+               return get_mode_null(mode);
 
        ta = value_of(a);
        tb = value_of(b);
 
-       if ((ta != tarval_bad) && (tb != tarval_bad) && (get_irn_mode(a) == get_irn_mode(b)))
-               return tarval_sub(ta, tb);
+       if ((ta != tarval_bad) && (tb != tarval_bad))
+               return tarval_sub(ta, tb, mode);
 
        return tarval_bad;
 }  /* computed_value_Sub */
@@ -233,74 +234,6 @@ static tarval *computed_value_Mul(ir_node *n) {
        return tarval_bad;
 }  /* computed_value_Mul */
 
-/**
- * Return the value of a floating point Quot.
- */
-static tarval *computed_value_Quot(ir_node *n) {
-       ir_node *a = get_Quot_left(n);
-       ir_node *b = get_Quot_right(n);
-
-       tarval *ta = value_of(a);
-       tarval *tb = value_of(b);
-
-       /* This was missing in original implementation. Why? */
-       if ((ta != tarval_bad) && (tb != tarval_bad) && (get_irn_mode(a) == get_irn_mode(b))) {
-               if (tb != get_mode_null(get_tarval_mode(tb)))   /* div by zero: return tarval_bad */
-                       return tarval_quo(ta, tb);
-       }
-       return tarval_bad;
-}  /* computed_value_Quot */
-
-/**
- * Calculate the value of an integer Div of two nodes.
- * Special case: 0 / b
- */
-static tarval *do_computed_value_Div(ir_node *a, ir_node *b) {
-       tarval *ta = value_of(a);
-       tarval *tb = value_of(b);
-
-       /* Compute c1 / c2 or 0 / a, a != 0 */
-       if (ta != tarval_bad) {
-               if ((tb != tarval_bad) && (tb != get_mode_null(get_irn_mode(b))))   /* div by zero: return tarval_bad */
-                       return tarval_div(ta, tb);
-               else if (ta == get_mode_null(get_tarval_mode(ta)))  /* 0 / b == 0 */
-                       return ta;
-       }
-       return tarval_bad;
-}  /* do_computed_value_Div */
-
-/**
- * Return the value of an integer Div.
- */
-static tarval *computed_value_Div(ir_node *n) {
-       return do_computed_value_Div(get_Div_left(n), get_Div_right(n));
-}  /* computed_value_Div */
-
-/**
- * Calculate the value of an integer Mod of two nodes.
- * Special case: a % 1
- */
-static tarval *do_computed_value_Mod(ir_node *a, ir_node *b) {
-       tarval *ta = value_of(a);
-       tarval *tb = value_of(b);
-
-       /* Compute c1 % c2 or a % 1 */
-       if (tb != tarval_bad) {
-               if ((ta != tarval_bad) && (tb != get_mode_null(get_tarval_mode(tb))))   /* div by zero: return tarval_bad */
-                       return tarval_mod(ta, tb);
-               else if (tb == get_mode_one(get_tarval_mode(tb)))    /* x mod 1 == 0 */
-                       return get_mode_null(get_irn_mode(a));
-       }
-       return tarval_bad;
-}  /* do_computed_value_Mod */
-
-/**
- * Return the value of an integer Mod.
- */
-static tarval *computed_value_Mod(ir_node *n) {
-       return do_computed_value_Mod(get_Mod_left(n), get_Mod_right(n));
-}  /* computed_value_Mod */
-
 /**
  * Return the value of an Abs.
  */
@@ -579,6 +512,54 @@ static tarval *computed_value_Proj_Cmp(ir_node *n) {
        return computed_value_Cmp_Confirm(a, aa, ab, proj_nr);
 }  /* computed_value_Proj_Cmp */
 
+/**
+ * Return the value of a floating point Quot.
+ */
+static tarval *do_computed_value_Quot(ir_node *a, ir_node *b) {
+       tarval  *ta = value_of(a);
+       tarval  *tb = value_of(b);
+
+       /* cannot optimize 0 / b = 0 because of NaN */
+       if (ta != tarval_bad && tb != tarval_bad)
+               return tarval_quo(ta, tb);
+       return tarval_bad;
+}  /* do_computed_value_Quot */
+
+/**
+ * Calculate the value of an integer Div of two nodes.
+ * Special case: 0 / b
+ */
+static tarval *do_computed_value_Div(ir_node *a, ir_node *b) {
+       tarval  *ta = value_of(a);
+       tarval  *tb;
+       ir_node *dummy;
+
+       /* Compute c1 / c2 or 0 / a, a != 0 */
+       if (tarval_is_null(ta) && value_not_zero(b, &dummy))
+               return ta;  /* 0 / b == 0 */
+       tb = value_of(b);
+       if (ta != tarval_bad && tb != tarval_bad)
+               return tarval_div(ta, tb);
+       return tarval_bad;
+}  /* do_computed_value_Div */
+
+/**
+ * Calculate the value of an integer Mod of two nodes.
+ * Special case: a % 1
+ */
+static tarval *do_computed_value_Mod(ir_node *a, ir_node *b) {
+       tarval *ta = value_of(a);
+       tarval *tb = value_of(b);
+
+       /* Compute a % 1 or c1 % c2 */
+       if (tarval_is_one(tb))
+               return get_mode_null(get_irn_mode(a));
+       if (ta != tarval_bad && tb != tarval_bad)
+               return tarval_mod(ta, tb);
+       return tarval_bad;
+}  /* do_computed_value_Mod */
+
+
 /**
  * Return the value of a Proj, handle Proj(Cmp), Proj(Div), Proj(Mod),
  * Proj(DivMod) and Proj(Quot).
@@ -602,12 +583,12 @@ static tarval *computed_value_Proj(ir_node *n) {
 
        case iro_Div:
                if (get_Proj_proj(n) == pn_Div_res)
-                       return computed_value(a);
+                       return do_computed_value_Div(get_Div_left(a), get_Div_right(a));
                break;
 
        case iro_Mod:
                if (get_Proj_proj(n) == pn_Mod_res)
-                       return computed_value(a);
+                       return do_computed_value_Mod(get_Mod_left(a), get_Mod_right(a));
                break;
 
        case iro_Quot:
@@ -662,10 +643,13 @@ static tarval *computed_value_Confirm(ir_node *n) {
         * remove_confirm is activated.
         */
        if (get_opt_remove_confirm()) {
-               return get_Confirm_cmp(n) == pn_Cmp_Eq ?
-                       value_of(get_Confirm_bound(n)) : tarval_bad;
+               if (get_Confirm_cmp(n) == pn_Cmp_Eq) {
+                       tarval *tv = value_of(get_Confirm_bound(n));
+                       if (tv != tarval_bad)
+                               return tv;
+               }
        }
-       return tarval_bad;
+       return value_of(get_Confirm_value(n));
 }  /* computed_value_Confirm */
 
 /**
@@ -703,9 +687,6 @@ static ir_op_ops *firm_set_default_computed_value(ir_opcode code, ir_op_ops *ops
        CASE(Sub);
        CASE(Minus);
        CASE(Mul);
-       CASE(Quot);
-       CASE(Div);
-       CASE(Mod);
        CASE(Abs);
        CASE(And);
        CASE(Or);
@@ -1000,10 +981,11 @@ static ir_node *equivalent_node_Add(ir_node *n) {
 static ir_node *equivalent_node_left_zero(ir_node *n) {
        ir_node *oldn = n;
 
-       ir_node *a = get_binop_left(n);
-       ir_node *b = get_binop_right(n);
+       ir_node *a  = get_binop_left(n);
+       ir_node *b  = get_binop_right(n);
+       tarval  *tb = value_of(b);
 
-       if (is_Const(b) && is_Const_null(b)) {
+       if (tarval_is_null(tb)) {
                n = a;
 
                DBG_OPT_ALGSIM1(oldn, a, b, n, FS_OPT_NEUTRAL_0);
@@ -1029,15 +1011,17 @@ static ir_node *equivalent_node_Sub(ir_node *n) {
        ir_node *oldn = n;
        ir_node *b;
        ir_mode *mode = get_irn_mode(n);
+       tarval  *tb;
 
        /* for FP these optimizations are only allowed if fp_strict_algebraic is disabled */
        if (mode_is_float(mode) && (get_irg_fp_model(current_ir_graph) & fp_strict_algebraic))
                return n;
 
-       b = get_Sub_right(n);
+       b  = get_Sub_right(n);
+       tb = value_of(b);
 
        /* Beware: modes might be different */
-       if (is_Const(b) && is_Const_null(b)) {
+       if (tarval_is_null(tb)) {
                ir_node *a = get_Sub_left(n);
                if (mode == get_irn_mode(a)) {
                        n = a;
@@ -1086,85 +1070,27 @@ static ir_node *equivalent_node_Mul(ir_node *n) {
        /* we can handle here only the n * n = n bit cases */
        if (get_irn_mode(n) == get_irn_mode(a)) {
                ir_node *b = get_Mul_right(n);
+               tarval  *tv;
 
-               /* Mul is commutative and has again an other neutral element. */
-               if (is_Const(a) && is_Const_one(a)) {
-                       n = b;
-                       DBG_OPT_ALGSIM1(oldn, a, b, n, FS_OPT_NEUTRAL_1);
-               } else if (is_Const(b) && is_Const_one(b)) {
+               /*
+                * Mul is commutative and has again an other neutral element.
+                * Constants are place right, so check this case first.
+                */
+               tv = value_of(b);
+               if (tarval_is_one(tv)) {
                        n = a;
                        DBG_OPT_ALGSIM1(oldn, a, b, n, FS_OPT_NEUTRAL_1);
+               } else {
+                       tv = value_of(a);
+                       if (tarval_is_one(tv)) {
+                               n = b;
+                               DBG_OPT_ALGSIM1(oldn, a, b, n, FS_OPT_NEUTRAL_1);
+                       }
                }
        }
        return n;
 }  /* equivalent_node_Mul */
 
-/**
- * Optimize a / 1 = a.
- */
-static ir_node *equivalent_node_Div(ir_node *n) {
-       ir_node *a = get_Div_left(n);
-       ir_node *b = get_Div_right(n);
-
-       /* Div is not commutative. */
-       if (is_Const(b) && is_Const_one(b)) { /* div(x, 1) == x */
-               /* Turn Div into a tuple (mem, bad, a) */
-               ir_node *mem = get_Div_mem(n);
-               ir_node *blk = get_irn_n(n, -1);
-               turn_into_tuple(n, pn_Div_max);
-               set_Tuple_pred(n, pn_Div_M,         mem);
-               set_Tuple_pred(n, pn_Div_X_regular, new_r_Jmp(current_ir_graph, blk));
-               set_Tuple_pred(n, pn_Div_X_except,  new_Bad());        /* no exception */
-               set_Tuple_pred(n, pn_Div_res,       a);
-       }
-       return n;
-}  /* equivalent_node_Div */
-
-/**
- * Optimize a / 1.0 = a.
- */
-static ir_node *equivalent_node_Quot(ir_node *n) {
-       ir_node *a = get_Quot_left(n);
-       ir_node *b = get_Quot_right(n);
-
-       /* Div is not commutative. */
-       if (is_Const(b) && is_Const_one(b)) { /* Quot(x, 1) == x */
-               /* Turn Quot into a tuple (mem, jmp, bad, a) */
-               ir_node *mem = get_Quot_mem(n);
-               ir_node *blk = get_irn_n(n, -1);
-               turn_into_tuple(n, pn_Quot_max);
-               set_Tuple_pred(n, pn_Quot_M,         mem);
-               set_Tuple_pred(n, pn_Quot_X_regular, new_r_Jmp(current_ir_graph, blk));
-               set_Tuple_pred(n, pn_Quot_X_except,  new_Bad());        /* no exception */
-               set_Tuple_pred(n, pn_Quot_res,       a);
-       }
-       return n;
-}  /* equivalent_node_Quot */
-
-/**
- * Optimize a / 1 = a.
- */
-static ir_node *equivalent_node_DivMod(ir_node *n) {
-       ir_node *b = get_DivMod_right(n);
-
-       /* Div is not commutative. */
-       if (is_Const(b) && is_Const_one(b)) { /* div(x, 1) == x */
-               /* Turn DivMod into a tuple (mem, jmp, bad, a, 0) */
-               ir_node *a = get_DivMod_left(n);
-               ir_node *mem = get_Div_mem(n);
-               ir_node *blk = get_irn_n(n, -1);
-               ir_mode *mode = get_DivMod_resmode(n);
-
-               turn_into_tuple(n, pn_DivMod_max);
-               set_Tuple_pred(n, pn_DivMod_M,         mem);
-               set_Tuple_pred(n, pn_DivMod_X_regular, new_r_Jmp(current_ir_graph, blk));
-               set_Tuple_pred(n, pn_DivMod_X_except,  new_Bad());        /* no exception */
-               set_Tuple_pred(n, pn_DivMod_res_div,   a);
-               set_Tuple_pred(n, pn_DivMod_res_mod,   new_Const(mode, get_mode_null(mode)));
-       }
-       return n;
-}  /* equivalent_node_DivMod */
-
 /**
  * Use algebraic simplification a | a = a | 0 = 0 | a = a.
  */
@@ -1294,6 +1220,23 @@ restart:
                                        }
                                }
                        }
+                       if (mode_is_int(n_mode) && mode_is_float(a_mode)) {
+                               /* ConvI(ConvF(I)) -> I, iff float mantissa >= int mode */
+                               size_t int_mantissa = get_mode_size_bits(n_mode) - (mode_is_signed(n_mode) ? 1 : 0);
+                               size_t float_mantissa;
+                               /* FIXME There is no way to get the mantissa size of a mode */
+                               switch (get_mode_size_bits(a_mode)) {
+                                       case 32: float_mantissa = 23 + 1; break; // + 1 for implicit 1
+                                       case 64: float_mantissa = 52 + 1; break;
+                                       case 80: float_mantissa = 64 + 1; break;
+                                       default: float_mantissa = 0;      break;
+                               }
+                               if (float_mantissa != 0 && float_mantissa >= int_mantissa) {
+                                       n = b;
+                                       DBG_OPT_ALGSIM1(oldn, a, b, n, FS_OPT_CONV);
+                                       return n;
+                               }
+                       }
                        if (is_Conv(b)) {
                                if (smaller_mode(b_mode, a_mode)) {
                                        if (get_Conv_strict(n))
@@ -1441,67 +1384,286 @@ static ir_node *equivalent_node_Sync(ir_node *n) {
 }  /* equivalent_node_Sync */
 
 /**
- * Optimize Proj(Tuple) and gigo() for ProjX in Bad block,
- * ProjX(Load) and ProjX(Store).
+ * Optimize Proj(Tuple).
  */
-static ir_node *equivalent_node_Proj(ir_node *proj) {
+static ir_node *equivalent_node_Proj_Tuple(ir_node *proj) {
+       ir_node *oldn  = proj;
+       ir_node *tuple = get_Proj_pred(proj);
+
+       /* Remove the Tuple/Proj combination. */
+       proj = get_Tuple_pred(tuple, get_Proj_proj(proj));
+       DBG_OPT_TUPLE(oldn, tuple, proj);
+
+       return proj;
+}  /* equivalent_node_Proj_Tuple */
+
+/**
+ * Optimize a / 1 = a.
+ */
+static ir_node *equivalent_node_Proj_Div(ir_node *proj) {
        ir_node *oldn = proj;
-       ir_node *a = get_Proj_pred(proj);
+       ir_node *div  = get_Proj_pred(proj);
+       ir_node *b    = get_Div_right(div);
+       tarval  *tb   = value_of(b);
 
-       if (is_Tuple(a)) {
-               /* Remove the Tuple/Proj combination. */
-               if ( get_Proj_proj(proj) <= get_Tuple_n_preds(a) ) {
-                       proj = get_Tuple_pred(a, get_Proj_proj(proj));
-                       DBG_OPT_TUPLE(oldn, a, proj);
-               } else {
-                        /* This should not happen! */
-                       assert(! "found a Proj with higher number than Tuple predecessors");
-                       proj = new_Bad();
-               }
-       } else if (get_irn_mode(proj) == mode_X) {
-               if (is_Block_dead(get_nodes_block(skip_Proj(proj)))) {
-                       /* Remove dead control flow -- early gigo(). */
-                       proj = new_Bad();
-               } else if (get_opt_ldst_only_null_ptr_exceptions()) {
-                       if (is_Load(a)) {
-                               /* get the Load address */
-                               ir_node *addr = get_Load_ptr(a);
-                               ir_node *blk  = get_irn_n(a, -1);
-                               ir_node *confirm;
-
-                               if (value_not_null(addr, &confirm)) {
-                                       if (confirm == NULL) {
-                                               /* this node may float if it did not depend on a Confirm */
-                                               set_irn_pinned(a, op_pin_state_floats);
-                                       }
-                                       if (get_Proj_proj(proj) == pn_Load_X_except) {
-                                               DBG_OPT_EXC_REM(proj);
-                                               return new_Bad();
-                                       } else
-                                               return new_r_Jmp(current_ir_graph, blk);
-                               }
-                       } else if (is_Store(a)) {
-                               /* get the load/store address */
-                               ir_node *addr = get_Store_ptr(a);
-                               ir_node *blk  = get_irn_n(a, -1);
-                               ir_node *confirm;
-
-                               if (value_not_null(addr, &confirm)) {
-                                       if (confirm == NULL) {
-                                               /* this node may float if it did not depend on a Confirm */
-                                               set_irn_pinned(a, op_pin_state_floats);
-                                       }
-                                       if (get_Proj_proj(proj) == pn_Store_X_except) {
-                                               DBG_OPT_EXC_REM(proj);
-                                               return new_Bad();
-                                       } else
-                                               return new_r_Jmp(current_ir_graph, blk);
+       /* Div is not commutative. */
+       if (tarval_is_one(tb)) { /* div(x, 1) == x */
+               switch (get_Proj_proj(proj)) {
+               case pn_Div_M:
+                       proj = get_Div_mem(div);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               case pn_Div_res:
+                       proj = get_Div_left(div);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               default:
+                       /* we cannot replace the exception Proj's here, this is done in
+                          transform_node_Proj_Div() */
+                       return proj;
+               }
+       }
+       return proj;
+}  /* equivalent_node_Proj_Div */
+
+/**
+ * Optimize a / 1.0 = a.
+ */
+static ir_node *equivalent_node_Proj_Quot(ir_node *proj) {
+       ir_node *oldn = proj;
+       ir_node *quot = get_Proj_pred(proj);
+       ir_node *b    = get_Quot_right(quot);
+       tarval  *tb   = value_of(b);
+
+       /* Div is not commutative. */
+       if (tarval_is_one(tb)) { /* Quot(x, 1) == x */
+               switch (get_Proj_proj(proj)) {
+               case pn_Quot_M:
+                       proj = get_Quot_mem(quot);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               case pn_Quot_res:
+                       proj = get_Quot_left(quot);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               default:
+                       /* we cannot replace the exception Proj's here, this is done in
+                          transform_node_Proj_Quot() */
+                       return proj;
+               }
+       }
+       return proj;
+}  /* equivalent_node_Proj_Quot */
+
+/**
+ * Optimize a / 1 = a.
+ */
+static ir_node *equivalent_node_Proj_DivMod(ir_node *proj) {
+       ir_node *oldn   = proj;
+       ir_node *divmod = get_Proj_pred(proj);
+       ir_node *b      = get_DivMod_right(divmod);
+       tarval  *tb     = value_of(b);
+
+       /* Div is not commutative. */
+       if (tarval_is_one(tb)) { /* div(x, 1) == x */
+               switch (get_Proj_proj(proj)) {
+               case pn_DivMod_M:
+                       proj = get_DivMod_mem(divmod);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               case pn_DivMod_res_div:
+                       proj = get_DivMod_left(divmod);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NEUTRAL_1);
+                       return proj;
+
+               default:
+                       /* we cannot replace the exception Proj's here, this is done in
+                          transform_node_Proj_DivMod().
+                          Note further that the pn_DivMod_res_div case is handled in
+                          computed_value_Proj(). */
+                       return proj;
+               }
+       }
+       return proj;
+}  /* equivalent_node_Proj_DivMod */
+
+/**
+ * Optimize CopyB(mem, x, x) into a Nop.
+ */
+static ir_node *equivalent_node_Proj_CopyB(ir_node *proj) {
+       ir_node *oldn  = proj;
+       ir_node *copyb = get_Proj_pred(proj);
+       ir_node *a     = get_CopyB_dst(copyb);
+       ir_node *b     = get_CopyB_src(copyb);
+
+       if (a == b) {
+               /* Turn CopyB into a tuple (mem, jmp, bad, bad) */
+               switch (get_Proj_proj(proj)) {
+               case pn_CopyB_M_regular:
+                       proj = get_CopyB_mem(copyb);
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NOP);
+                       break;
+
+               case pn_CopyB_M_except:
+               case pn_CopyB_X_except:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_irg_bad(current_ir_graph);
+                       break;
+               }
+       }
+       return proj;
+}  /* equivalent_node_Proj_CopyB */
+
+/**
+ * Optimize Bounds(idx, idx, upper) into idx.
+ */
+static ir_node *equivalent_node_Proj_Bound(ir_node *proj) {
+       ir_node *oldn  = proj;
+       ir_node *bound = get_Proj_pred(proj);
+       ir_node *idx   = get_Bound_index(bound);
+       ir_node *pred  = skip_Proj(idx);
+       int ret_tuple  = 0;
+
+       if (idx == get_Bound_lower(bound))
+               ret_tuple = 1;
+       else if (is_Bound(pred)) {
+               /*
+                * idx was Bounds checked in the same MacroBlock previously,
+                * it is still valid if lower <= pred_lower && pred_upper <= upper.
+                */
+               ir_node *lower = get_Bound_lower(bound);
+               ir_node *upper = get_Bound_upper(bound);
+               if (get_Bound_lower(pred) == lower &&
+                       get_Bound_upper(pred) == upper &&
+                       get_irn_MacroBlock(bound) == get_irn_MacroBlock(pred)) {
+                       /*
+                        * One could expect that we simply return the previous
+                        * Bound here. However, this would be wrong, as we could
+                        * add an exception Proj to a new location then.
+                        * So, we must turn in into a tuple.
+                        */
+                       ret_tuple = 1;
+               }
+       }
+       if (ret_tuple) {
+               /* Turn Bound into a tuple (mem, jmp, bad, idx) */
+               switch (get_Proj_proj(proj)) {
+               case pn_Bound_M:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_Bound_mem(bound);
+                       break;
+               case pn_Bound_X_except:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_irg_bad(current_ir_graph);
+                       break;
+               case pn_Bound_res:
+                       proj = idx;
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NOP);
+                       break;
+               default:
+                       /* cannot optimize pn_Bound_X_regular, handled in transform ... */
+                       ;
+               }
+       }
+       return proj;
+}  /* equivalent_node_Proj_Bound */
+
+/**
+ * Optimize an Exception Proj(Load) with a non-null address.
+ */
+static ir_node *equivalent_node_Proj_Load(ir_node *proj) {
+       if (get_opt_ldst_only_null_ptr_exceptions()) {
+               if (get_irn_mode(proj) == mode_X) {
+                       ir_node *oldn = proj;
+                       ir_node *load = get_Proj_pred(proj);
+
+                       /* get the Load address */
+                       ir_node *addr = get_Load_ptr(load);
+                       ir_node *blk  = get_nodes_block(load);
+                       ir_node *confirm;
+
+                       if (value_not_null(addr, &confirm)) {
+                               if (get_Proj_proj(proj) == pn_Load_X_except) {
+                                       DBG_OPT_EXC_REM(proj);
+                                       return get_irg_bad(current_ir_graph);
                                }
                        }
                }
        }
+       return proj;
+}  /* equivalent_node_Proj_Load */
 
+/**
+ * Optimize an Exception Proj(Store) with a non-null address.
+ */
+static ir_node *equivalent_node_Proj_Store(ir_node *proj) {
+       if (get_opt_ldst_only_null_ptr_exceptions()) {
+               if (get_irn_mode(proj) == mode_X) {
+                       ir_node *oldn  = proj;
+                       ir_node *store = get_Proj_pred(proj);
+
+                       /* get the load/store address */
+                       ir_node *addr = get_Store_ptr(store);
+                       ir_node *blk  = get_nodes_block(store);
+                       ir_node *confirm;
+
+                       if (value_not_null(addr, &confirm)) {
+                               if (get_Proj_proj(proj) == pn_Store_X_except) {
+                                       DBG_OPT_EXC_REM(proj);
+                                       return get_irg_bad(current_ir_graph);
+                               }
+                       }
+               }
+       }
        return proj;
+}  /* equivalent_node_Proj_Store */
+
+/**
+ * Does all optimizations on nodes that must be done on it's Proj's
+ * because of creating new nodes.
+ */
+static ir_node *equivalent_node_Proj(ir_node *proj) {
+       ir_node *n = get_Proj_pred(proj);
+
+       switch (get_irn_opcode(n)) {
+       case iro_Div:
+               return equivalent_node_Proj_Div(proj);
+
+       case iro_DivMod:
+               return equivalent_node_Proj_DivMod(proj);
+
+       case iro_Quot:
+               return equivalent_node_Proj_DivMod(proj);
+
+       case iro_Tuple:
+               return equivalent_node_Proj_Tuple(proj);
+
+       case iro_CopyB:
+               return equivalent_node_Proj_CopyB(proj);
+
+       case iro_Bound:
+               return equivalent_node_Proj_Bound(proj);
+
+       case iro_Load:
+               return equivalent_node_Proj_Load(proj);
+
+       case iro_Store:
+               return equivalent_node_Proj_Store(proj);
+
+       default:
+               if (get_irn_mode(proj) == mode_X) {
+                       if (is_Block_dead(get_nodes_block(n))) {
+                               /* Remove dead control flow -- early gigo(). */
+                               proj = get_irg_bad(current_ir_graph);
+                       }
+               }
+               return proj;
+       }
 }  /* equivalent_node_Proj */
 
 /**
@@ -1512,7 +1674,7 @@ static ir_node *equivalent_node_Id(ir_node *n) {
 
        do {
                n = get_Id_pred(n);
-       } while (get_irn_op(n) == op_Id);
+       } while (is_Id(n));
 
        DBG_OPT_ID(oldn, n);
        return n;
@@ -1581,7 +1743,7 @@ static ir_node *equivalent_node_Mux(ir_node *n)
                         * Note: normalization puts the constant on the right side,
                         * so we check only one case.
                         */
-                       if (cmp_l == t && is_Const(cmp_r) && is_Const_null(cmp_r)) {
+                       if (cmp_l == t && tarval_is_null(value_of(cmp_r))) {
                                /* Mux(t CMP 0, X, t) */
                                if (is_Minus(f) && get_Minus_op(f) == t) {
                                        /* Mux(t CMP 0, -t, t) */
@@ -1612,29 +1774,6 @@ static ir_node *equivalent_node_Psi(ir_node *n) {
        return n;
 }  /* equivalent_node_Psi */
 
-/**
- * Optimize -a CMP -b into b CMP a.
- * This works only for for modes where unary Minus
- * cannot Overflow.
- * Note that two-complement integers can Overflow
- * so it will NOT work.
- *
- * For == and != can be handled in Proj(Cmp)
- */
-static ir_node *equivalent_node_Cmp(ir_node *n) {
-       ir_node *left  = get_Cmp_left(n);
-       ir_node *right = get_Cmp_right(n);
-
-       if (is_Minus(left) && is_Minus(right) &&
-               !mode_overflow_on_unary_Minus(get_irn_mode(left))) {
-               left  = get_Minus_op(left);
-               right = get_Minus_op(right);
-               set_Cmp_left(n, right);
-               set_Cmp_right(n, left);
-       }
-       return n;
-}  /* equivalent_node_Cmp */
-
 /**
  * Remove Confirm nodes if setting is on.
  * Replace Confirms(x, '=', Constlike) by Constlike.
@@ -1643,78 +1782,20 @@ static ir_node *equivalent_node_Confirm(ir_node *n) {
        ir_node *pred = get_Confirm_value(n);
        pn_Cmp  pnc   = get_Confirm_cmp(n);
 
-       if (is_Confirm(pred) && pnc == get_Confirm_cmp(pred)) {
+       while (is_Confirm(pred) && pnc == get_Confirm_cmp(pred)) {
                /*
                 * rare case: two identical Confirms one after another,
                 * replace the second one with the first.
                 */
-               n = pred;
+               n    = pred;
+               pred = get_Confirm_value(n);
+               pnc  = get_Confirm_cmp(n);
        }
        if (get_opt_remove_confirm())
                return get_Confirm_value(n);
        return n;
 }
 
-/**
- * Optimize CopyB(mem, x, x) into a Nop.
- */
-static ir_node *equivalent_node_CopyB(ir_node *n) {
-       ir_node *a = get_CopyB_dst(n);
-       ir_node *b = get_CopyB_src(n);
-
-       if (a == b) {
-               /* Turn CopyB into a tuple (mem, jmp, bad, bad) */
-               ir_node *mem = get_CopyB_mem(n);
-               ir_node *blk = get_nodes_block(n);
-               turn_into_tuple(n, pn_CopyB_max);
-               set_Tuple_pred(n, pn_CopyB_M,         mem);
-               set_Tuple_pred(n, pn_CopyB_X_regular, new_r_Jmp(current_ir_graph, blk));
-               set_Tuple_pred(n, pn_CopyB_X_except,  new_Bad());        /* no exception */
-               set_Tuple_pred(n, pn_CopyB_M_except,  new_Bad());
-       }
-       return n;
-}  /* equivalent_node_CopyB */
-
-/**
- * Optimize Bounds(idx, idx, upper) into idx.
- */
-static ir_node *equivalent_node_Bound(ir_node *n) {
-       ir_node *idx  = get_Bound_index(n);
-       ir_node *pred = skip_Proj(idx);
-       int ret_tuple = 0;
-
-       if (is_Bound(pred)) {
-               /*
-                * idx was Bounds checked in the same MacroBlock previously,
-                * it is still valid if lower <= pred_lower && pred_upper <= upper.
-                */
-               ir_node *lower = get_Bound_lower(n);
-               ir_node *upper = get_Bound_upper(n);
-               if (get_Bound_lower(pred) == lower &&
-                       get_Bound_upper(pred) == upper &&
-                       get_irn_MacroBlock(n) == get_irn_MacroBlock(pred)) {
-                       /*
-                        * One could expect that we simply return the previous
-                        * Bound here. However, this would be wrong, as we could
-                        * add an exception Proj to a new location then.
-                        * So, we must turn in into a tuple.
-                        */
-                       ret_tuple = 1;
-               }
-       }
-       if (ret_tuple) {
-               /* Turn Bound into a tuple (mem, jmp, bad, idx) */
-               ir_node *mem = get_Bound_mem(n);
-               ir_node *blk = get_nodes_block(n);
-               turn_into_tuple(n, pn_Bound_max);
-               set_Tuple_pred(n, pn_Bound_M,         mem);
-               set_Tuple_pred(n, pn_Bound_X_regular, new_r_Jmp(current_ir_graph, blk));       /* no exception */
-               set_Tuple_pred(n, pn_Bound_X_except,  new_Bad());       /* no exception */
-               set_Tuple_pred(n, pn_Bound_res,       idx);
-       }
-       return n;
-}  /* equivalent_node_Bound */
-
 /**
  * equivalent_node() returns a node equivalent to input n. It skips all nodes that
  * perform no actual computation, as, e.g., the Id nodes.  It does not create
@@ -1759,9 +1840,6 @@ static ir_op_ops *firm_set_default_equivalent_node(ir_opcode code, ir_op_ops *op
        CASE(Not);
        CASE(Minus);
        CASE(Mul);
-       CASE(Div);
-       CASE(Quot);
-       CASE(DivMod);
        CASE(And);
        CASE(Conv);
        CASE(Cast);
@@ -1771,10 +1849,7 @@ static ir_op_ops *firm_set_default_equivalent_node(ir_opcode code, ir_op_ops *op
        CASE(Id);
        CASE(Mux);
        CASE(Psi);
-       CASE(Cmp);
        CASE(Confirm);
-       CASE(CopyB);
-       CASE(Bound);
        default:
                /* leave NULL */;
        }
@@ -1798,6 +1873,24 @@ static int is_const_Phi(ir_node *n) {
                return 1;
 }  /* is_const_Phi */
 
+typedef tarval *(*tarval_sub_type)(tarval *a, tarval *b, ir_mode *mode);
+typedef tarval *(*tarval_binop_type)(tarval *a, tarval *b);
+
+/**
+ * Wrapper for the tarval binop evaluation, tarval_sub has one more parameter.
+ */
+static tarval *do_eval(tarval *(*eval)(), tarval *a, tarval *b, ir_mode *mode) {
+       if (eval == tarval_sub) {
+               tarval_sub_type func = (tarval_sub_type)eval;
+
+               return func(a, b, mode);
+       } else {
+               tarval_binop_type func = (tarval_binop_type)eval;
+
+               return func(a, b);
+       }
+}
+
 /**
  * Apply an evaluator on a binop with a constant operators (and one Phi).
  *
@@ -1809,7 +1902,7 @@ static int is_const_Phi(ir_node *n) {
  *
  * @return a new Phi node if the conversion was successful, NULL else
  */
-static ir_node *apply_binop_on_phi(ir_node *phi, tarval *other, tarval *(*eval)(tarval *, tarval *), ir_mode *mode, int left) {
+static ir_node *apply_binop_on_phi(ir_node *phi, tarval *other, tarval *(*eval)(), ir_mode *mode, int left) {
        tarval   *tv;
        void     **res;
        ir_node  *pred;
@@ -1821,7 +1914,7 @@ static ir_node *apply_binop_on_phi(ir_node *phi, tarval *other, tarval *(*eval)(
                for (i = 0; i < n; ++i) {
                        pred = get_irn_n(phi, i);
                        tv   = get_Const_tarval(pred);
-                       tv   = eval(other, tv);
+                       tv   = do_eval(eval, other, tv, mode);
 
                        if (tv == tarval_bad) {
                                /* folding failed, bad */
@@ -1833,7 +1926,7 @@ static ir_node *apply_binop_on_phi(ir_node *phi, tarval *other, tarval *(*eval)(
                for (i = 0; i < n; ++i) {
                        pred = get_irn_n(phi, i);
                        tv   = get_Const_tarval(pred);
-                       tv   = eval(tv, other);
+                       tv   = do_eval(eval, tv, other, mode);
 
                        if (tv == tarval_bad) {
                                /* folding failed, bad */
@@ -1861,7 +1954,7 @@ static ir_node *apply_binop_on_phi(ir_node *phi, tarval *other, tarval *(*eval)(
  *
  * @return a new Phi node if the conversion was successful, NULL else
  */
-static ir_node *apply_binop_on_2_phis(ir_node *a, ir_node *b, tarval *(*eval)(tarval *, tarval *), ir_mode *mode) {
+static ir_node *apply_binop_on_2_phis(ir_node *a, ir_node *b, tarval *(*eval)(), ir_mode *mode) {
        tarval   *tv_l, *tv_r, *tv;
        void     **res;
        ir_node  *pred;
@@ -1879,7 +1972,7 @@ static ir_node *apply_binop_on_2_phis(ir_node *a, ir_node *b, tarval *(*eval)(ta
                tv_l = get_Const_tarval(pred);
                pred = get_irn_n(b, i);
                tv_r = get_Const_tarval(pred);
-               tv   = eval(tv_l, tv_r);
+               tv   = do_eval(eval, tv_l, tv_r, mode);
 
                if (tv == tarval_bad) {
                        /* folding failed, bad */
@@ -3006,6 +3099,32 @@ static ir_node *transform_node_Abs(ir_node *n) {
        return n;
 }  /* transform_node_Abs */
 
+/**
+ * Optimize -a CMP -b into b CMP a.
+ * This works only for for modes where unary Minus
+ * cannot Overflow.
+ * Note that two-complement integers can Overflow
+ * so it will NOT work.
+ *
+ * For == and != can be handled in Proj(Cmp)
+ */
+static ir_node *transform_node_Cmp(ir_node *n) {
+       ir_node *oldn = n;
+       ir_node *left  = get_Cmp_left(n);
+       ir_node *right = get_Cmp_right(n);
+
+       if (is_Minus(left) && is_Minus(right) &&
+               !mode_overflow_on_unary_Minus(get_irn_mode(left))) {
+               left  = get_Minus_op(left);
+               right = get_Minus_op(right);
+               n = new_rd_Cmp(get_irn_dbg_info(n), current_ir_graph,
+                       get_nodes_block(n), left, right);
+               DBG_OPT_ALGSIM0(oldn, n, FS_OPT_CMP_OP_OP);
+       }
+       return n;
+}  /* transform_node_Cmp */
+
+
 /**
  * Transform a Cond node.
  *
@@ -3519,6 +3638,67 @@ static ir_node *transform_node_Cast(ir_node *n) {
        return n;
 }  /* transform_node_Cast */
 
+/**
+ * Transform a Proj(Load) with a non-null address.
+ */
+static ir_node *transform_node_Proj_Load(ir_node *proj) {
+       if (get_opt_ldst_only_null_ptr_exceptions()) {
+               if (get_irn_mode(proj) == mode_X) {
+                       ir_node *oldn = proj;
+                       ir_node *load = get_Proj_pred(proj);
+
+                       /* get the Load address */
+                       ir_node *addr = get_Load_ptr(load);
+                       ir_node *blk  = get_nodes_block(load);
+                       ir_node *confirm;
+
+                       if (value_not_null(addr, &confirm)) {
+                               if (confirm == NULL) {
+                                       /* this node may float if it did not depend on a Confirm */
+                                       set_irn_pinned(load, op_pin_state_floats);
+                               }
+                               if (get_Proj_proj(proj) == pn_Load_X_except) {
+                                       DBG_OPT_EXC_REM(proj);
+                                       return get_irg_bad(current_ir_graph);
+                               } else {
+                                       return new_r_Jmp(current_ir_graph, blk);
+                               }
+                       }
+               }
+       }
+       return proj;
+}  /* transform_node_Proj_Load */
+
+/**
+ * Transform a Proj(Store) with a non-null address.
+ */
+static ir_node *transform_node_Proj_Store(ir_node *proj) {
+       if (get_opt_ldst_only_null_ptr_exceptions()) {
+               if (get_irn_mode(proj) == mode_X) {
+                       ir_node *oldn  = proj;
+                       ir_node *store = get_Proj_pred(proj);
+
+                       /* get the load/store address */
+                       ir_node *addr = get_Store_ptr(store);
+                       ir_node *blk  = get_nodes_block(store);
+                       ir_node *confirm;
+
+                       if (value_not_null(addr, &confirm)) {
+                               if (confirm == NULL) {
+                                       /* this node may float if it did not depend on a Confirm */
+                                       set_irn_pinned(store, op_pin_state_floats);
+                               }
+                               if (get_Proj_proj(proj) == pn_Store_X_except) {
+                                       DBG_OPT_EXC_REM(proj);
+                                       return get_irg_bad(current_ir_graph);
+                               } else
+                                       return new_r_Jmp(current_ir_graph, blk);
+                       }
+               }
+       }
+       return proj;
+}  /* transform_node_Proj_Store */
+
 /**
  * Transform a Proj(Div) with a non-zero value.
  * Removes the exceptions and routes the memory to the NoMem node.
@@ -4047,7 +4227,7 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
                                /* c > 0 : a < c  ==>  a <= (c-1)    a >= c  ==>  a > (c-1) */
                                if ((proj_nr == pn_Cmp_Lt || proj_nr == pn_Cmp_Ge) &&
                                        tarval_cmp(tv, get_mode_null(mode)) == pn_Cmp_Gt) {
-                                       tv = tarval_sub(tv, get_mode_one(mode));
+                                       tv = tarval_sub(tv, get_mode_one(mode), NULL);
 
                                        if (tv != tarval_bad) {
                                                proj_nr ^= pn_Cmp_Eq;
@@ -4115,7 +4295,7 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
                                                        }
 
                                                        if (tv2 != tarval_bad) {
-                                                               tv2 = tarval_sub(tv, tv2);
+                                                               tv2 = tarval_sub(tv, tv2, NULL);
 
                                                                if (tv2 != tarval_bad) {
                                                                        left    = a;
@@ -4127,7 +4307,7 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
                                                }
                                                /* -a == c ==> a == -c, -a != c ==> a != -c */
                                                else if (is_Minus(left)) {
-                                                       tarval *tv2 = tarval_sub(get_mode_null(mode), tv);
+                                                       tarval *tv2 = tarval_sub(get_mode_null(mode), tv, NULL);
 
                                                        if (tv2 != tarval_bad) {
                                                                left    = get_Minus_op(left);
@@ -4284,7 +4464,7 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
                                                tarval  *cond   = new_tarval_from_long(get_mode_size_bits(mode), get_tarval_mode(tv1));
                                                ir_node *sl, *blk;
 
-                                               cond = tarval_sub(cond, tv1);
+                                               cond = tarval_sub(cond, tv1, NULL);
                                                cond = tarval_shrs(tv, cond);
 
                                                if (!tarval_is_all_one(cond) && !tarval_is_null(cond)) {
@@ -4326,7 +4506,7 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
                                        ir_node *blk  = get_irn_n(op, -1);
                                        ir_mode *mode = get_irn_mode(v);
 
-                                       tv = tarval_sub(tv, get_mode_one(mode));
+                                       tv = tarval_sub(tv, get_mode_one(mode), NULL);
                                        left = new_rd_And(get_irn_dbg_info(op), current_ir_graph, blk, v, new_Const(mode, tv), mode);
                                        changed |= 1;
                                        DBG_OPT_ALGSIM0(n, n, FS_OPT_CMP_MOD_TO_AND);
@@ -4346,6 +4526,91 @@ static ir_node *transform_node_Proj_Cmp(ir_node *proj) {
        return proj;
 }  /* transform_node_Proj_Cmp */
 
+/**
+ * Optimize CopyB(mem, x, x) into a Nop.
+ */
+static ir_node *transform_node_Proj_CopyB(ir_node *proj) {
+       ir_node *oldn  = proj;
+       ir_node *copyb = get_Proj_pred(proj);
+       ir_node *a     = get_CopyB_dst(copyb);
+       ir_node *b     = get_CopyB_src(copyb);
+
+       if (a == b) {
+               switch (get_Proj_proj(proj)) {
+               case pn_CopyB_X_regular:
+                       /* Turn CopyB into a tuple (mem, jmp, bad, bad) */
+                       DBG_OPT_EXC_REM(proj);
+                       proj = new_r_Jmp(current_ir_graph, get_nodes_block(copyb));
+                       break;
+               case pn_CopyB_M_except:
+               case pn_CopyB_X_except:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_irg_bad(current_ir_graph);
+                       break;
+               default:
+                       break;
+               }
+       }
+       return proj;
+}  /* transform_node_Proj_CopyB */
+
+/**
+ * Optimize Bounds(idx, idx, upper) into idx.
+ */
+static ir_node *transform_node_Proj_Bound(ir_node *proj) {
+       ir_node *oldn  = proj;
+       ir_node *bound = get_Proj_pred(proj);
+       ir_node *idx   = get_Bound_index(bound);
+       ir_node *pred  = skip_Proj(idx);
+       int ret_tuple  = 0;
+
+       if (idx == get_Bound_lower(bound))
+               ret_tuple = 1;
+       else if (is_Bound(pred)) {
+               /*
+               * idx was Bounds checked in the same MacroBlock previously,
+               * it is still valid if lower <= pred_lower && pred_upper <= upper.
+               */
+               ir_node *lower = get_Bound_lower(bound);
+               ir_node *upper = get_Bound_upper(bound);
+               if (get_Bound_lower(pred) == lower &&
+                       get_Bound_upper(pred) == upper &&
+                       get_irn_MacroBlock(bound) == get_irn_MacroBlock(pred)) {
+                       /*
+                        * One could expect that we simply return the previous
+                        * Bound here. However, this would be wrong, as we could
+                        * add an exception Proj to a new location then.
+                        * So, we must turn in into a tuple.
+                        */
+                       ret_tuple = 1;
+               }
+       }
+       if (ret_tuple) {
+               /* Turn Bound into a tuple (mem, jmp, bad, idx) */
+               switch (get_Proj_proj(proj)) {
+               case pn_Bound_M:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_Bound_mem(bound);
+                       break;
+               case pn_Bound_X_except:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = get_irg_bad(current_ir_graph);
+                       break;
+               case pn_Bound_res:
+                       proj = idx;
+                       DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NOP);
+                       break;
+               case pn_Bound_X_regular:
+                       DBG_OPT_EXC_REM(proj);
+                       proj = new_r_Jmp(current_ir_graph, get_nodes_block(bound));
+                       break;
+               default:
+                       break;
+               }
+       }
+       return proj;
+}  /* transform_node_Proj_Bound */
+
 /**
  * Does all optimizations on nodes that must be done on it's Proj's
  * because of creating new nodes.
@@ -4354,6 +4619,12 @@ static ir_node *transform_node_Proj(ir_node *proj) {
        ir_node *n = get_Proj_pred(proj);
 
        switch (get_irn_opcode(n)) {
+       case iro_Load:
+               return transform_node_Proj_Load(proj);
+
+       case iro_Store:
+               return transform_node_Proj_Store(proj);
+
        case iro_Div:
                return transform_node_Proj_Div(proj);
 
@@ -4373,6 +4644,12 @@ static ir_node *transform_node_Proj(ir_node *proj) {
                /* should not happen, but if it does will be optimized away */
                return equivalent_node_Proj(proj);
 
+       case iro_CopyB:
+               return transform_node_Proj_CopyB(proj);
+
+       case iro_Bound:
+               return transform_node_Proj_CopyB(proj);
+
        default:
                /* do nothing */
                return proj;
@@ -4915,7 +5192,7 @@ static ir_node *transform_node_shl_shr(ir_node *n) {
 
        pnc = tarval_cmp(tv_shl, tv_shr);
        if (pnc == pn_Cmp_Lt || pnc == pn_Cmp_Eq) {
-               tv_shift  = tarval_sub(tv_shr, tv_shl);
+               tv_shift  = tarval_sub(tv_shr, tv_shl, NULL);
                new_const = new_Const(get_tarval_mode(tv_shift), tv_shift);
                if (need_shrs) {
                        new_shift = new_rd_Shrs(dbgi, irg, block, x, new_const, mode);
@@ -4924,7 +5201,7 @@ static ir_node *transform_node_shl_shr(ir_node *n) {
                }
        } else {
                assert(pnc == pn_Cmp_Gt);
-               tv_shift  = tarval_sub(tv_shl, tv_shr);
+               tv_shift  = tarval_sub(tv_shl, tv_shr, NULL);
                new_const = new_Const(get_tarval_mode(tv_shift), tv_shift);
                new_shift = new_rd_Shl(dbgi, irg, block, x, new_const, mode);
        }
@@ -5160,10 +5437,10 @@ static ir_node *transform_node_Mux(ir_node *n) {
                tarval *diff, *min;
 
                if (tarval_cmp(a, b) & pn_Cmp_Gt) {
-                       diff = tarval_sub(a, b);
+                       diff = tarval_sub(a, b, NULL);
                        min  = b;
                } else {
-                       diff = tarval_sub(b, a);
+                       diff = tarval_sub(b, a, NULL);
                        min  = a;
                }
 
@@ -5174,8 +5451,8 @@ static ir_node *transform_node_Mux(ir_node *n) {
 
 
                        conds[0] = sel;
-                       vals[0] = new_Const(mode, tarval_sub(a, min));
-                       vals[1] = new_Const(mode, tarval_sub(b, min));
+                       vals[0] = new_Const(mode, tarval_sub(a, min, NULL));
+                       vals[1] = new_Const(mode, tarval_sub(b, min, NULL));
                        n = new_rd_Psi(dbg, irg, block, 1, conds, vals, mode);
                        n = new_rd_Add(dbg, irg, block, n, new_Const(mode, min), mode);
                        return n;
@@ -5388,6 +5665,7 @@ static ir_op_ops *firm_set_default_transform_node(ir_opcode code, ir_op_ops *ops
        CASE(Quot);
        CASE(Abs);
        CASE(Cond);
+       CASE(Cmp);
        CASE(And);
        CASE(Or);
        CASE(Eor);
@@ -5708,38 +5986,14 @@ int identities_cmp(const void *elt, const void *key) {
 
 /*
  * Calculate a hash value of a node.
+ *
+ * @param node  The IR-node
  */
 unsigned ir_node_hash(const ir_node *node) {
-       unsigned h;
-       int i, irn_arity;
-
-       if (node->op == op_Const) {
-               /* special value for const, as they only differ in their tarval. */
-               h = HASH_PTR(node->attr.con.tv);
-               h = 9*h + HASH_PTR(get_irn_mode(node));
-       } else if (node->op == op_SymConst) {
-               /* special value for const, as they only differ in their symbol. */
-               h = HASH_PTR(node->attr.symc.sym.type_p);
-               h = 9*h + HASH_PTR(get_irn_mode(node));
-       } else {
-
-               /* hash table value = 9*(9*(9*(9*(9*arity+in[0])+in[1])+ ...)+mode)+code */
-               h = irn_arity = get_irn_intra_arity(node);
-
-               /* consider all in nodes... except the block if not a control flow. */
-               for (i = is_cfop(node) ? -1 : 0;  i < irn_arity;  i++) {
-                       h = 9*h + HASH_PTR(get_irn_intra_n(node, i));
-               }
-
-               /* ...mode,... */
-               h = 9*h + HASH_PTR(get_irn_mode(node));
-               /* ...and code */
-               h = 9*h + HASH_PTR(get_irn_op(node));
-       }
-
-       return h;
+       return node->op->ops.hash(node);
 }  /* ir_node_hash */
 
+
 pset *new_identities(void) {
        return new_pset(identities_cmp, N_IR_NODES);
 }  /* new_identities */
@@ -6187,10 +6441,69 @@ ir_node *optimize_in_place(ir_node *n) {
        return optimize_in_place_2(n);
 }  /* optimize_in_place */
 
+/**
+ * Calculate a hash value of a Const node.
+ */
+static unsigned hash_Const(const ir_node *node) {
+       unsigned h;
+
+       /* special value for const, as they only differ in their tarval. */
+       h = HASH_PTR(node->attr.con.tv);
+       h = 9*h + HASH_PTR(get_irn_mode(node));
+
+       return h;
+}  /* hash_Const */
+
+/**
+ * Calculate a hash value of a SymConst node.
+ */
+static unsigned hash_SymConst(const ir_node *node) {
+       unsigned h;
+
+       /* special value for const, as they only differ in their symbol. */
+       h = HASH_PTR(node->attr.symc.sym.type_p);
+       h = 9*h + HASH_PTR(get_irn_mode(node));
+
+       return h;
+}  /* hash_SymConst */
+
+/**
+ * Set the default hash operation in an ir_op_ops.
+ *
+ * @param code   the opcode for the default operation
+ * @param ops    the operations initialized
+ *
+ * @return
+ *    The operations.
+ */
+static ir_op_ops *firm_set_default_hash(ir_opcode code, ir_op_ops *ops)
+{
+#define CASE(a)                                    \
+       case iro_##a:                                  \
+               ops->hash  = hash_##a; \
+               break
+
+       /* hash function already set */
+       if (ops->hash != NULL)
+               return ops;
+
+       switch (code) {
+       CASE(Const);
+       CASE(SymConst);
+       default:
+               /* use input/mode default hash if no function was given */
+               ops->hash = firm_default_hash;
+       }
+
+       return ops;
+#undef CASE
+}
+
 /*
  * Sets the default operation for an ir_ops.
  */
 ir_op_ops *firm_set_default_operations(ir_opcode code, ir_op_ops *ops) {
+       ops = firm_set_default_hash(code, ops);
        ops = firm_set_default_computed_value(code, ops);
        ops = firm_set_default_equivalent_node(code, ops);
        ops = firm_set_default_transform_node(code, ops);