fix opt/jumpthreading7
[libfirm] / ir / ir / iropt.c
index cfa7c3b..679410d 100644 (file)
@@ -672,10 +672,6 @@ static ir_op_ops *firm_set_default_computed_value(ir_opcode code, ir_op_ops *ops
  *
  * If all predecessors of a block are bad or lies in a dead
  * block, the current block is dead as well.
- *
- * Note, that blocks are NEVER turned into Bad's, instead
- * the dead_block flag is set. So, never test for is_Bad(block),
- * always use is_dead_Block(block).
  */
 static ir_node *equivalent_node_Block(ir_node *n)
 {
@@ -683,44 +679,25 @@ static ir_node *equivalent_node_Block(ir_node *n)
        int     n_preds;
        ir_graph *irg;
 
-       /* don't optimize dead or labeled blocks */
-       if (is_Block_dead(n) || has_Block_entity(n))
+       /* don't optimize labeled blocks */
+       if (has_Block_entity(n))
+               return n;
+       if (!get_Block_matured(n))
                return n;
 
        n_preds = get_Block_n_cfgpreds(n);
 
-       /* The Block constructor does not call optimize, but mature_immBlock()
-          calls the optimization. */
-       assert(get_Block_matured(n));
-
        irg = get_irn_irg(n);
 
        /* Straightening: a single entry Block following a single exit Block
-          can be merged, if it is not the Start block. */
-       /* !!! Beware, all Phi-nodes of n must have been optimized away.
-          This should be true, as the block is matured before optimize is called.
-          But what about Phi-cycles with the Phi0/Id that could not be resolved?
-          Remaining Phi nodes are just Ids. */
+        * can be merged. */
        if (n_preds == 1) {
-               ir_node *pred = skip_Proj(get_Block_cfgpred(n, 0));
+               ir_node *pred = get_Block_cfgpred(n, 0);
 
                if (is_Jmp(pred)) {
-                       ir_node *predblock = get_nodes_block(pred);
-                       if (predblock == oldn) {
-                               /* Jmp jumps into the block it is in -- deal self cycle. */
-                               n = set_Block_dead(n);
-                               DBG_OPT_DEAD_BLOCK(oldn, n);
-                       } else {
-                               n = predblock;
-                               DBG_OPT_STG(oldn, n);
-                       }
-               } else if (is_Cond(pred)) {
-                       ir_node *predblock = get_nodes_block(pred);
-                       if (predblock == oldn) {
-                               /* Jmp jumps into the block it is in -- deal self cycle. */
-                               n = set_Block_dead(n);
-                               DBG_OPT_DEAD_BLOCK(oldn, n);
-                       }
+                       ir_node *pred_block = get_nodes_block(pred);
+                       DBG_OPT_STG(n, pred_block);
+                       return pred_block;
                }
        } else if (n_preds == 2) {
                /* Test whether Cond jumps twice to this block
@@ -735,64 +712,18 @@ static ir_node *equivalent_node_Block(ir_node *n)
 
                    if (cond == get_Proj_pred(b) && is_Cond(cond) &&
                        get_irn_mode(get_Cond_selector(cond)) == mode_b) {
-                               /* Also a single entry Block following a single exit Block.  Phis have
-                                  twice the same operand and will be optimized away. */
+                               /* Also a single entry Block following a single exit Block.
+                                * Phis have twice the same operand and will be optimized away.
+                                */
                                n = get_nodes_block(cond);
                                DBG_OPT_IFSIM1(oldn, a, b, n);
                        }
                }
-       } else if (get_opt_unreachable_code() &&
-                  (n != get_irg_start_block(irg)) &&
-                  (n != get_irg_end_block(irg))) {
-               int i;
-
-               /* If all inputs are dead, this block is dead too, except if it is
-                  the start or end block.  This is one step of unreachable code
-                  elimination */
-               for (i = get_Block_n_cfgpreds(n) - 1; i >= 0; --i) {
-                       ir_node *pred = get_Block_cfgpred(n, i);
-                       ir_node *pred_blk;
-
-                       if (is_Bad(pred)) continue;
-                       pred_blk = get_nodes_block(skip_Proj(pred));
-
-                       if (is_Block_dead(pred_blk)) continue;
-
-                       if (pred_blk != n) {
-                               /* really found a living input */
-                               break;
-                       }
-               }
-               if (i < 0) {
-                       n = set_Block_dead(n);
-                       DBG_OPT_DEAD_BLOCK(oldn, n);
-               }
        }
 
        return n;
 }  /* equivalent_node_Block */
 
-/**
- * Returns a equivalent node for a Jmp, a Bad :-)
- * Of course this only happens if the Block of the Jmp is dead.
- */
-static ir_node *equivalent_node_Jmp(ir_node *n)
-{
-       ir_node *oldn = n;
-
-       /* unreachable code elimination */
-       if (is_Block_dead(get_nodes_block(n))) {
-               ir_graph *irg = get_irn_irg(n);
-               n = get_irg_bad(irg);
-               DBG_OPT_DEAD_BLOCK(oldn, n);
-       }
-       return n;
-}  /* equivalent_node_Jmp */
-
-/** Raise is handled in the same way as Jmp. */
-#define equivalent_node_Raise   equivalent_node_Jmp
-
-
 /* We do not evaluate Cond here as we replace it by a new node, a Jmp.
    See transform_node_Proj_Cond(). */
 
@@ -1309,51 +1240,26 @@ static ir_node *equivalent_node_Phi(ir_node *n)
        n_preds = get_Phi_n_preds(n);
 
        block = get_nodes_block(n);
-       /* Control dead */
-       if (is_Block_dead(block)) {
-               ir_graph *irg = get_irn_irg(n);
-               return get_irg_bad(irg);
-       }
 
