rename tarval to ir_tarval
[libfirm] / ir / opt / cfopt.c
index 326edce..68b1221 100644 (file)
@@ -104,9 +104,10 @@ static int remove_senseless_conds(ir_node *bl)
                                ir_node *cond_j = skip_Proj(pred_j);
 
                                if (cond_j == cond_i) {
-                                       ir_node *jmp = new_r_Jmp(get_nodes_block(cond_i));
+                                       ir_graph *irg = get_irn_irg(bl);
+                                       ir_node  *jmp = new_r_Jmp(get_nodes_block(cond_i));
                                        set_irn_n(bl, i, jmp);
-                                       set_irn_n(bl, j, new_Bad());
+                                       set_irn_n(bl, j, new_r_Bad(irg));
 
                                        DBG_OPT_IFSIM2(cond_i, jmp);
                                        changed = 1;
@@ -171,9 +172,6 @@ static void merge_blocks(ir_node *node, void *ctx)
                                   promote it directly below. Nevertheless, we sometimes reach a block
                                   the first time through a dataflow node.  In this case we optimized the
                                   block as such and have to promote the Bad here. */
-                               assert((get_opt_control_flow_straightening() ||
-                                       get_opt_control_flow_weak_simplification()) &&
-                                       ("strange flag setting"));
                                exchange(b, new_block);
                                env->changed = 1;
                                b = new_block;
@@ -184,7 +182,8 @@ static void merge_blocks(ir_node *node, void *ctx)
                         * prevented, so just set it's cf to Bad.
                         */
                        if (is_Block_dead(new_block)) {
-                               exchange(node, new_Bad());
+                               ir_graph *irg = get_irn_irg(node);
+                               exchange(node, new_r_Bad(irg));
                                env->changed = 1;
                        }
                }
