Message ID | 2bd4ccdc-0bb2-62a0-61ca-e224770cc07c@foss.arm.com |
---|---|
State | New |
Headers | show |
On Tue, Jun 13, 2017 at 11:10 AM, Thomas Preudhomme <thomas.preudhomme@foss.arm.com> wrote: > Hi, > > gen-vect-26.c tests the vectorizer but only requires vect_cmdline_needed > effective target. It should also depends on vect_int to make sure a > vector unit is available on the target. This patch fixes that. > > ChangeLog entry is as follows: > > *** gcc/testsuite/ChangeLog *** > > 2017-06-05 Thomas Preud'homme <thomas.preudhomme@arm.com> > > * gcc.dg/tree-ssa/gen-vect-26.c: Also require vect_int effective > target. > > > Testing: Testcase is now skipped when targeting Cortex-M3. > > Is this ok for trunk? No -- the test tests for generic vectorization using integer mode vectors. See how we pass -mno-sse on x86_64. Richard. > Best regards, > > Thomas
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/gen-vect-26.c b/gcc/testsuite/gcc.dg/tree-ssa/gen-vect-26.c index 8e5f1410612b075914000dcdc643b2838ee3dcd9..8edeb0bbfd31b3926382da27bfafa4f331066ba9 100644 --- a/gcc/testsuite/gcc.dg/tree-ssa/gen-vect-26.c +++ b/gcc/testsuite/gcc.dg/tree-ssa/gen-vect-26.c @@ -1,4 +1,4 @@ -/* { dg-do run { target vect_cmdline_needed } } */ +/* { dg-do run { target { vect_cmdline_needed && vect_int } } } */ /* { dg-options "-O2 -ftree-vectorize -fdump-tree-vect-details -fvect-cost-model=dynamic" } */ /* { dg-options "-O2 -ftree-vectorize -fdump-tree-vect-details -fvect-cost-model=dynamic -mno-sse" { target { i?86-*-* x86_64-*-* } } } */