Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)

2017-01-05 Thread Jakub Jelinek
On Thu, Jan 05, 2017 at 08:53:49PM +0100, Eric Botcazou wrote: > > In this case there is no -fsso-struct option (what would it mean), so > > -fno-sso-struct isn't there either (again, what would that mean). > > The only thing that would make sense IMHO would be to allow > > not just big-endian and

Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)

2017-01-05 Thread Eric Botcazou
> In this case there is no -fsso-struct option (what would it mean), so > -fno-sso-struct isn't there either (again, what would that mean). > The only thing that would make sense IMHO would be to allow > not just big-endian and little-endian, but also native, so one can > cancel earlier -fsso-struc

Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)

2017-01-04 Thread Richard Biener
On Wed, Jan 4, 2017 at 12:40 PM, Jakub Jelinek wrote: > On Wed, Jan 04, 2017 at 12:27:09PM +0100, Richard Biener wrote: >> On Mon, Jan 2, 2017 at 8:27 PM, Jakub Jelinek wrote: >> > Enum options should not allow negative form, otherwise the option handling >> > ICEs on it. -fsso-struct= allows on

Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)

2017-01-04 Thread Jakub Jelinek
On Wed, Jan 04, 2017 at 12:27:09PM +0100, Richard Biener wrote: > On Mon, Jan 2, 2017 at 8:27 PM, Jakub Jelinek wrote: > > Enum options should not allow negative form, otherwise the option handling > > ICEs on it. -fsso-struct= allows only big-endian or little-endian, > > -fno-sso-struct=big-endi

Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)

2017-01-04 Thread Richard Biener
On Mon, Jan 2, 2017 at 8:27 PM, Jakub Jelinek wrote: > Hi! > > Enum options should not allow negative form, otherwise the option handling > ICEs on it. -fsso-struct= allows only big-endian or little-endian, > -fno-sso-struct=big-endian or -fno-sso-struct=whatever makes no sense. > > Bootstrapped/