Remove unnecessary conversions to wider modes as operands of Cmp.

[libfirm] / ir / opt / ldstopt.c

diff --git a/ir/opt/ldstopt.c b/ir/opt/ldstopt.c
index 588a55c..84a52a4 100644 (file)
--- a/ir/opt/ldstopt.c
+++ b/ir/opt/ldstopt.c
@@ -979,7 +979,12 @@ static unsigned optimize_phi(ir_node *phi, walk_env_t *wenv)
        if (n <= 0)
                return 0;
 
-       store = skip_Proj(get_Phi_pred(phi, 0));
+       /* must be only one user */
+       projM = get_Phi_pred(phi, 0);
+       if (get_irn_n_edges(projM) != 1)
+               return 0;
+
+       store = skip_Proj(projM);
        old_store = store;
        if (get_irn_op(store) != op_Store)
                return 0;
@@ -1007,8 +1012,12 @@ static unsigned optimize_phi(ir_node *phi, walk_env_t *wenv)
        exc  = info->exc_block;
 
        for (i = 1; i < n; ++i) {
-               ir_node *pred = skip_Proj(get_Phi_pred(phi, i));
+               ir_node *pred = get_Phi_pred(phi, i);
+
+               if (get_irn_n_edges(pred) != 1)
+                       return 0;
 
+               pred = skip_Proj(pred);
                if (get_irn_op(pred) != op_Store)
                        return 0;
 
@@ -1063,18 +1072,18 @@ static unsigned optimize_phi(ir_node *phi, walk_env_t *wenv)
           first because we otherwise may loose a store when exchanging its
           memory Proj.
         */
-       for (i = 0; i < n; ++i)
+       for (i = n - 1; i >= 0; --i) {
+               ir_node *store;
+
                projMs[i] = get_Phi_pred(phi, i);
+               assert(is_Proj(projMs[i]));
 
-       /* first step: collect all inputs and kill the node */
-       for (i = n - 1; i >= 0; --i) {
-               ir_node *store = get_Proj_pred(projMs[i]);
-               info = get_irn_link(store);
+               store = get_Proj_pred(projMs[i]);
+               info  = get_irn_link(store);
 
                inM[i] = get_Store_mem(store);
                inD[i] = get_Store_value(store);
                idx[i] = info->exc_idx;
-               kill_node(store);
        }
        block = get_nodes_block(phi);
 
@@ -1084,11 +1093,15 @@ static unsigned optimize_phi(ir_node *phi, walk_env_t *wenv)
        /* third step: create a new data Phi */
        phiD = new_rd_Phi(get_irn_dbg_info(phi), current_ir_graph, block, n, inD, mode);
 
-       /* rewire memory */
+       /* rewire memory and kill the node */
        for (i = n - 1; i >= 0; --i) {
-               ir_node *proj = projMs[i];
+               ir_node *proj  = projMs[i];
 
-               exchange(proj, inM[i]);
+               if(is_Proj(proj)) {
+                       ir_node *store = get_Proj_pred(proj);
+                       exchange(proj, inM[i]);
+                       kill_node(store);
+               }
        }
 
        /* fourth step: create the Store */