On 11/29/18 11:26 AM, Segher Boessenkool wrote: > On Wed, Nov 28, 2018 at 03:27:19PM -0600, Peter Bergner wrote: >> PR87496 shows a bug where we ICE if we attempt to use -mabi=ieeelongdouble >> and -mno-popcntd. The IEEE128 support requires full ISA 2.06 (aka POWER7) >> support, so we really should throw an error when using those options >> together. Ditto for -mabi=ieeelongdouble and -mno-vsx. The patch below >> does that. >> >> Ok for mainline once bootstrap and regtesting are complete and clean? > > Okay. Eventually we shouldn't allow selecting popcntd independently from > -mcpu=, but that day isn't here yet. So, okay for trunk, and backports > if wanted. Thanks!
Ok, committed to mainline. It looks like GCC8 needs the same patch. I'll have to look closer at GCC7 on whether it needs it too, since the code seems to be a little different. Thanks Peter