@@ -216,8 +215,9 @@ static void remove_unreachable_blocks_and_conds(ir_node *block, void *env)
                        ir_node *pred_bl = get_nodes_block(skip_Proj(skipped));
 
                        if (is_Block_dead(pred_bl) || (get_Block_dom_depth(pred_bl) < 0)) {
+                               ir_graph *irg = get_irn_irg(block);
                                set_Block_dead(pred_bl);
-                               exchange(pred_X, new_Bad());
+                               exchange(pred_X, new_r_Bad(irg));
                                *changed = 1;
                        } else if (skipped != pred_X) {
                                set_Block_cfgpred(block, i, skipped);
@@ -327,12 +327,6 @@ static int test_whether_dispensable(ir_node *b, int pos)
                return 0;
 
        if (is_Block_removable(pred)) {
-               if (!get_opt_optimize() || !get_opt_control_flow_strong_simplification()) {
-                       /* Mark block so that is will not be removed: optimization is turned off. */
-                       set_Block_non_removable(pred);
-                       return 1;
-               }
-
                /* Seems to be empty. At least we detected this in collect_nodes. */
                if (get_irn_link(b) == NULL) {
                        /* There are no Phi nodes ==> all predecessors are dispensable. */
@@ -497,7 +491,8 @@ static void optimize_blocks(ir_node *b, void *ctx)
 
                                if (get_Block_idom(b) != predb) {
                                        /* predb is not the dominator. There can't be uses of pred's Phi nodes, kill them .*/
-                                       exchange(phi, new_Bad());
+                                       ir_graph *irg = get_irn_irg(b);
+                                       exchange(phi, new_r_Bad(irg));
                                } else {
                                        /* predb is the direct dominator of b. There might be uses of the Phi nodes from
                                           predb in further block, so move this phi from the predecessor into the block b */
@@ -581,7 +576,7 @@ static void optimize_blocks(ir_node *b, void *ctx)
                                        in[n_preds++] = pred_X;
                        }
                        /* Remove block as it might be kept alive. */
-                       exchange(pred, b/*new_Bad()*/);
+                       exchange(pred, b/*new_r_Bad(irg)*/);
                } else {
                        /* case 3: */
                        in[n_preds++] = get_Block_cfgpred(b, i);
@@ -643,19 +638,20 @@ static int handle_switch_cond(ir_node *cond)
        } else if (get_irn_link(proj2) == NULL) {
                /* We have two Proj's here. Check if the Cond has
                   a constant argument */
-               tarval *tv = value_of(sel);
+               ir_tarval *tv = value_of(sel);
 
                if (tv != tarval_bad) {
                        /* we have a constant switch */
-                       long num     = get_tarval_long(tv);
-                       long def_num = get_Cond_default_proj(cond);
+                       long      num     = get_tarval_long(tv);
+                       long      def_num = get_Cond_default_proj(cond);
+                       ir_graph *irg     = get_irn_irg(cond);
 
                        if (def_num == get_Proj_proj(proj1)) {
                                /* first one is the defProj */
                                if (num == get_Proj_proj(proj2)) {
                                        jmp = new_r_Jmp(blk);
                                        exchange(proj2, jmp);
-                                       exchange(proj1, new_Bad());
+                                       exchange(proj1, new_r_Bad(irg));
                                        return 1;
                                }
                        } else if (def_num == get_Proj_proj(proj2)) {
@@ -663,7 +659,7 @@ static int handle_switch_cond(ir_node *cond)
                                if (num == get_Proj_proj(proj1)) {
                                        jmp = new_r_Jmp(blk);
                                        exchange(proj1, jmp);
-                                       exchange(proj2, new_Bad());
+                                       exchange(proj2, new_r_Bad(irg));
                                        return 1;
                                }
                        } else {
@@ -671,12 +667,12 @@ static int handle_switch_cond(ir_node *cond)
                                if (num == get_Proj_proj(proj1)) {
                                        jmp = new_r_Jmp(blk);
                                        exchange(proj1, jmp);
-                                       exchange(proj2, new_Bad());
+                                       exchange(proj2, new_r_Bad(irg));
                                        return 1;
                                } else if (num == get_Proj_proj(proj2)) {
                                        jmp = new_r_Jmp(blk);
                                        exchange(proj2, jmp);
-                                       exchange(proj1, new_Bad());
+                                       exchange(proj1, new_r_Bad(irg));
                                        return 1;
                                }
                        }
@@ -706,7 +702,6 @@ void optimize_cf(ir_graph *irg)
        int i, j, n, changed;
        ir_node **in = NULL;
        ir_node *cond, *end = get_irg_end(irg);
-       ir_graph *rem = current_ir_graph;
        plist_element_t *el;
        merge_env env;
 
@@ -716,8 +711,6 @@ void optimize_cf(ir_graph *irg)
        assert(get_irg_pinned(irg) != op_pin_state_floats &&
               "Control flow optimization need a pinned graph");
 
-       current_ir_graph = irg;
-
        /* FIXME: control flow opt destroys block edges. So edges are deactivated here. Fix the edges! */
        edges_deactivate(irg);
 
@@ -740,7 +733,7 @@ restart:
                if (is_Block(ka)) {
                        /* do NOT keep dead blocks */
                        if (is_Block_dead(ka) || get_Block_dom_depth(ka) < 0) {
-                               set_End_keepalive(end, i, new_Bad());
+                               set_End_keepalive(end, i, new_r_Bad(irg));
                                changed = 1;
                        }
                } else {
@@ -748,7 +741,7 @@ restart:
 
                        if (is_Bad(block) || is_Block_dead(block) || get_Block_dom_depth(block) < 0) {
                                /* do NOT keep nodes in dead blocks */
-                               set_End_keepalive(end, i, new_Bad());
+                               set_End_keepalive(end, i, new_r_Bad(irg));
                                changed = 1;
                        }
                }
@@ -802,7 +795,7 @@ restart:
                if (is_Block(ka)) {
                        /* do NOT keep dead blocks */
                        if (is_Block_dead(ka) || get_Block_dom_depth(ka) < 0) {
-                               set_End_keepalive(end, i, new_Bad());
+                               set_End_keepalive(end, i, new_r_Bad(irg));
                                changed = 1;
                        }
                } else {
@@ -810,7 +803,7 @@ restart:
 
                        if (is_Bad(block) || is_Block_dead(block) || get_Block_dom_depth(block) < 0) {
                                /* do NOT keep nodes in dead blocks */
-                               set_End_keepalive(end, i, new_Bad());
+                               set_End_keepalive(end, i, new_r_Bad(irg));
                                changed = 1;
                        }
                }
@@ -880,8 +873,6 @@ restart:
                        fprintf(stderr, "VERIFY_BAD in optimize_cf()\n");
                }
        }
-
-       current_ir_graph = rem;
 }
 
 /* Creates an ir_graph pass for optimize_cf. */