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

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
(In reply to Uroš Bizjak from comment #2)
> I think you also need:
> 
> diff --git a/gcc/common/config/i386/i386-common.c
> b/gcc/common/config/i386/i386-common.c
> index 70b3c3f2fc31..edb55e542451 100644
> --- a/gcc/common/config/i386/i386-common.c
> +++ b/gcc/common/config/i386/i386-common.c
> @@ -181,7 +181,7 @@ along with GCC; see the file COPYING3.  If not see
>  #define OPTION_MASK_ISA_AVX_UNSET \
>    (OPTION_MASK_ISA_AVX | OPTION_MASK_ISA_FMA_UNSET \
>     | OPTION_MASK_ISA_FMA4_UNSET | OPTION_MASK_ISA_F16C_UNSET \
> -   | OPTION_MASK_ISA_AVX2_UNSET | OPTION_MASK_ISA_XSAVE_UNSET)
> +   | OPTION_MASK_ISA_AVX2_UNSET)
>  #define OPTION_MASK_ISA_FMA_UNSET OPTION_MASK_ISA_FMA
>  #define OPTION_MASK_ISA_FXSR_UNSET OPTION_MASK_ISA_FXSR
>  #define OPTION_MASK_ISA_XSAVE_UNSET \
> 
> so -mno-avx won't turn off -mxsave.

Why shouldn't it turn off -mxsave?  -mavx sets -mxsave too.
I mean, otherwise if one does say on -m32 do -mavx -mno-avx, then won't that
turn on -mxsave and not turn it off again?  Or with target pragmas.
I believe we generally want consistency, if option ABC turns automatically
option DEF, then turning off option DEF explicitly disables option ABC.

Reply via email to