Index: tree-ssa-loop-manip.c =================================================================== --- tree-ssa-loop-manip.c (revision 127266) +++ tree-ssa-loop-manip.c (working copy) @@ -104,7 +104,7 @@ create_iv (tree base, tree step, tree va } /* Gimplify the step if necessary. We put the computations in front of the loop (i.e. the step should be loop invariant). */ - step = force_gimple_operand (step, &stmts, true, var); + step = force_gimple_operand (step, &stmts, true, NULL_TREE); if (stmts) bsi_insert_on_edge_immediate (pe, stmts); Index: tree-ssa-loop-ivopts.c =================================================================== --- tree-ssa-loop-ivopts.c (revision 127266) +++ tree-ssa-loop-ivopts.c (working copy) @@ -901,7 +901,12 @@ find_bivs (struct ivopts_data *data) type = TREE_TYPE (PHI_RESULT (phi)); base = fold_convert (type, base); if (step) - step = fold_convert (type, step); + { + if (POINTER_TYPE_P (type)) + step = fold_convert (sizetype, step); + else + step = fold_convert (type, step); + } set_iv (data, PHI_RESULT (phi), base, step); found = true; @@ -1934,7 +1939,9 @@ add_candidate_1 (struct ivopts_data *dat { orig_type = TREE_TYPE (base); type = generic_type_for (orig_type); - if (type != orig_type) + /* Don't convert the base to the generic type for pointers as the generic + type is an integer type with the same size as the pointer type. */ + if (type != orig_type && !POINTER_TYPE_P (orig_type)) { base = fold_convert (type, base); step = fold_convert (type, step); @@ -2131,13 +2138,17 @@ add_iv_value_candidates (struct ivopts_d { unsigned HOST_WIDE_INT offset; tree base; + tree basetype; add_candidate (data, iv->base, iv->step, false, use); /* The same, but with initial value zero. Make such variable important, since it is generic enough so that possibly many uses may be based on it. */ - add_candidate (data, build_int_cst (TREE_TYPE (iv->base), 0), + basetype = TREE_TYPE (iv->base); + if (POINTER_TYPE_P (basetype)) + basetype = sizetype; + add_candidate (data, build_int_cst (basetype, 0), iv->step, true, use); /* Third, try removing the constant offset. */ @@ -3562,10 +3573,13 @@ cand_value_at (struct loop *loop, struct aff_tree step, delta, nit; struct iv *iv = cand->iv; tree type = TREE_TYPE (iv->base); + tree steptype = type; + if (POINTER_TYPE_P (type)) + steptype = sizetype; - tree_to_aff_combination (iv->step, type, &step); + tree_to_aff_combination (iv->step, steptype, &step); tree_to_aff_combination (niter, TREE_TYPE (niter), &nit); - aff_combination_convert (&nit, type); + aff_combination_convert (&nit, steptype); aff_combination_mult (&nit, &step, &delta); if (stmt_after_increment (loop, cand, at)) aff_combination_add (&delta, &step); Index: tree-affine.c =================================================================== --- tree-affine.c (revision 127266) +++ tree-affine.c (working copy) @@ -279,7 +279,6 @@ tree_to_aff_combination (tree expr, tree case POINTER_PLUS_EXPR: tree_to_aff_combination (TREE_OPERAND (expr, 0), type, comb); tree_to_aff_combination (TREE_OPERAND (expr, 1), sizetype, &tmp); - aff_combination_convert (&tmp, type); aff_combination_add (comb, &tmp); return; @@ -350,29 +349,40 @@ add_elt_to_tree (tree expr, tree type, t aff_tree *comb) { enum tree_code code; + tree type1 = type; + if (POINTER_TYPE_P (type)) + type1 = sizetype; scale = double_int_ext_for_comb (scale, comb); - elt = fold_convert (type, elt); + elt = fold_convert (type1, elt); if (double_int_one_p (scale)) { if (!expr) - return elt; + return fold_convert (type, elt); + if (POINTER_TYPE_P (type)) + return fold_build2 (POINTER_PLUS_EXPR, type, expr, elt); return fold_build2 (PLUS_EXPR, type, expr, elt); } if (double_int_minus_one_p (scale)) { if (!expr) - return fold_build1 (NEGATE_EXPR, type, elt); + return fold_convert (type, fold_build1 (NEGATE_EXPR, type1, elt)); + if (POINTER_TYPE_P (type)) + { + elt = fold_build1 (NEGATE_EXPR, type1, elt); + return fold_build2 (POINTER_PLUS_EXPR, type, expr, elt); + } return fold_build2 (MINUS_EXPR, type, expr, elt); } if (!expr) - return fold_build2 (MULT_EXPR, type, elt, - double_int_to_tree (type, scale)); + return fold_convert (type, + fold_build2 (MULT_EXPR, type1, elt, + double_int_to_tree (type1, scale))); if (double_int_negative_p (scale)) { @@ -382,8 +392,14 @@ add_elt_to_tree (tree expr, tree type, t else code = PLUS_EXPR; - elt = fold_build2 (MULT_EXPR, type, elt, - double_int_to_tree (type, scale)); + elt = fold_build2 (MULT_EXPR, type1, elt, + double_int_to_tree (type1, scale)); + if (POINTER_TYPE_P (type)) + { + if (code == MINUS_EXPR) + elt = fold_build1 (NEGATE_EXPR, type1, elt); + return fold_build2 (POINTER_PLUS_EXPR, type, expr, elt); + } return fold_build2 (code, type, expr, elt); } @@ -396,6 +412,9 @@ aff_combination_to_tree (aff_tree *comb) tree expr = comb->rest; unsigned i; double_int off, sgn; + tree type1 = type; + if (POINTER_TYPE_P (type)) + type1 = sizetype; gcc_assert (comb->n == MAX_AFF_ELTS || comb->rest == NULL_TREE); @@ -415,7 +434,7 @@ aff_combination_to_tree (aff_tree *comb) off = comb->offset; sgn = double_int_one; } - return add_elt_to_tree (expr, type, double_int_to_tree (type, off), sgn, + return add_elt_to_tree (expr, type, double_int_to_tree (type1, off), sgn, comb); }