Hi Marc!
On 2023-11-06T11:59:18+0100, Marc Poulhiès <poulh...@adacore.com> wrote:
> These 3 tests fails parsing the 'vect' dump when not using -mavx. Make
> the dependency explicit.
But that means that the tests are now enabled *only* for
effective-target 'avx_runtime', so, for example, on GCN I see:
-PASS: gcc.dg/vect/vect-ifcvt-18.c (test for excess errors)
-PASS: gcc.dg/vect/vect-ifcvt-18.c execution test
+UNSUPPORTED: gcc.dg/vect/vect-ifcvt-18.c
-PASS: gcc.dg/vect/vect-simd-clone-16f.c (test for excess errors)
-PASS: gcc.dg/vect/vect-simd-clone-16f.c execution test
-PASS: gcc.dg/vect/vect-simd-clone-16f.c scan-tree-dump-times vect
"[\\n\\r] [^\\n]* = foo\\.simdclone" 2
+UNSUPPORTED: gcc.dg/vect/vect-simd-clone-16f.c
-PASS: gcc.dg/vect/vect-simd-clone-18f.c (test for excess errors)
-PASS: gcc.dg/vect/vect-simd-clone-18f.c execution test
-PASS: gcc.dg/vect/vect-simd-clone-18f.c scan-tree-dump-times vect
"[\\n\\r] [^\\n]* = foo\\.simdclone" 2
+UNSUPPORTED: gcc.dg/vect/vect-simd-clone-18f.c
..., which was not the intention, I suppose?
Grüße
Thomas
> gcc/testsuite/ChangeLog:
>
> * gcc.dg/vect/vect-ifcvt-18.c: Add dep on avx_runtime.
> * gcc.dg/vect/vect-simd-clone-16f.c: Likewise.
> * gcc.dg/vect/vect-simd-clone-18f.c: Likewise.
> ---
> Tested on x86_64-linux and x86_64-elf.
>
> Ok for master?
>
> gcc/testsuite/gcc.dg/vect/vect-ifcvt-18.c | 3 ++-
> gcc/testsuite/gcc.dg/vect/vect-simd-clone-16f.c | 4 ++--
> gcc/testsuite/gcc.dg/vect/vect-simd-clone-18f.c | 4 ++--
> 3 files changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/gcc/testsuite/gcc.dg/vect/vect-ifcvt-18.c
> b/gcc/testsuite/gcc.dg/vect/vect-ifcvt-18.c
> index c1d3c27d819..607194496e9 100644
> --- a/gcc/testsuite/gcc.dg/vect/vect-ifcvt-18.c
> +++ b/gcc/testsuite/gcc.dg/vect/vect-ifcvt-18.c
> @@ -1,6 +1,7 @@
> /* { dg-require-effective-target vect_condition } */
> /* { dg-require-effective-target vect_float } */
> -/* { dg-additional-options "-Ofast -mavx" { target avx_runtime } } */
> +/* { dg-require-effective-target avx_runtime } */
> +/* { dg-additional-options "-Ofast -mavx" } */
>
>
> int A0[4] = {36,39,42,45};
> diff --git a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-16f.c
> b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-16f.c
> index 7cd29e894d0..c6615dc626d 100644
> --- a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-16f.c
> +++ b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-16f.c
> @@ -1,6 +1,6 @@
> /* { dg-require-effective-target vect_simd_clones } */
> -/* { dg-additional-options "-fopenmp-simd --param vect-epilogues-nomask=0" }
> */
> -/* { dg-additional-options "-mavx" { target avx_runtime } } */
> +/* { dg-additional-options "-fopenmp-simd --param vect-epilogues-nomask=0
> -mavx" } */
> +/* { dg-require-effective-target avx_runtime } */
> /* { dg-additional-options "-mno-avx512f" { target { { i?86*-*-* x86_64-*-*
> } && { ! lp64 } } } } */
>
> #define TYPE __INT64_TYPE__
> diff --git a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-18f.c
> b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-18f.c
> index 4dd51381d73..787b918d0c4 100644
> --- a/gcc/testsuite/gcc.dg/vect/vect-simd-clone-18f.c
> +++ b/gcc/testsuite/gcc.dg/vect/vect-simd-clone-18f.c
> @@ -1,6 +1,6 @@
> /* { dg-require-effective-target vect_simd_clones } */
> -/* { dg-additional-options "-fopenmp-simd --param vect-epilogues-nomask=0" }
> */
> -/* { dg-additional-options "-mavx" { target avx_runtime } } */
> +/* { dg-additional-options "-fopenmp-simd --param vect-epilogues-nomask=0
> -mavx" } */
> +/* { dg-require-effective-target avx_runtime } */
> /* { dg-additional-options "-mno-avx512f" { target { { i?86*-*-* x86_64-*-*
> } && { ! lp64 } } } } */
>
> #define TYPE __INT64_TYPE__
> --
> 2.42.0
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634
München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas
Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht
München, HRB 106955
