[Bug target/87198] ICE in extract_insn, at recog.c:2304

ubizjak at gmail dot com gcc-bugzilla@gcc.gnu.org
Mon Sep 3 11:18:00 GMT 2018


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

--- Comment #4 from Uroš Bizjak <ubizjak at gmail dot com> ---
(In reply to Jakub Jelinek from comment #3)
> (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.

Please see recent discussion in PR87171#c4.


More information about the Gcc-bugs mailing list