diff options
| author | Richard Guenther <rguenther@suse.de> | 2010-01-09 12:04:17 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2010-01-09 12:04:17 +0000 |
| commit | 73c865fab7acc35517e28e09dc417db6a5697f12 (patch) | |
| tree | ee8a9e64e051a1d0c3d8486cfd0df2a03b079041 /gcc | |
| parent | 0462fd5e01941647bfb8b3706443fd9de8b585c2 (diff) | |
| download | gcc-73c865fab7acc35517e28e09dc417db6a5697f12.zip gcc-73c865fab7acc35517e28e09dc417db6a5697f12.tar.gz gcc-73c865fab7acc35517e28e09dc417db6a5697f12.tar.bz2 | |
re PR middle-end/42512 (integer wrong code bug with loop)
2010-01-09 Richard Guenther <rguenther@suse.de>
PR middle-end/42512
* tree-scalar-evolution.c (interpret_loop_phi): Make sure
the evolution is compatible with the initial condition.
* gcc.c-torture/execute/pr42512.c: New testcase.
From-SVN: r155757
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.c-torture/execute/pr42512.c | 13 | ||||
| -rw-r--r-- | gcc/tree-scalar-evolution.c | 17 |
4 files changed, 41 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 708dbf6..99f5989 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-01-09 Richard Guenther <rguenther@suse.de> + + PR middle-end/42512 + * tree-scalar-evolution.c (interpret_loop_phi): Make sure + the evolution is compatible with the initial condition. + 2010-01-09 Jakub Jelinek <jakub@redhat.com> * gcc.c (process_command): Update copyright notice dates. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index fc9ee83..87d7a18 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-01-09 Richard Guenther <rguenther@suse.de> + + PR middle-end/42512 + * gcc.c-torture/execute/pr42512.c: New testcase. + 2010-01-09 Tobias Burnus <burnus@net-b.de> PR fortran/41298 diff --git a/gcc/testsuite/gcc.c-torture/execute/pr42512.c b/gcc/testsuite/gcc.c-torture/execute/pr42512.c new file mode 100644 index 0000000..f4e5ced --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/execute/pr42512.c @@ -0,0 +1,13 @@ +extern void abort (void); + +short g_3; + +int main (void) +{ + int l_2; + for (l_2 = -1; l_2 != 0; l_2 = (unsigned char)(l_2 - 1)) + g_3 |= l_2; + if (g_3 != -1) + abort (); + return 0; +} diff --git a/gcc/tree-scalar-evolution.c b/gcc/tree-scalar-evolution.c index 087ba79..3ebc54e 100644 --- a/gcc/tree-scalar-evolution.c +++ b/gcc/tree-scalar-evolution.c @@ -1642,6 +1642,23 @@ interpret_loop_phi (struct loop *loop, gimple loop_phi_node) init_cond = analyze_initial_condition (loop_phi_node); res = analyze_evolution_in_loop (loop_phi_node, init_cond); + /* Verify we maintained the correct initial condition throughout + possible conversions in the SSA chain. */ + if (res != chrec_dont_know) + { + tree new_init = res; + if (CONVERT_EXPR_P (res) + && TREE_CODE (TREE_OPERAND (res, 0)) == POLYNOMIAL_CHREC) + new_init = fold_convert (TREE_TYPE (res), + CHREC_LEFT (TREE_OPERAND (res, 0))); + else if (TREE_CODE (res) == POLYNOMIAL_CHREC) + new_init = CHREC_LEFT (res); + STRIP_USELESS_TYPE_CONVERSION (new_init); + gcc_assert (TREE_CODE (new_init) != POLYNOMIAL_CHREC); + if (!operand_equal_p (init_cond, new_init, 0)) + return chrec_dont_know; + } + return res; } |