re PR middle-end/44203 (New prefetch test failures)
2010-05-25 Christian Borntraeger <borntraeger@de.ibm.com> PR 44203 * tree-ssa-loop-prefetch.c: Fix logic for step calculation to match the original (and intended) behaviour before r159557. This changeset changed a=a+b*c to a=(a+b)*b which was obviously wrong in two ways. From-SVN: r159816
This commit is contained in:
parent
be6d3f0e89
commit
8fde8b401d
|
@ -1,3 +1,11 @@
|
||||||
|
2010-05-25 Christian Borntraeger <borntraeger@de.ibm.com>
|
||||||
|
|
||||||
|
PR 44203
|
||||||
|
* tree-ssa-loop-prefetch.c: Fix logic for step calculation to
|
||||||
|
match the original (and intended) behaviour before r159557. This
|
||||||
|
changeset changed a=a+b*c to a=(a+b)*b which was obviously wrong
|
||||||
|
in two ways.
|
||||||
|
|
||||||
2010-05-25 Richard Guenther <rguenther@suse.de>
|
2010-05-25 Richard Guenther <rguenther@suse.de>
|
||||||
|
|
||||||
* doc/invoke.texi: Document -Ofast.
|
* doc/invoke.texi: Document -Ofast.
|
||||||
|
|
|
@ -423,25 +423,24 @@ idx_analyze_ref (tree base, tree *index, void *data)
|
||||||
ibase = build_int_cst (TREE_TYPE (ibase), 0);
|
ibase = build_int_cst (TREE_TYPE (ibase), 0);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (*ar_data->step == NULL_TREE)
|
|
||||||
*ar_data->step = step;
|
|
||||||
else
|
|
||||||
*ar_data->step = fold_build2 (PLUS_EXPR, sizetype,
|
|
||||||
fold_convert (sizetype, *ar_data->step),
|
|
||||||
fold_convert (sizetype, step));
|
|
||||||
if (TREE_CODE (base) == ARRAY_REF)
|
if (TREE_CODE (base) == ARRAY_REF)
|
||||||
{
|
{
|
||||||
stepsize = array_ref_element_size (base);
|
stepsize = array_ref_element_size (base);
|
||||||
if (!cst_and_fits_in_hwi (stepsize))
|
if (!cst_and_fits_in_hwi (stepsize))
|
||||||
return false;
|
return false;
|
||||||
imult = int_cst_value (stepsize);
|
imult = int_cst_value (stepsize);
|
||||||
|
step = fold_build2 (MULT_EXPR, sizetype,
|
||||||
*ar_data->step = fold_build2 (MULT_EXPR, sizetype,
|
fold_convert (sizetype, step),
|
||||||
fold_convert (sizetype, *ar_data->step),
|
fold_convert (sizetype, stepsize));
|
||||||
fold_convert (sizetype, step));
|
|
||||||
idelta *= imult;
|
idelta *= imult;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (*ar_data->step == NULL_TREE)
|
||||||
|
*ar_data->step = step;
|
||||||
|
else
|
||||||
|
*ar_data->step = fold_build2 (PLUS_EXPR, sizetype,
|
||||||
|
fold_convert (sizetype, *ar_data->step),
|
||||||
|
fold_convert (sizetype, step));
|
||||||
*ar_data->delta += idelta;
|
*ar_data->delta += idelta;
|
||||||
*index = ibase;
|
*index = ibase;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue