diff options
| author | Richard Guenther <rguenther@suse.de> | 2010-11-09 16:54:05 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2010-11-09 16:54:05 +0000 |
| commit | a3357f7dffc52969c120c2ecb9f1c013833ccb9b (patch) | |
| tree | 753d495c70dbddff79d759e0baeab03d162cb294 /gcc | |
| parent | 0644953e64a91b24e2504900aa70e394c28ffc51 (diff) | |
| download | gcc-a3357f7dffc52969c120c2ecb9f1c013833ccb9b.zip gcc-a3357f7dffc52969c120c2ecb9f1c013833ccb9b.tar.gz gcc-a3357f7dffc52969c120c2ecb9f1c013833ccb9b.tar.bz2 | |
re PR tree-optimization/46355 (ICE: SIGSEGV in create_preheader (cfgloopmanip.c:1336) with -O -fstrict-overflow -ftree-loop-distribution)
2010-11-09 Richard Guenther <rguenther@suse.de>
PR tree-optimization/46355
* tree-loop-distribution.c (tree_loop_distribution): Do not
distribute loops without a single exit.
* gcc.dg/pr46355.c: New testcase.
From-SVN: r166498
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/pr46355.c | 14 | ||||
| -rw-r--r-- | gcc/tree-loop-distribution.c | 7 |
4 files changed, 31 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index b41bb87..a828481 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2010-11-09 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/46355 + * tree-loop-distribution.c (tree_loop_distribution): Do not + distribute loops without a single exit. + 2010-11-09 Andreas Krebbel <Andreas.Krebbel@de.ibm.com> PR rtl-optimization/46237 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 70493d9..1473884 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,10 @@ 2010-11-09 Richard Guenther <rguenther@suse.de> + PR tree-optimization/46355 + * gcc.dg/pr46355.c: New testcase. + +2010-11-09 Richard Guenther <rguenther@suse.de> + PR tree-optimization/46177 * gcc.dg/pr46177-2.c: New testcase. diff --git a/gcc/testsuite/gcc.dg/pr46355.c b/gcc/testsuite/gcc.dg/pr46355.c new file mode 100644 index 0000000..e644ec5 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr46355.c @@ -0,0 +1,14 @@ +/* { dg-do compile } */ +/* { dg-options "-O -ftree-loop-distribution -fstrict-overflow" } */ + +void +foo (int *dest, int i, int u, int v) +{ + int j = i; + while (i) + { + dest[j--] = v; + dest[j--] = u; + } +} + diff --git a/gcc/tree-loop-distribution.c b/gcc/tree-loop-distribution.c index 59585cd..007c4f3 100644 --- a/gcc/tree-loop-distribution.c +++ b/gcc/tree-loop-distribution.c @@ -1145,7 +1145,12 @@ tree_loop_distribution (void) FOR_EACH_LOOP (li, loop, 0) { - VEC (gimple, heap) *work_list = VEC_alloc (gimple, heap, 3); + VEC (gimple, heap) *work_list = NULL; + + /* If the loop doesn't have a single exit we will fail anyway, + so do that early. */ + if (!single_exit (loop)) + continue; /* If both flag_tree_loop_distribute_patterns and flag_tree_loop_distribution are set, then only |