https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744

--- Comment #9 from Hongtao.liu <crazylht at gmail dot com> ---
(In reply to Hongtao.liu from comment #7)
> (In reply to Uroš Bizjak from comment #5)
> > (In reply to Hongtao.liu from comment #2)
> > 
> > > Need to add define_insn for movp2qi/movp2hi?
> > 
> > Yes, this is needed to cover some corner cases. Please see attachment 49114 
> > [details]
> > [details].
> > 
> > However, the patch assumes that avx512vp2intersect implies mavx512dq,
> 
> Let me check this part.

Confirmed, avx512vp2intersect implies avx512dq.

Should bellow in the same patch with attachment 49114, or a separate patch?

---
diff --git a/gcc/common/config/i386/i386-common.c
b/gcc/common/config/i386/i386-common.c
index bb14305ad7b..5305145a8c9 100644
--- a/gcc/common/config/i386/i386-common.c
+++ b/gcc/common/config/i386/i386-common.c
@@ -906,8 +906,8 @@ ix86_handle_option (struct gcc_options *opts,
          opts->x_ix86_isa_flags2 |= OPTION_MASK_ISA2_AVX512VP2INTERSECT_SET;
          opts->x_ix86_isa_flags2_explicit |=
            OPTION_MASK_ISA2_AVX512VP2INTERSECT_SET;
-         opts->x_ix86_isa_flags |= OPTION_MASK_ISA_AVX512F_SET;
-         opts->x_ix86_isa_flags_explicit |= OPTION_MASK_ISA_AVX512F_SET;
+         opts->x_ix86_isa_flags |= OPTION_MASK_ISA_AVX512DQ_SET;
+         opts->x_ix86_isa_flags_explicit |= OPTION_MASK_ISA_AVX512DQ_SET;
        }
       else
        {
---

Reply via email to