‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Thursday, March 5, 2020 6:59 PM, Segher Boessenkool
<seg...@kernel.crashing.org> wrote:
> On Thu, Mar 05, 2020 at 05:04:16PM +0000, GT wrote:
>
> > 2. Multiple other testcases in testsuite/gcc.dg/vect/ have this line at
> > the top:
> > /* { dg-additional-options "-mavx" { target avx_runtime } } */
> > An example is vect-simd-16.c
> >
> >
> > 2.1 Should not these testcases be in directory testsuite/gcc.target/i386/ ?
> > 2.2 To run vect-simd-16.c on PPC64, is it enough to put a copy of the file
> > in testsuite/gcc.target/powerpc/ ? (After modifying target appropriately.)
>
>
> It certainly would be nice if generic tests did not often have target-
> specific stuff in them. In this case, it could be hidden in vect.exp,
> perhaps?
I got all the tests from RUNTESTFLAGS="vect.exp=*simd*" to PASS. The ICE errors
I was
getting came from simdlen clauses of #pragma omp declare simd directives. The
vectorization code currently assumes that simdlen(n) is always valid and does
not
check whether the hardware supports n vectors. The tests assume a 256-bit wide
vector unit. VSX width is 128-bit and hence the failures.
We need to selectively change the simdlen clauses depending on the target on
which
the tests are being run. The typical test which needs this feature is
gcc/testsuite/gcc.dg/vect/vect-simd-clone-1.c. Should I conditionally select
#pragma omp directives using #ifdefs in the source files? Or is there a dejagnu
feature which is preferred?
Bert.
