From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 121634 invoked by alias); 19 Feb 2016 12:10:06 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 121581 invoked by uid 89); 19 Feb 2016 12:10:05 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 spammy=evening X-HELO: DUB004-OMC1S32.hotmail.com Received: from dub004-omc1s32.hotmail.com (HELO DUB004-OMC1S32.hotmail.com) (157.55.0.231) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA256 encrypted) ESMTPS; Fri, 19 Feb 2016 12:10:04 +0000 Received: from emea01-db3-obe.outbound.protection.outlook.com ([157.55.0.239]) by DUB004-OMC1S32.hotmail.com over TLS secured channel with Microsoft SMTPSVC(7.5.7601.23008); Fri, 19 Feb 2016 04:10:01 -0800 Received: from HE1PR07MB0905.eurprd07.prod.outlook.com (10.162.26.12) by HE1PR07MB0907.eurprd07.prod.outlook.com (10.162.26.14) with Microsoft SMTP Server (TLS) id 15.1.409.15; Fri, 19 Feb 2016 12:09:59 +0000 Received: from HE1PR07MB0905.eurprd07.prod.outlook.com ([10.162.26.12]) by HE1PR07MB0905.eurprd07.prod.outlook.com ([10.162.26.12]) with mapi id 15.01.0409.017; Fri, 19 Feb 2016 12:09:58 +0000 From: Bernd Edlinger To: Jakub Jelinek CC: "gcc-patches@gcc.gnu.org" , Jason Merrill , Jonathan Wakely Subject: Re: [C++ PATCH] Fix option handling when -std=gnu++14 is not used (PR 69865) Date: Fri, 19 Feb 2016 12:10:00 -0000 Message-ID: References: <20160219105612.GA3017@tucnak.redhat.com> <20160219113126.GB3017@tucnak.redhat.com> ,<20160219115945.GC3017@tucnak.redhat.com> In-Reply-To: <20160219115945.GC3017@tucnak.redhat.com> authentication-results: gcc.gnu.org; dkim=none (message not signed) header.d=none;gcc.gnu.org; dmarc=none action=none header.from=hotmail.de; x-ms-exchange-messagesentrepresentingtype: 1 x-tmn: [RAfabrzniLExmcXG0+5LYR/fiK0uLT7B] x-microsoft-exchange-diagnostics: 1;HE1PR07MB0907;23:WdiegGb8VwhiPB0M9Bxi76wppoIwDFm8H2T5V811OQOALq1lDR9C1Sy6GCdy9ywAiQF7JY4aBOehVj9CxRqtFSG+TLKJEVJmokCMXwp0M/Od0FFIywnwGVA8jicu92OWEbuzaXR1WYYBvMAfbd4UIeo6WEiYGQjRvFBy7kBCE2/bnSbJMQlv+9Uo4IgnkBvwaI+dVVE7MNobRrn/7ypFxw==;5:VtAapnU7gC5LC2j4KyGeIQ4xYeAvJYyCfv7flq5kQiNV82KxzXWlatLsRQmYD/ViVP/0ZLl/bV4guFQSzO4lwzs4oJAz7+WIQPaylskQDugaVaky7qQwG5oSymurReih6GpJUYJYHtE0Y+5jeWWw6g==;24:fw0ltosAW5cDAWY9r8jdCZBx5VKe98rFDZLaJvr/CntfJxQ+r7JmYUgkJ4JWuN3I89o+ntk6nkf2q/CCCAt8ZcnC3wJw2VZmAuCroTMGCxw= x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:HE1PR07MB0907; x-ms-office365-filtering-correlation-id: df0863ea-fc41-4c43-ff58-08d339259632 x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(432015012)(82015046);SRVR:HE1PR07MB0907;BCL:0;PCL:0;RULEID:;SRVR:HE1PR07MB0907; x-forefront-prvs: 08572BD77F x-forefront-antispam-report: SFV:NSPM;SFS:(7070004)(98900003);DIR:OUT;SFP:1901;SCL:1;SRVR:HE1PR07MB0907;H:HE1PR07MB0905.eurprd07.prod.outlook.com;FPR:;SPF:None;MLV:ovrnspm;LANG:en; spamdiagnosticoutput: 1:23 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: sct-15-1-409-10-msonline-outlook-d6129.templateTenant X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Feb 2016 12:09:58.3838 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR07MB0907 X-SW-Source: 2016-02/txt/msg01319.txt.bz2 On 19.02.2016 12:59, Jakub Jelinek wrote: >=20 > 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. =20 > > IMNSHO we should just keep it consistent with what g++ e.g. 5.x did. >Thus, > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dc++98 2>&1 | grep _INLINE_ > #define __GNUC_GNU_INLINE__ 1 > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dc++11 2>&1 | grep _INLINE_ > #define __GNUC_STDC_INLINE__ 1 > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dc++14 2>&1 | grep _INLINE_ > #define __GNUC_STDC_INLINE__ 1 > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dgnu++98 2>&1 | grep _INLINE_ > #define __GNUC_GNU_INLINE__ 1 > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dgnu++11 2>&1 | grep _INLINE_ > #define __GNUC_STDC_INLINE__ 1 > $ g++ -E -dD -xc++ /dev/null -O2 -std=3Dgnu++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=3D options, and just > change the output in the default case (last invocation), to > #define __GNUC_STDC_INLINE__ 1 > because the default is different. >=20 > Jakub OK. I can do that. I will send a new patch in the evening. Thanks Bernd.