On Thu, Dec 07, 2023 at 05:28:09PM +0100, Marc Poulhiès wrote:
> The test fails parsing the 'vect' dump when not using -mavx. Make the
> dependency explicit.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/vect/vect-simd-clone-17f.c: Add dep on avx_runtime.
> ---
> Tested on x86_64-linux and x86_64-elf.
>
> Ok for master?
>
> gcc/testsuite/gcc.dg/vect/vect-simd-clone-17f.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-17f.c
> b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-17f.c
> index 177521dc445..eb2b149981f 100644
> --- a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-17f.c
> +++ b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-17f.c
> @@ -1,6 +1,7 @@
> /* { dg-require-effective-target vect_simd_clones } */
> +/* { dg-require-effective-target avx_runtime } */
> /* { dg-additional-options "-fopenmp-simd --param vect-epilogues-nomask=0" }
> */
> -/* { dg-additional-options "-mavx" { target avx_runtime } } */
> +/* { dg-additional-options "-mavx" } */
> /* { dg-additional-options "-mno-avx512f" { target { { i?86*-*-* x86_64-*-*
> } && { ! lp64 } } } } */
>
> #define TYPE __INT64_TYPE__
This looks wrong, then it won't be tested at all on non-x86 targets.
Jakub