-       if (n_preds == 0) return n;           /* Phi of dead Region without predecessors. */
+       /* Phi of dead Region without predecessors. */
+       if (n_preds == 0)
+               return n;
 
        /* Find first non-self-referencing input */
        for (i = 0; i < n_preds; ++i) {
                first_val = get_Phi_pred(n, i);
-               if (   (first_val != n)                            /* not self pointer */
-#if 0
-                   /* BEWARE: when the if is changed to 1, Phis will ignore their Bad
-                    * predecessors. Then, Phi nodes in unreachable code might be removed,
-                    * causing nodes pointing to themselev (Adds for instance).
-                    * This is really bad and causes endless recursion on several
-                    * code pathes, so we do NOT optimize such code.
-                    * This is not that bad as it sounds, optimize_cf() removes bad control flow
-                    * (and bad Phi predecessors), so live code is optimized later.
-                    */
-                       && (! is_Bad(get_Block_cfgpred(block, i)))
-#endif
-                  ) {        /* value not dead */
-                       break;          /* then found first value. */
+               /* not self pointer */
+               if (first_val != n) {
+                       /* then found first value. */
+                       break;
                }
        }
 
-       if (i >= n_preds) {
-               ir_graph *irg = get_irn_irg(n);
-               /* A totally Bad or self-referencing Phi (we didn't break the above loop) */
-               return get_irg_bad(irg);
-       }
-
        /* search for rest of inputs, determine if any of these
        are non-self-referencing */
        while (++i < n_preds) {
                ir_node *scnd_val = get_Phi_pred(n, i);
-               if (   (scnd_val != n)
-                   && (scnd_val != first_val)
-#if 0
-                   /* see above */
-                   && (! is_Bad(get_Block_cfgpred(block, i)))
-#endif
-                       ) {
+               if (scnd_val != n && scnd_val != first_val) {
                        break;
                }
        }
@@ -1366,49 +1272,6 @@ static ir_node *equivalent_node_Phi(ir_node *n)
        return n;
 }  /* equivalent_node_Phi */
 
-/**
- * Several optimizations:
- * - fold Sync-nodes, iff they have only one predecessor except
- *   themselves.
- */
-static ir_node *equivalent_node_Sync(ir_node *n)
-{
-       int arity = get_Sync_n_preds(n);
-       int i;
-
-       for (i = 0; i < arity;) {
-               ir_node *pred = get_Sync_pred(n, i);
-               int      j;
-
-               /* Remove Bad predecessors */
-               if (is_Bad(pred)) {
-                       del_Sync_n(n, i);
-                       --arity;
-                       continue;
-               }
-
-               /* Remove duplicate predecessors */
-               for (j = 0;; ++j) {
-                       if (j >= i) {
-                               ++i;
-                               break;
-                       }
-                       if (get_Sync_pred(n, j) == pred) {
-                               del_Sync_n(n, i);
-                               --arity;
-                               break;
-                       }
-               }
-       }
-
-       if (arity == 0) {
-               ir_graph *irg = get_irn_irg(n);
-               return get_irg_bad(irg);
-       }
-       if (arity == 1) return get_Sync_pred(n, 0);
-       return n;
-}  /* equivalent_node_Sync */
-
 /**
  * Optimize Proj(Tuple).
  */
@@ -1473,13 +1336,6 @@ static ir_node *equivalent_node_Proj_CopyB(ir_node *proj)
                        proj = get_CopyB_mem(copyb);
                        DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NOP);
                        break;
-
-               case pn_CopyB_X_except: {
-                       ir_graph *irg = get_irn_irg(proj);
-                       DBG_OPT_EXC_REM(proj);
-                       proj = get_irg_bad(irg);
-                       break;
-               }
                }
        }
        return proj;
@@ -1523,12 +1379,6 @@ static ir_node *equivalent_node_Proj_Bound(ir_node *proj)
                        DBG_OPT_EXC_REM(proj);
                        proj = get_Bound_mem(bound);
                        break;
-               case pn_Bound_X_except: {
-                       ir_graph *irg = get_irn_irg(proj);
-                       DBG_OPT_EXC_REM(proj);
-                       proj = get_irg_bad(irg);
-                       break;
-               }
                case pn_Bound_res:
                        proj = idx;
                        DBG_OPT_ALGSIM0(oldn, proj, FS_OPT_NOP);
@@ -1541,56 +1391,6 @@ static ir_node *equivalent_node_Proj_Bound(ir_node *proj)
        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 *load = get_Proj_pred(proj);
-
-                       /* get the Load address */
-                       const ir_node *addr = get_Load_ptr(load);
-                       const ir_node *confirm;
-
-                       if (value_not_null(addr, &confirm)) {
-                               if (get_Proj_proj(proj) == pn_Load_X_except) {
-                                       ir_graph *irg = get_irn_irg(proj);
-                                       DBG_OPT_EXC_REM(proj);
-                                       return get_irg_bad(irg);
-                               }
-                       }
-               }
-       }
-       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 *store = get_Proj_pred(proj);
-
-                       /* get the load/store address */
-                       const ir_node *addr = get_Store_ptr(store);
-                       const ir_node *confirm;
-
-                       if (value_not_null(addr, &confirm)) {
-                               if (get_Proj_proj(proj) == pn_Store_X_except) {
-                                       ir_graph *irg = get_irn_irg(proj);
-                                       DBG_OPT_EXC_REM(proj);
-                                       return get_irg_bad(irg);
-                               }
-                       }
-               }
-       }
-       return proj;
-}  /* equivalent_node_Proj_Store */
-
 /**
  * Does all optimizations on nodes that must be done on its Projs
  * because of creating new nodes.
@@ -1598,14 +1398,6 @@ static ir_node *equivalent_node_Proj_Store(ir_node *proj)
 static ir_node *equivalent_node_Proj(ir_node *proj)
 {
        ir_node *n = get_Proj_pred(proj);
-
-       if (get_irn_mode(proj) == mode_X) {
-               if (is_Block_dead(get_nodes_block(n))) {
-                       /* Remove dead control flow -- early gigo(). */
-                       ir_graph *irg = get_irn_irg(proj);
-                       return get_irg_bad(irg);
-               }
-       }
        if (n->op->ops.equivalent_node_Proj)
                return n->op->ops.equivalent_node_Proj(proj);
        return proj;
