On Fri, Dec 22, 2017 at 11:35 AM, Tsimbalist, Igor V <igor.v.tsimbal...@intel.com> wrote: > This is a follow up patch for pr83488 to fix an error in setting > > OPTION_MASK_ISA_AVX512VNNI_SET and OPTION_MASK_ISA_AVX512F_SET bits. > > There were both set in ix86_isa_flags2 while being defined in > > different ISA sets. Additionally move OPTION_MASK_ISA_AVX512VNNI_SET > > to ix86_isa_flags as it can be used with OPTION_MASK_ISA_AVX512VL_SET. > > > > gcc/ > > * common/config/i386/i386-common.c > (OPTION_MASK_ISA_AVX512VNNI_SET): > > Or in OPTION_MASK_ISA_AVX512F_SET. > > (OPTION_MASK_ISA_AVX512F_UNSET): Or in > > OPTION_MASK_ISA_AVX512VNNI_UNSET. > > (ix86_handle_option): Adjust for > > OPTION_MASK_ISA_AVX512VNNI_*SET being in ix86_isa_flags. > > * config/i386/i386-builtin.def: Move VNNI builtins from > ARGS2 > > section to ARGS. > > * config/i386/i386-c.c: Check for OPTION_MASK_ISA_AVX512VNNI > in > > isa_flag instead of isa_flag2. > > * config/i386/i386.c (ix86_target_string): Move -mavx512vnni > from > > isa_opts2 to isa_opts. > > * config/i386/i386.opt (mavx512vnni): Move from > ix86_isa_flags2 > > to ix86_isa_flags. > > > > Ok for trunk?
OK. Thanks, Uros.
