diff options
| author | Ira Rosen <irar@gcc.gnu.org> | 2007-02-22 12:30:12 +0000 |
|---|---|---|
| committer | Ira Rosen <irar@gcc.gnu.org> | 2007-02-22 12:30:12 +0000 |
| commit | 8fca6de5641592fdaa5e03a415b1727f5e1ddcaf (patch) | |
| tree | d9253aab39b73d5f69ae734d600a9eb4aeeb1242 /gcc | |
| parent | e8020d1ff6d2b529d4a3f0467d883dc52f0e0758 (diff) | |
| download | gcc-8fca6de5641592fdaa5e03a415b1727f5e1ddcaf.zip gcc-8fca6de5641592fdaa5e03a415b1727f5e1ddcaf.tar.gz gcc-8fca6de5641592fdaa5e03a415b1727f5e1ddcaf.tar.bz2 | |
re PR tree-optimization/30843 (ice for legal code with -ftree-vectorize -O2)
PR tree-optimization/30843
* tree-vect-transform.c (vect_transform_loop): Remove strided scalar
stores only after all the group is vectorized.
From-SVN: r122225
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/vect/pr30843.c | 25 | ||||
| -rw-r--r-- | gcc/tree-vect-transform.c | 14 |
4 files changed, 40 insertions, 11 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 829872f..29b23b7 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2007-02-22 Ira Rosen <irar@il.ibm.com> + + PR tree-optimization/30843 + * tree-vect-transform.c (vect_transform_loop): Remove strided scalar + stores only after all the group is vectorized. + 2007-02-22 Dorit Nuzman <dorit@il.ibm.com> PR tree-optimization/30858 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 8556c43..bc8535c 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,4 +1,10 @@ 2007-02-22 Dorit Nuzman <dorit@il.ibm.com> + Ira Rosen <irar@il.ibm.com> + + PR tree-optimization/30843 + * gcc.dg/vect/pr30843.c: New test. + +2007-02-22 Dorit Nuzman <dorit@il.ibm.com> PR tree-optimization/30858 * gcc.dg/vect/pr30858.c: New test. diff --git a/gcc/testsuite/gcc.dg/vect/pr30843.c b/gcc/testsuite/gcc.dg/vect/pr30843.c new file mode 100644 index 0000000..616b618 --- /dev/null +++ b/gcc/testsuite/gcc.dg/vect/pr30843.c @@ -0,0 +1,25 @@ +/* { dg-do compile } */ +/* { dg-require-effective-target vect_long } */ + +#include <stdarg.h> +#include "tree-vect.h" + +#define N 16 + +void dacP98FillRGBMap (unsigned char *pBuffer) +{ + unsigned long dw, dw1; + unsigned long *pdw = (unsigned long *)(pBuffer); + + for( dw = 256, dw1 = 0; dw; dw--, dw1 += 0x01010101) + { + *pdw++ = dw1; + *pdw++ = dw1; + *pdw++ = dw1; + *pdw++ = dw1; + } +} + +/* { dg-final { scan-tree-dump-times "vectorized 1 loops" 1 "vect" { target { vect_interleave } } } } */ +/* { dg-final { cleanup-tree-dump "vect" } } */ + diff --git a/gcc/tree-vect-transform.c b/gcc/tree-vect-transform.c index 109705d..09f1993 100644 --- a/gcc/tree-vect-transform.c +++ b/gcc/tree-vect-transform.c @@ -5059,7 +5059,7 @@ vect_transform_loop (loop_vec_info loop_vinfo) struct loop *loop = LOOP_VINFO_LOOP (loop_vinfo); basic_block *bbs = LOOP_VINFO_BBS (loop_vinfo); int nbbs = loop->num_nodes; - block_stmt_iterator si; + block_stmt_iterator si, next_si; int i; tree ratio = NULL; int vectorization_factor = LOOP_VINFO_VECT_FACTOR (loop_vinfo); @@ -5212,12 +5212,14 @@ vect_transform_loop (loop_vec_info loop_vinfo) while (next) { + next_si = bsi_for_stmt (next); next_stmt_info = vinfo_for_stmt (next); /* Free the attached stmt_vec_info and remove the stmt. */ ann = stmt_ann (next); tmp = DR_GROUP_NEXT_DR (next_stmt_info); free (next_stmt_info); set_stmt_info (ann, NULL); + bsi_remove (&next_si, true); next = tmp; } bsi_remove (&si, true); @@ -5233,16 +5235,6 @@ vect_transform_loop (loop_vec_info loop_vinfo) continue; } } - else - { - if (strided_store) - { - /* This is case of skipped interleaved store. We don't free - its stmt_vec_info. */ - bsi_remove (&si, true); - continue; - } - } bsi_next (&si); } /* stmts in BB */ } /* BBs in loop */ |