This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [C++ PATCH] Fix option handling when -std=gnu++14 is not used (PR 69865)
- From: Bernd Edlinger <bernd dot edlinger at hotmail dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Jason Merrill <jason at redhat dot com>, Jonathan Wakely <jwakely at redhat dot com>
- Date: Fri, 19 Feb 2016 12:09:58 +0000
- Subject: Re: [C++ PATCH] Fix option handling when -std=gnu++14 is not used (PR 69865)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: gcc.gnu.org; dkim=none (message not signed) header.d=none;gcc.gnu.org; dmarc=none action=none header.from=hotmail.de;
- References: <HE1PR07MB090548A2180950184AF65A60E4A00 at HE1PR07MB0905 dot eurprd07 dot prod dot outlook dot com> <20160219105612 dot GA3017 at tucnak dot redhat dot com> <HE1PR07MB09054F1EE94A5C0E57E41AAEE4A00 at HE1PR07MB0905 dot eurprd07 dot prod dot outlook dot com> <20160219113126 dot GB3017 at tucnak dot redhat dot com> <HE1PR07MB09050AD7717125F4428FF773E4A00 at HE1PR07MB0905 dot eurprd07 dot prod dot outlook dot com>,<20160219115945 dot GC3017 at tucnak dot redhat dot com>
- Spamdiagnosticmetadata: NSPM
- Spamdiagnosticoutput: 1:23
On 19.02.2016 12:59, Jakub Jelinek wrote:
>
> Of course not, and that would be the wrong thing to do.
> The definition spot of libc_name_p comes from gperf itself, the prototype
> from cfns.gperf, which we can of course adjust.
>
Yes, now I understand. Thanks.
>
> IMNSHO we should just keep it consistent with what g++ e.g. 5.x did.
>Thus,
> $ g++ -E -dD -xc++ /dev/null -O2 -std=c++98 2>&1 | grep _INLINE_
> #define __GNUC_GNU_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 -std=c++11 2>&1 | grep _INLINE_
> #define __GNUC_STDC_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 -std=c++14 2>&1 | grep _INLINE_
> #define __GNUC_STDC_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 -std=gnu++98 2>&1 | grep _INLINE_
> #define __GNUC_GNU_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 -std=gnu++11 2>&1 | grep _INLINE_
> #define __GNUC_STDC_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 -std=gnu++14 2>&1 | grep _INLINE_
> #define __GNUC_STDC_INLINE__ 1
> $ g++ -E -dD -xc++ /dev/null -O2 2>&1 | grep _INLINE_
> #define __GNUC_GNU_INLINE__ 1
> We want to define what we did with the explicit -std= options, and just
> change the output in the default case (last invocation), to
> #define __GNUC_STDC_INLINE__ 1
> because the default is different.
>
> Jakub
OK. I can do that.
I will send a new patch in the evening.
Thanks
Bernd.