@@ -1785,8 +1577,6 @@ static ir_op_ops *firm_set_default_equivalent_node(ir_opcode code, ir_op_ops *op
 
        switch (code) {
        CASE(Block);
-       CASE(Jmp);
-       CASE(Raise);
        CASE(Eor);
        CASE(Add);
        CASE(Shl);
@@ -1801,13 +1591,10 @@ static ir_op_ops *firm_set_default_equivalent_node(ir_opcode code, ir_op_ops *op
        CASE(And);
        CASE(Conv);
        CASE(Phi);
-       CASE(Sync);
        CASE_PROJ(Tuple);
        CASE_PROJ(Div);
        CASE_PROJ(CopyB);
        CASE_PROJ(Bound);
-       CASE_PROJ(Load);
-       CASE_PROJ(Store);
        CASE(Proj);
        CASE(Id);
        CASE(Mux);
@@ -2859,7 +2646,7 @@ make_tuple:
                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(blk));
-               set_Tuple_pred(n, pn_Div_X_except,  get_irg_bad(irg));
+               set_Tuple_pred(n, pn_Div_X_except,  new_r_Bad(irg, mode_X));
                set_Tuple_pred(n, pn_Div_res,       value);
        }
        return n;
@@ -2951,7 +2738,7 @@ make_tuple:
                turn_into_tuple(n, pn_Mod_max);
                set_Tuple_pred(n, pn_Mod_M,         mem);
                set_Tuple_pred(n, pn_Mod_X_regular, new_r_Jmp(blk));
-               set_Tuple_pred(n, pn_Mod_X_except,  get_irg_bad(irg));
+               set_Tuple_pred(n, pn_Mod_X_except,  new_r_Bad(irg, mode_X));
                set_Tuple_pred(n, pn_Mod_res,       value);
        }
        return n;
@@ -2984,11 +2771,11 @@ static ir_node *transform_node_Cond(ir_node *n)
                jmp = new_r_Jmp(blk);
                turn_into_tuple(n, pn_Cond_max);
                if (ta == tarval_b_true) {
-                       set_Tuple_pred(n, pn_Cond_false, get_irg_bad(irg));
+                       set_Tuple_pred(n, pn_Cond_false, new_r_Bad(irg, mode_X));
                        set_Tuple_pred(n, pn_Cond_true, jmp);
                } else {
                        set_Tuple_pred(n, pn_Cond_false, jmp);
-                       set_Tuple_pred(n, pn_Cond_true, get_irg_bad(irg));
+                       set_Tuple_pred(n, pn_Cond_true, new_r_Bad(irg, mode_X));
                }
                /* We might generate an endless loop, so keep it alive. */
                add_End_keepalive(get_irg_end(irg), blk);
@@ -3107,27 +2894,6 @@ static ir_node *transform_bitwise_distributive(ir_node *n,
        return n;
 }
 
-int ir_is_equality_cmp_0(const ir_node *node)
-{
-       ir_relation relation = get_Cmp_relation(node);
-       ir_node    *left     = get_Cmp_left(node);
-       ir_node    *right    = get_Cmp_right(node);
-       ir_mode    *mode     = get_irn_mode(left);
-
-       /* this probably makes no sense if unordered is involved */
-       assert(!mode_is_float(mode));
-
-       if (!is_Const(right) || !is_Const_null(right))
-               return false;
-       if (relation == ir_relation_equal)
-               return true;
-       if (mode_is_signed(mode) && relation == ir_relation_less_greater)
-               return true;
-       if (relation == ir_relation_greater)
-               return true;
-       return false;
-}
-
 /**
  * Create a 0 constant of given mode.
  */
