https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83488
--- Comment #20 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
(In reply to igor.v.tsimbalist from comment #19)
> Created attachment 42947 [details]
> 512VNNI patch
What is the reason for moving it over? At least right now you don't have any |
OPTION_MASK_ISA_AVX512VL or | OPTION_MASK_ISA_64BIT with it, and because there
is a high chance we'll get some new isas that will need that, better keep the
bits in isa_flags free for those.
I meant just:
case OPT_mavx512vnni:
if (value)
{
opts->x_ix86_isa_flags2 |= OPTION_MASK_ISA_AVX512VNNI_SET;
opts->x_ix86_isa_flags2_explicit |= OPTION_MASK_ISA_AVX512VNNI_SET;
- opts->x_ix86_isa_flags2 |= OPTION_MASK_ISA_AVX512F_SET;
- opts->x_ix86_isa_flags2_explicit |= OPTION_MASK_ISA_AVX512F_SET;
+ opts->x_ix86_isa_flags |= OPTION_MASK_ISA_AVX512F_SET;
+ opts->x_ix86_isa_flags_explicit |= OPTION_MASK_ISA_AVX512F_SET;
}
you had in your earlier patch.
