diff options
| author | Christian Borntraeger <borntraeger@de.ibm.com> | 2010-05-25 11:18:07 +0000 |
|---|---|---|
| committer | Andreas Krebbel <krebbel@gcc.gnu.org> | 2010-05-25 11:18:07 +0000 |
| commit | 8fde8b401da8b351f0ad3ed914e2b9cc9f3e9899 (patch) | |
| tree | 40f47cc3c195e5fad0387d27e2b9762a3e6a78b0 /gcc/tree-ssa-loop-prefetch.c | |
| parent | be6d3f0e89340dc7f7c2281aa747aa6560c05715 (diff) | |
| download | gcc-8fde8b401da8b351f0ad3ed914e2b9cc9f3e9899.zip gcc-8fde8b401da8b351f0ad3ed914e2b9cc9f3e9899.tar.gz gcc-8fde8b401da8b351f0ad3ed914e2b9cc9f3e9899.tar.bz2 | |
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
Diffstat (limited to 'gcc/tree-ssa-loop-prefetch.c')
| -rw-r--r-- | gcc/tree-ssa-loop-prefetch.c | 19 |
1 files changed, 9 insertions, 10 deletions
diff --git a/gcc/tree-ssa-loop-prefetch.c b/gcc/tree-ssa-loop-prefetch.c index 633dd33..d63ede1 100644 --- a/gcc/tree-ssa-loop-prefetch.c +++ b/gcc/tree-ssa-loop-prefetch.c @@ -423,25 +423,24 @@ idx_analyze_ref (tree base, tree *index, void *data) 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) { stepsize = array_ref_element_size (base); if (!cst_and_fits_in_hwi (stepsize)) return false; imult = int_cst_value (stepsize); - - *ar_data->step = fold_build2 (MULT_EXPR, sizetype, - fold_convert (sizetype, *ar_data->step), - fold_convert (sizetype, step)); + step = fold_build2 (MULT_EXPR, sizetype, + fold_convert (sizetype, step), + fold_convert (sizetype, stepsize)); 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; *index = ibase; |