@@ -3151,35 +2917,35 @@ static ir_node *transform_node_And(ir_node *n)
        vrp_attr *a_vrp, *b_vrp;
 
        if (is_Cmp(a) && is_Cmp(b)) {
-               ir_node *a_left  = get_Cmp_left(a);
-               ir_node *a_right = get_Cmp_left(a);
-               ir_node *b_left  = get_Cmp_left(b);
-               ir_node *b_right = get_Cmp_right(b);
+               ir_node    *a_left     = get_Cmp_left(a);
+               ir_node    *a_right    = get_Cmp_right(a);
+               ir_node    *b_left     = get_Cmp_left(b);
+               ir_node    *b_right    = get_Cmp_right(b);
+               ir_relation a_relation = get_Cmp_relation(a);
+               ir_relation b_relation = get_Cmp_relation(b);
                /* we can combine the relations of two compares with the same
                 * operands */
                if (a_left == b_left && b_left == b_right) {
                        dbg_info   *dbgi         = get_irn_dbg_info(n);
                        ir_node    *block        = get_nodes_block(n);
-                       ir_relation a_relation   = get_Cmp_relation(a);
-                       ir_relation b_relation   = get_Cmp_relation(b);
                        ir_relation new_relation = a_relation & b_relation;
                        return new_rd_Cmp(dbgi, block, a_left, a_right, new_relation);
                }
-               /* Cmp(a, 0) and Cmp(b,0) can be optimized to Cmp(a|b, 0) */
-               if (ir_is_equality_cmp_0(a) && ir_is_equality_cmp_0(b)
-                               && (get_Cmp_relation(a) & ir_relation_equal) == (get_Cmp_relation(b) & ir_relation_equal)) {
-                       dbg_info    *dbgi     = get_irn_dbg_info(n);
-                       ir_node     *block    = get_nodes_block(n);
-                       ir_relation  relation = get_Cmp_relation(a);
-                       ir_mode     *mode     = get_irn_mode(a_left);
-                       ir_node     *n_b_left = b_left;
-                       if (get_irn_mode(n_b_left) != mode) {
-                               n_b_left = new_rd_Conv(dbgi, block, b_left, mode);
-                       }
-                       ir_node     *or       = new_rd_Or(dbgi, block, a_left, b_left, mode);
-                       ir_graph    *irg      = get_irn_irg(n);
-                       ir_node     *zero     = create_zero_const(irg, mode);
-                       return new_rd_Cmp(dbgi, block, or, zero, relation);
+               /* Cmp(a==0) and Cmp(b==0) can be optimized to Cmp(a|b==0) */
+               if (is_Const(a_right) && is_Const_null(a_right)
+                               && is_Const(b_right) && is_Const_null(b_right)
+                               && a_relation == b_relation && a_relation == ir_relation_equal
+                               && !mode_is_float(get_irn_mode(a_left))
+                               && !mode_is_float(get_irn_mode(b_left))) {
+                       dbg_info *dbgi     = get_irn_dbg_info(n);
+                       ir_node  *block    = get_nodes_block(n);
+                       ir_mode  *mode     = get_irn_mode(a_left);
+                       ir_node  *n_b_left = get_irn_mode(b_left) != mode ?
+                               new_rd_Conv(dbgi, block, b_left, mode) : b_left;
+                       ir_node  *or       = new_rd_Or(dbgi, block, a_left, n_b_left, mode);
+                       ir_graph *irg      = get_irn_irg(n);
+                       ir_node  *zero     = create_zero_const(irg, mode);
+                       return new_rd_Cmp(dbgi, block, or, zero, ir_relation_equal);
                }
        }
 
@@ -3563,7 +3329,7 @@ static ir_node *transform_node_Proj_Load(ir_node *proj)
                                if (get_Proj_proj(proj) == pn_Load_X_except) {
                                        ir_graph *irg = get_irn_irg(proj);
                                        DBG_OPT_EXC_REM(proj);
-                                       return get_irg_bad(irg);
+                                       return new_r_Bad(irg, mode_X);
                                } else {
                                        ir_node *blk = get_nodes_block(load);
                                        return new_r_Jmp(blk);
@@ -3595,7 +3361,7 @@ static ir_node *transform_node_Proj_Store(ir_node *proj)
                                if (get_Proj_proj(proj) == pn_Store_X_except) {
                                        ir_graph *irg = get_irn_irg(proj);
                                        DBG_OPT_EXC_REM(proj);
-                                       return get_irg_bad(irg);
+                                       return new_r_Bad(irg, mode_X);
                                } else {
                                        ir_node *blk = get_nodes_block(store);
                                        return new_r_Jmp(blk);
@@ -3637,7 +3403,7 @@ static ir_node *transform_node_Proj_Div(ir_node *proj)
                        ir_graph *irg = get_irn_irg(proj);
                        /* we found an exception handler, remove it */
                        DBG_OPT_EXC_REM(proj);
-                       return get_irg_bad(irg);
+                       return new_r_Bad(irg, mode_X);
                }
 
                case pn_Div_M: {
@@ -3692,7 +3458,7 @@ static ir_node *transform_node_Proj_Mod(ir_node *proj)
                        ir_graph *irg = get_irn_irg(proj);
                        /* we found an exception handler, remove it */
                        DBG_OPT_EXC_REM(proj);
-                       return get_irg_bad(irg);
+                       return new_r_Bad(irg, mode_X);
                }
 
                case pn_Mod_M: {
@@ -3749,7 +3515,7 @@ static ir_node *transform_node_Proj_Cond(ir_node *proj)
                                } else {
                                        ir_graph *irg = get_irn_irg(proj);
                                        /* this case will NEVER be taken, kill it */
-                                       return get_irg_bad(irg);
+                                       return new_r_Bad(irg, mode_X);
                                }
                        }
                } else {
@@ -3763,19 +3529,19 @@ static ir_node *transform_node_Proj_Cond(ir_node *proj)
                                        ir_relation cmp_result = tarval_cmp(b_vrp->range_bottom, tp);
                                        ir_relation cmp_result2 = tarval_cmp(b_vrp->range_top, tp);
 
-                                       if ((cmp_result & ir_relation_greater) == cmp_result && (cmp_result2
-                                                               & ir_relation_less) == cmp_result2) {
+                                       if ((cmp_result & ir_relation_greater) == cmp_result
+                                           && (cmp_result2 & ir_relation_less) == cmp_result2) {
                                                ir_graph *irg = get_irn_irg(proj);
-                                               return get_irg_bad(irg);
+                                               return new_r_Bad(irg, mode_X);
                                        }
                                } else if (b_vrp->range_type == VRP_ANTIRANGE) {
                                        ir_relation cmp_result = tarval_cmp(b_vrp->range_bottom, tp);
                                        ir_relation cmp_result2 = tarval_cmp(b_vrp->range_top, tp);
 
-                                       if ((cmp_result & ir_relation_less_equal) == cmp_result && (cmp_result2
-                                                               & ir_relation_greater_equal) == cmp_result2) {
+                                       if ((cmp_result & ir_relation_less_equal) == cmp_result
+                                            && (cmp_result2 & ir_relation_greater_equal) == cmp_result2) {
                                                ir_graph *irg = get_irn_irg(proj);
-                                               return get_irg_bad(irg);
+                                               return new_r_Bad(irg, mode_X);
                                        }
                                }
 
@@ -3784,7 +3550,7 @@ static ir_node *transform_node_Proj_Cond(ir_node *proj)
                                                                b_vrp->bits_set
                                                                ) == ir_relation_equal)) {
                                        ir_graph *irg = get_irn_irg(proj);
-                                       return get_irg_bad(irg);
+                                       return new_r_Bad(irg, mode_X);
                                }
 
                                if (!(tarval_cmp(
@@ -3794,10 +3560,8 @@ static ir_node *transform_node_Proj_Cond(ir_node *proj)
                                                                tarval_not(b_vrp->bits_not_set))
                                                                 == ir_relation_equal)) {
                                        ir_graph *irg = get_irn_irg(proj);
-                                       return get_irg_bad(irg);
+                                       return new_r_Bad(irg, mode_X);
                                }
