On Wed, Apr 24, 2024 at 1:46 PM Haochen Jiang <haochen.ji...@intel.com> wrote:
>
> Hi all,
>
> When we are using -mavx10.1-256 in command line and avx10.1-256 in
> target attribute together, zmm should never be generated. But current
> GCC will generate zmm since it wrongly enables EVEX512 for non-explicitly
> set AVX512. This patch will fix that issue.
>
> Regtested on x86_64-pc-linux-gnu. Ok for trunk?
Ok.
>
> gcc/ChangeLog:
>
> * config/i386/i386-options.cc (ix86_valid_target_attribute_tree):
> Check whether AVX512F is explicitly enabled.
>
> gcc/testsuite/ChangeLog:
>
> * gcc.target/i386/avx10_1-24.c: New test.
> ---
> gcc/config/i386/i386-options.cc | 1 +
> gcc/testsuite/gcc.target/i386/avx10_1-24.c | 7 +++++++
> 2 files changed, 8 insertions(+)
> create mode 100644 gcc/testsuite/gcc.target/i386/avx10_1-24.c
>
> diff --git a/gcc/config/i386/i386-options.cc b/gcc/config/i386/i386-options.cc
> index 68a2e1c6910..ac48b5c61c4 100644
> --- a/gcc/config/i386/i386-options.cc
> +++ b/gcc/config/i386/i386-options.cc
> @@ -1431,6 +1431,7 @@ ix86_valid_target_attribute_tree (tree fndecl, tree
> args,
> scenario. */
> if ((def->x_ix86_isa_flags2 & OPTION_MASK_ISA2_AVX10_1_256)
> && (opts->x_ix86_isa_flags & OPTION_MASK_ISA_AVX512F)
> + && (opts->x_ix86_isa_flags_explicit & OPTION_MASK_ISA_AVX512F)
> && !(def->x_ix86_isa_flags2_explicit & OPTION_MASK_ISA2_EVEX512)
> && !(opts->x_ix86_isa_flags2_explicit & OPTION_MASK_ISA2_EVEX512))
> opts->x_ix86_isa_flags2 |= OPTION_MASK_ISA2_EVEX512;
> diff --git a/gcc/testsuite/gcc.target/i386/avx10_1-24.c
> b/gcc/testsuite/gcc.target/i386/avx10_1-24.c
> new file mode 100644
> index 00000000000..2e93f041760
> --- /dev/null
> +++ b/gcc/testsuite/gcc.target/i386/avx10_1-24.c
> @@ -0,0 +1,7 @@
> +/* { dg-do compile } */
> +/* { dg-options "-O2 -march=x86-64 -mavx10.1" } */
> +/* { dg-final { scan-assembler-not "%zmm" } } */
> +
> +typedef float __m512 __attribute__ ((__vector_size__ (64), __may_alias__));
> +
> +void __attribute__((target("avx10.1-256"))) callee256(__m512 *a, __m512 *b)
> { *a = *b; }
> --
> 2.31.1
>
--
BR,
Hongtao
