This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org, Richard Biener <richard dot guenther at gmail dot com>
- Date: Thu, 5 Jan 2017 22:09:02 +0100
- Subject: Re: [PATCH] Fix ICE with -fno-sso-struct=none (PR driver/78957)
- Authentication-results: sourceware.org; auth=none
- References: <20170102192726.GH21933@tucnak> <CAFiYyc3jFQXnsGSV5iuJuEc6cm0Sa=3=amDcirP46brRpRvXjw@mail.gmail.com> <20170104114031.GI21933@tucnak> <247277927.gUkCtdRrLC@polaris>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
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 little-endian, but also native, so one can
> > cancel earlier -fsso-struct= like
> > gcc ... -fsso-struct=little-endian ... -fsso-struct=native ...
> > and have it act as if neither of those options appeared.
>
> That makes sense and is immediate since the logic is already based on the
> native endianness of the target.
>
> Tested on x86-64_suse-linux, OK for the mainline?
>
>
> 2017-01-05 Eric Botcazou <ebotcazou@adacore.com>
>
> * c-family/c.opt (fsso-struct): Add 'native' value.
c-family/ has its own ChangeLog, so just * c.opt (...): ...
> * doc/invoke.texi (C Dialect Options): Document it.
>
>
> 2017-01-05 Eric Botcazou <ebotcazou@adacore.com>
>
> * gcc.dg/sso-10.c: New test.
Ok, thanks.
Jakub