Correct inaccurate comment: The only corner case of wrong CF in NEG+ADD as replacemen...

[libfirm] / ir / lower / lower_hl.c

diff --git a/ir/lower/lower_hl.c b/ir/lower/lower_hl.c
index ca574e1..ab4f599 100644 (file)
--- a/ir/lower/lower_hl.c
+++ b/ir/lower/lower_hl.c
@@ -71,18 +71,18 @@ static void lower_sel(ir_node *sel)
 
                if (is_Array_type(owner)) {
                        ir_type *arr_ty = owner;
-                       int      dims   = get_array_n_dimensions(arr_ty);
-                       int     *map    = ALLOCAN(int, dims);
+                       size_t   dims   = get_array_n_dimensions(arr_ty);
+                       size_t  *map    = ALLOCAN(size_t, dims);
                        ir_mode *mode_Int = get_reference_mode_signed_eq(mode);
                        ir_tarval *tv;
                        ir_node *last_size;
-                       int      i;
+                       size_t   i;
 
-                       assert(dims == get_Sel_n_indexs(sel)
+                       assert(dims == (size_t)get_Sel_n_indexs(sel)
                                && "array dimension must match number of indices of Sel node");
 
                        for (i = 0; i < dims; i++) {
-                               int order = get_array_order(arr_ty, i);
+                               size_t order = get_array_order(arr_ty, i);
 
                                assert(order < dims &&
                                        "order of a dimension must be smaller than the arrays dim");
@@ -104,8 +104,8 @@ static void lower_sel(ir_node *sel)
                         * whereas u_i is the upper bound of the current dimension
                         * and l_i the lower bound of the current dimension.
                         */
-                       for (i = dims - 1; i >= 0; i--) {
-                               int dim = map[i];
+                       for (i = dims; i > 0;) {
+                               size_t dim = map[--i];
                                ir_node *lb, *ub, *elms, *n, *ind;
 
                                elms = NULL;
@@ -297,7 +297,7 @@ static int is_integral_size(int size)
 static void lower_bitfields_loads(ir_node *proj, ir_node *load)
 {
        ir_node *sel = get_Load_ptr(load);
-       ir_node *block, *n_proj, *res, *ptr;
+       ir_node *block, *res, *ptr;
        ir_graph *irg;
        ir_entity *ent;
        ir_type *bf_type;
@@ -350,7 +350,7 @@ static void lower_bitfields_loads(ir_node *proj, ir_node *load)
        /* create new proj, switch off CSE or we may get the old one back */
        old_cse = get_opt_cse();
        set_opt_cse(0);
-       res = n_proj = new_r_Proj(load, mode, pn_Load_res);
+       res = new_r_Proj(load, mode, pn_Load_res);
        set_opt_cse(old_cse);
 
        if (mode_is_signed(mode)) { /* signed */
@@ -523,8 +523,6 @@ void lower_highlevel_graph(ir_graph *irg, int lower_bitfields)
 
        /* Finally: lower SymConst-Size and Sel nodes, Casts, unaligned Load/Stores. */
        irg_walk_graph(irg, NULL, lower_irnode, NULL);
-
-       set_irg_outs_inconsistent(irg);
 }
 
 typedef struct pass_t {