-
-
                        }
                }
        }
@@ -4213,19 +3977,6 @@ static ir_node *transform_node_Cmp(ir_node *n)
 
                                /* the following reassociations work only for == and != */
                                if (relation == ir_relation_equal || relation == ir_relation_less_greater) {
-
-#if 0 /* Might be not that good in general */
-                                       /* a-b == 0  ==>  a == b,  a-b != 0  ==>  a != b */
-                                       if (tarval_is_null(tv) && is_Sub(left)) {
-                                               right = get_Sub_right(left);
-                                               left  = get_Sub_left(left);
-
-                                               tv = value_of(right);
-                                               changed = 1;
-                                               DBG_OPT_ALGSIM0(n, n, FS_OPT_CMP_OP_C);
-                                       }
-#endif
-
                                        if (tv != tarval_bad) {
                                                /* a-c1 == c2  ==>  a == c2+c1,  a-c1 != c2  ==>  a != c2+c1 */
                                                if (is_Sub(left)) {
@@ -4506,10 +4257,12 @@ static ir_node *transform_node_Proj_CopyB(ir_node *proj)
                        DBG_OPT_EXC_REM(proj);
                        proj = new_r_Jmp(get_nodes_block(copyb));
                        break;
-               case pn_CopyB_X_except:
+               case pn_CopyB_X_except: {
+                       ir_graph *irg = get_irn_irg(proj);
                        DBG_OPT_EXC_REM(proj);
-                       proj = get_irg_bad(get_irn_irg(proj));
+                       proj = new_r_Bad(irg, mode_X);
                        break;
+               }
                default:
                        break;
                }
@@ -4557,7 +4310,7 @@ static ir_node *transform_node_Proj_Bound(ir_node *proj)
                        break;
                case pn_Bound_X_except:
                        DBG_OPT_EXC_REM(proj);
-                       proj = get_irg_bad(get_irn_irg(proj));
+                       proj = new_r_Bad(get_irn_irg(proj), mode_X);
                        break;
                case pn_Bound_res:
                        proj = idx;
@@ -4587,14 +4340,57 @@ static ir_node *transform_node_Proj(ir_node *proj)
        return proj;
 }  /* transform_node_Proj */
 
-/**
- * Move Confirms down through Phi nodes.
- */
+static bool is_block_unreachable(const ir_node *block)
+{
+       const ir_graph *irg = get_irn_irg(block);
+       if (!is_irg_state(irg, IR_GRAPH_STATE_BAD_BLOCK))
+               return false;
+       return get_Block_dom_depth(block) < 0;
+}
+
+static ir_node *transform_node_Block(ir_node *block)
+{
+       ir_graph *irg   = get_irn_irg(block);
+       int       arity = get_irn_arity(block);
+       ir_node  *bad   = NULL;
+       int       i;
+
+       if (!is_irg_state(irg, IR_GRAPH_STATE_BAD_BLOCK))
+               return block;
+
+       for (i = 0; i < arity; ++i) {
+               ir_node *pred       = get_Block_cfgpred(block, i);
+               ir_node *pred_block = get_nodes_block(pred);
+               if (!is_Bad(pred) && !is_block_unreachable(pred_block))
+                       continue;
+               if (bad == NULL)
+                       bad = new_r_Bad(irg, mode_X);
+               set_irn_n(block, i, bad);
+       }
+
+       return block;
+}
+
 static ir_node *transform_node_Phi(ir_node *phi)
 {
-       int i, n;
-       ir_mode *mode = get_irn_mode(phi);
+       int       n     = get_irn_arity(phi);
+       ir_mode  *mode  = get_irn_mode(phi);
+       ir_node  *block = get_nodes_block(phi);
+       ir_graph *irg   = get_irn_irg(phi);
+       ir_node  *bad   = NULL;
+       int       i;
+
+       /* Set phi-operands for bad-block inputs to bad */
+       for (i = 0; i < n; ++i) {
+               ir_node *pred = get_Block_cfgpred(block, i);
+               if (is_Bad(pred) || is_block_unreachable(get_nodes_block(pred))) {
+                       if (bad == NULL)
+                               bad = new_r_Bad(irg, mode);
+                       set_irn_n(phi, i, bad);
+               }
+       }
 
+       /* Move Confirms down through Phi nodes. */
        if (mode_is_reference(mode)) {
                n = get_irn_arity(phi);
 
@@ -4823,6 +4619,22 @@ static ir_node *transform_node_Or_Rotl(ir_node *irn_or)
        return n;
 }  /* transform_node_Or_Rotl */
 
+static bool is_cmp_unequal_zero(const ir_node *node)
+{
+       ir_relation relation = get_Cmp_relation(node);
+       ir_node    *left     = get_Cmp_left(node);
+       ir_node    *right    = get_Cmp_right(node);
+       ir_mode    *mode     = get_irn_mode(left);
+
+       if (!is_Const(right) || !is_Const_null(right))
+               return false;
+       if (mode_is_signed(mode)) {
+               return relation == ir_relation_less_greater;
+       } else {
+               return relation == ir_relation_greater;
+       }
+}
+
 /**
  * Transform an Or.
  */
@@ -4860,6 +4672,20 @@ static ir_node *transform_node_Or(ir_node *n)
                        ir_relation new_relation = a_relation | b_relation;
                        return new_rd_Cmp(dbgi, block, a_left, a_right, new_relation);
                }
+               /* Cmp(a!=0) or Cmp(b!=0) => Cmp(a|b != 0) */
+               if (is_cmp_unequal_zero(a) && is_cmp_unequal_zero(b)
+                               && !mode_is_float(get_irn_mode(a_left))
+                               && !mode_is_float(get_irn_mode(b_left))) {
+                       ir_graph *irg      = get_irn_irg(n);
+                       dbg_info *dbgi     = get_irn_dbg_info(n);
+                       ir_node  *block    = get_nodes_block(n);
+                       ir_mode  *mode     = get_irn_mode(a_left);
+                       ir_node  *n_b_left = get_irn_mode(b_left) != mode ?
+                               new_rd_Conv(dbgi, block, b_left, mode) : b_left;
+                       ir_node  *or   = new_rd_Or(dbgi, block, a_left, n_b_left, mode);
+                       ir_node  *zero = create_zero_const(irg, mode);
+                       return new_rd_Cmp(dbgi, block, or, zero, ir_relation_less_greater);
+               }
        }
 
        mode = get_irn_mode(n);
@@ -5376,17 +5202,14 @@ static ir_node *transform_node_End(ir_node *n)
 
        for (i = j = 0; i < n_keepalives; ++i) {
                ir_node *ka = get_End_keepalive(n, i);
-               if (is_Block(ka)) {
-                       if (! is_Block_dead(ka)) {
-                               in[j++] = ka;
-                       }
+               ir_node *block;
+               /* no need to keep Bad */
+               if (is_Bad(ka))
                        continue;
-               } else if (is_irn_pinned_in_irg(ka) && is_Block_dead(get_nodes_block(ka))) {
+               /* dont keep unreachable code */
+               block = is_Block(ka) ? ka : get_nodes_block(ka);
+               if (is_block_unreachable(block))
                        continue;
-               } else if (is_Bad(ka)) {
-                       /* no need to keep Bad */
-                       continue;
-               }
                in[j++] = ka;
        }
        if (j != n_keepalives)
@@ -5647,6 +5470,24 @@ static ir_node *transform_node_Sync(ir_node *n)
                int      pred_arity;
                int      j;
 
+               /* Remove Bad predecessors */
+               if (is_Bad(pred)) {
+                       del_Sync_n(n, i);
+                       --arity;
+                       continue;
+               }
+
+               /* Remove duplicate predecessors */
+               for (j = 0; j < i; ++j) {
+                       if (get_Sync_pred(n, j) == pred) {
+                               del_Sync_n(n, i);
+                               --arity;
+                               break;
+                       }
+               }
+               if (j < i)
+                       continue;
+
                if (!is_Sync(pred)) {
                        ++i;
                        continue;
@@ -5671,11 +5512,78 @@ static ir_node *transform_node_Sync(ir_node *n)
                }
        }
 
+       if (arity == 0) {
+               ir_graph *irg = get_irn_irg(n);
+               return new_r_Bad(irg, mode_M);
+       }
+       if (arity == 1) {
+               return get_Sync_pred(n, 0);
+       }
+
        /* rehash the sync node */
        add_identities(n);
+       return n;
+}
+
+static ir_node *transform_node_Load(ir_node *n)
+{
+       /* if our memory predecessor is a load from the same address, then reuse the
+        * previous result */
+       ir_node *mem = get_Load_mem(n);
+       ir_node *mem_pred;
+
+       if (!is_Proj(mem))
+               return n;
+       /* don't touch volatile loads */
+       if (get_Load_volatility(n) == volatility_is_volatile)
+               return n;
+       mem_pred = get_Proj_pred(mem);
+       if (is_Load(mem_pred)) {
+               ir_node *pred_load = mem_pred;
+
+               /* conservatively compare the 2 loads. TODO: This could be less strict
+                * with fixup code in some situations (like smaller/bigger modes) */
+               if (get_Load_ptr(pred_load) != get_Load_ptr(n))
+                       return n;
+               if (get_Load_mode(pred_load) != get_Load_mode(n))
+                       return n;
+               /* all combinations of aligned/unaligned pred/n should be fine so we do
+                * not compare the unaligned attribute */
+               {
+                       ir_node  *block = get_nodes_block(n);
+                       ir_node  *jmp   = new_r_Jmp(block);
+                       ir_graph *irg   = get_irn_irg(n);
+                       ir_node  *bad   = new_r_Bad(irg, mode_X);
+                       ir_mode  *mode  = get_Load_mode(n);
+                       ir_node  *res   = new_r_Proj(pred_load, mode, pn_Load_res);
+                       ir_node  *in[pn_Load_max] = { mem, jmp, bad, res };
+                       ir_node  *tuple = new_r_Tuple(block, ARRAY_SIZE(in), in);
+                       return tuple;
+               }
+       } else if (is_Store(mem_pred)) {
+               ir_node *pred_store = mem_pred;
+               ir_node *value      = get_Store_value(pred_store);
+
+               if (get_Store_ptr(pred_store) != get_Load_ptr(n))
+                       return n;
+               if (get_irn_mode(value) != get_Load_mode(n))
+                       return n;
+               /* all combinations of aligned/unaligned pred/n should be fine so we do
+                * not compare the unaligned attribute */
+               {
+                       ir_node  *block = get_nodes_block(n);
+                       ir_node  *jmp   = new_r_Jmp(block);
+                       ir_graph *irg   = get_irn_irg(n);
+                       ir_node  *bad   = new_r_Bad(irg, mode_X);
+                       ir_node  *res   = value;
+                       ir_node  *in[pn_Load_max] = { mem, jmp, bad, res };
+                       ir_node  *tuple = new_r_Tuple(block, ARRAY_SIZE(in), in);
+                       return tuple;
+               }
+       }
 
        return n;
-}  /* transform_node_Sync */
+}
 
 /**
  * optimize a trampoline Call into a direct Call
@@ -5732,9 +5640,6 @@ static ir_node *transform_node_Call(ir_node *call)
        }
        var = get_method_variadicity(mtp);
        set_method_variadicity(ctp, var);
-       if (var == variadicity_variadic) {
-               set_method_first_variadic_param_index(ctp, get_method_first_variadic_param_index(mtp) + 1);
-       }
        /* When we resolve a trampoline, the function must be called by a this-call */
        set_method_calling_convention(ctp, get_method_calling_convention(mtp) | cc_this_call);
        set_method_additional_properties(ctp, get_method_additional_properties(mtp));
@@ -5802,6 +5707,7 @@ static ir_op_ops *firm_set_default_transform_node(ir_opcode code, ir_op_ops *ops
        switch (code) {
        CASE(Add);
        CASE(And);
+       CASE(Block);
        CASE(Call);
        CASE(Cmp);
        CASE(Conv);
@@ -5823,10 +5729,10 @@ static ir_op_ops *firm_set_default_transform_node(ir_opcode code, ir_op_ops *ops
        CASE(Sync);
        CASE_PROJ(Bound);
        CASE_PROJ(CopyB);
-       CASE_PROJ(Load);
        CASE_PROJ(Store);
        CASE_PROJ_EX(Cond);
        CASE_PROJ_EX(Div);
+       CASE_PROJ_EX(Load);
        CASE_PROJ_EX(Mod);
        default:
          /* leave NULL */;
@@ -5931,7 +5837,7 @@ static int node_cmp_attr_Load(const ir_node *a, const ir_node *b)
                /* NEVER do CSE on volatile Loads */
                return 1;
        /* do not CSE Loads with different alignment. Be conservative. */
-       if (get_Load_align(a) != get_Load_align(b))
+       if (get_Load_unaligned(a) != get_Load_unaligned(b))
                return 1;
 
        return get_Load_mode(a) != get_Load_mode(b);
@@ -5941,7 +5847,7 @@ static int node_cmp_attr_Load(const ir_node *a, const ir_node *b)
 static int node_cmp_attr_Store(const ir_node *a, const ir_node *b)
 {
        /* do not CSE Stores with different alignment. Be conservative. */
-       if (get_Store_align(a) != get_Store_align(b))
+       if (get_Store_unaligned(a) != get_Store_unaligned(b))
                return 1;
 
        /* NEVER do CSE on volatile Stores */
@@ -6015,29 +5921,31 @@ static int node_cmp_attr_ASM(const ir_node *a, const ir_node *b)
        /* Should we really check the constraints here? Should be better, but is strange. */
        n = get_ASM_n_input_constraints(a);
        if (n != get_ASM_n_input_constraints(b))
-               return 0;
+               return 1;
 
        ca = get_ASM_input_constraints(a);
        cb = get_ASM_input_constraints(b);
        for (i = 0; i < n; ++i) {
-               if (ca[i].pos != cb[i].pos || ca[i].constraint != cb[i].constraint)
+               if (ca[i].pos != cb[i].pos || ca[i].constraint != cb[i].constraint
+                   || ca[i].mode != cb[i].mode)
                        return 1;
        }
 
        n = get_ASM_n_output_constraints(a);
        if (n != get_ASM_n_output_constraints(b))
-               return 0;
+               return 1;
 
        ca = get_ASM_output_constraints(a);
        cb = get_ASM_output_constraints(b);
        for (i = 0; i < n; ++i) {
-               if (ca[i].pos != cb[i].pos || ca[i].constraint != cb[i].constraint)
+               if (ca[i].pos != cb[i].pos || ca[i].constraint != cb[i].constraint
+                   || ca[i].mode != cb[i].mode)
                        return 1;
        }
 
        n = get_ASM_n_clobbers(a);
        if (n != get_ASM_n_clobbers(b))
-               return 0;
+               return 1;
 
        cla = get_ASM_clobbers(a);
        clb = get_ASM_clobbers(b);
@@ -6227,7 +6135,7 @@ ir_node *identify_remember(ir_node *n)
 
        if (nn != n) {
                /* n is reachable again */
-               edges_node_revival(nn, get_irn_irg(nn));
+               edges_node_revival(nn);
        }
 
        return nn;
@@ -6276,100 +6184,6 @@ void visit_all_identities(ir_graph *irg, irg_walk_func visit, void *env)
        current_ir_graph = rem;
 }  /* visit_all_identities */
 
-/**
- * Garbage in, garbage out. If a node has a dead input, i.e., the
- * Bad node is input to the node, return the Bad node.
- */
-static ir_node *gigo(ir_node *node)
-{
-       int i, irn_arity;
-       ir_op *op = get_irn_op(node);
-
-       /* remove garbage blocks by looking at control flow that leaves the block
-          and replacing the control flow by Bad. */
-       if (get_irn_mode(node) == mode_X) {
-               ir_node  *block = get_nodes_block(skip_Proj(node));
-               ir_graph *irg   = get_irn_irg(block);
-
-               /* Don't optimize nodes in immature blocks. */
-               if (!get_Block_matured(block))
-                       return node;
-               /* Don't optimize End, may have Bads. */
-               if (op == op_End) return node;
-
-               if (is_Block(block)) {
-                       if (is_Block_dead(block)) {
-                               /* control flow from dead block is dead */
-                               return get_irg_bad(irg);
-                       }
-
-                       for (i = get_irn_arity(block) - 1; i >= 0; --i) {
-                               if (!is_Bad(get_irn_n(block, i)))
-                                       break;
-                       }
-                       if (i < 0) {
-                               ir_graph *irg = get_irn_irg(block);
-                               /* the start block is never dead */
-                               if (block != get_irg_start_block(irg)
-                                       && block != get_irg_end_block(irg)) {
-                                       /*
-                                        * Do NOT kill control flow without setting
-                                        * the block to dead of bad things can happen:
-                                        * We get a Block that is not reachable be irg_block_walk()
-                                        * but can be found by irg_walk()!
-                                        */
-                                       set_Block_dead(block);
-                                       return get_irg_bad(irg);
-                               }
-                       }
-               }
-       }
-
-       /* Blocks, Phis and Tuples may have dead inputs, e.g., if one of the
-          blocks predecessors is dead. */
-       if (op != op_Block && op != op_Phi && op != op_Tuple && op != op_Anchor) {
-               ir_graph *irg = get_irn_irg(node);
-               irn_arity = get_irn_arity(node);
-
-               /*
-                * Beware: we can only read the block of a non-floating node.
-                */
-               if (is_irn_pinned_in_irg(node) &&
-                       is_Block_dead(get_nodes_block(skip_Proj(node))))
-                       return get_irg_bad(irg);
-
-               for (i = 0; i < irn_arity; i++) {
-                       ir_node *pred = get_irn_n(node, i);
-
-                       if (is_Bad(pred))
-                               return get_irg_bad(irg);
-#if 0
-                       /* Propagating Unknowns here seems to be a bad idea, because
-                          sometimes we need a node as a input and did not want that
-                          it kills its user.
-                          However, it might be useful to move this into a later phase
-                          (if you think that optimizing such code is useful). */
-                       if (is_Unknown(pred) && mode_is_data(get_irn_mode(node)))
-                               return new_r_Unknown(irg, get_irn_mode(node));
-#endif
-               }
-       }
-#if 0
-       /* With this code we violate the agreement that local_optimize
-          only leaves Bads in Block, Phi and Tuple nodes. */
-       /* If Block has only Bads as predecessors it's garbage. */
-       /* If Phi has only Bads as predecessors it's garbage. */
-       if ((op == op_Block && get_Block_matured(node)) || op == op_Phi)  {
-               irn_arity = get_irn_arity(node);
-               for (i = 0; i < irn_arity; i++) {
-                       if (!is_Bad(get_irn_n(node, i))) break;
-               }
-               if (i == irn_arity) node = get_irg_bad(irg);
-       }
-#endif
-       return node;
-}  /* gigo */
-
 /**
  * These optimizations deallocate nodes from the obstack.
  * It can only be called if it is guaranteed that no other nodes
@@ -6387,17 +6201,6 @@ ir_node *optimize_node(ir_node *n)
        /* Always optimize Phi nodes: part of the construction. */
        if ((!get_opt_optimize()) && (iro != iro_Phi)) return n;
 
-       /* Remove nodes with dead (Bad) input.
-          Run always for transformation induced Bads. */
-       n = gigo(n);
-       if (n != oldn) {
-               edges_node_deleted(oldn, irg);
-
-               /* We found an existing, better node, so we can deallocate the old node. */
-               irg_kill_node(irg, oldn);
-               return n;
-       }
-
        /* constant expression evaluation / constant folding */
        if (get_opt_constant_folding()) {
                /* neither constants nor Tuple values can be evaluated */
@@ -6422,7 +6225,7 @@ ir_node *optimize_node(ir_node *n)
                                memcpy(oldn->in, n->in, ARR_LEN(n->in) * sizeof(n->in[0]));
 
                                /* note the inplace edges module */
-                               edges_node_deleted(n, irg);
+                               edges_node_deleted(n);
 
                                /* evaluation was successful -- replace the node. */
                                irg_kill_node(irg, n);
@@ -6453,7 +6256,7 @@ ir_node *optimize_node(ir_node *n)
                n = identify_cons(n);
 
        if (n != oldn) {
-               edges_node_deleted(oldn, irg);
+               edges_node_deleted(oldn);
 
                /* We found an existing, better node, so we can deallocate the old node. */
                irg_kill_node(irg, oldn);
@@ -6496,12 +6299,6 @@ ir_node *optimize_in_place_2(ir_node *n)
        if (iro == iro_Deleted)
                return n;
 
-       /* Remove nodes with dead (Bad) input.
-          Run always for transformation induced Bads.  */
-       n = gigo(n);
-       if (is_Bad(n))
-               return n;
-
        /* constant expression evaluation / constant folding */
        if (get_opt_constant_folding()) {
                /* neither constants nor Tuple values can be evaluated */
@@ -6574,8 +6371,6 @@ ir_node *optimize_in_place(ir_node *n)
 
        if (get_opt_global_cse())
                set_irg_pinned(irg, op_pin_state_floats);
-       if (get_irg_outs_state(irg) == outs_consistent)
-               set_irg_outs_inconsistent(irg);
 
        /* FIXME: Maybe we could also test whether optimizing the node can
           change the control graph. */
@@ -6618,7 +6413,7 @@ static unsigned hash_SymConst(const ir_node *node)
  * @return
  *    The operations.
  */
-static ir_op_ops *firm_set_default_hash(ir_opcode code, ir_op_ops *ops)
+static ir_op_ops *firm_set_default_hash(unsigned code, ir_op_ops *ops)
 {
 #define CASE(a)                                    \
        case iro_##a:                                  \
@@ -6644,7 +6439,7 @@ static ir_op_ops *firm_set_default_hash(ir_opcode code, ir_op_ops *ops)
 /*
  * Sets the default operation for an ir_ops.
  */
-ir_op_ops *firm_set_default_operations(ir_opcode code, ir_op_ops *ops)
+ir_op_ops *firm_set_default_operations(unsigned code, ir_op_ops *ops)
 {
        ops = firm_set_default_hash(code, ops);
        ops = firm_set_default_computed_value(code, ops);