From patchwork Thu Nov 3 21:52:33 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bernd Edlinger X-Patchwork-Id: 691048 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3t8zGy54JYz9tkH for ; Fri, 4 Nov 2016 08:52:57 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b=G+gIj8KL; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:subject:date:message-id:content-type:mime-version; q=dns; s= default; b=FSXeHcOGHqwb2PG2a/EeZyMMPiVNNsZ7oJF0W8E1F6XZNJbfORDo9 lcXp0OTpGv2peKyur4PsVQvZdF7+3SXh8fqtwK8ZtiRJdr56t5pLqfD6vujH62Wp ZHsN62yfP7IVrt2LORr9fPG2S5ZYvVeSWNnFkZcWxEEtwFm4Ayo/ks= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:from :to:subject:date:message-id:content-type:mime-version; s= default; bh=ETf8ZpKipEGF+zFISOippyf+M48=; b=G+gIj8KL91pBuQmpr4cn COra/+x67cTnV7GAtcs3xtuejuPIMh8HUCLNvGHMMGPEyZfa7/iobdMipbj9PCXn hDN3JmfWdt6O2vHqj5CQAELDDZF77NiJr5vgxvtCTtXI07IQFe5NXbZtoXw1GTiV edLlqbfp272vn22D58qREAQ= Received: (qmail 72505 invoked by alias); 3 Nov 2016 21:52:48 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 72493 invoked by uid 89); 3 Nov 2016 21:52:47 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=1.3 required=5.0 tests=AWL, BAYES_50, FREEMAIL_FROM, KAM_ASCII_DIVIDERS, RCVD_IN_DNSWL_NONE, SPF_PASS autolearn=no version=3.3.2 spammy=giovanni, Origin, giovannibajo, bci X-HELO: COL004-OMC2S11.hotmail.com Received: from col004-omc2s11.hotmail.com (HELO COL004-OMC2S11.hotmail.com) (65.55.34.85) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 03 Nov 2016 21:52:37 +0000 Received: from EUR03-VE1-obe.outbound.protection.outlook.com ([65.55.34.73]) by COL004-OMC2S11.hotmail.com over TLS secured channel with Microsoft SMTPSVC(7.5.7601.23008); Thu, 3 Nov 2016 14:52:36 -0700 Received: from DB5EUR03FT061.eop-EUR03.prod.protection.outlook.com (10.152.20.55) by DB5EUR03HT065.eop-EUR03.prod.protection.outlook.com (10.152.21.181) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.693.6; Thu, 3 Nov 2016 21:52:33 +0000 Received: from AM4PR0701MB2162.eurprd07.prod.outlook.com (10.152.20.52) by DB5EUR03FT061.mail.protection.outlook.com (10.152.21.234) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P384) id 15.1.693.6 via Frontend Transport; Thu, 3 Nov 2016 21:52:34 +0000 Received: from AM4PR0701MB2162.eurprd07.prod.outlook.com ([10.167.132.147]) by AM4PR0701MB2162.eurprd07.prod.outlook.com ([10.167.132.147]) with mapi id 15.01.0679.020; Thu, 3 Nov 2016 21:52:34 +0000 From: Bernd Edlinger To: "gcc-patches@gcc.gnu.org" , Jason Merrill Subject: [PATCH] Fix wrong declarations of builtin functions Date: Thu, 3 Nov 2016 21:52:33 +0000 Message-ID: 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-incomingtopheadermarker: OriginalChecksum:; UpperCasedChecksum:; SizeAsReceived:7089; Count:34 x-ms-exchange-messagesentrepresentingtype: 1 x-incomingheadercount: 34 x-eopattributedmessage: 0 x-microsoft-exchange-diagnostics: 1; DB5EUR03HT065; 7:uIrWypuxTa0OLIFjN4/Iqi5w9xxynvfOh+DwqJo+rrepJhabhX/LRw/pPyAhxngnXntVIWia8YzExT153Fh6Jgu0LhqCKHES2BpgXHc5jlWi9DATQV6A1N+NG4Fr7OK74Z9+p6Ton4LOEbmHkRVWEQZcycSTXOT4IVuaTVea/q4oq7zVTaHyvavC+23e+Hxf5EWZCm3hM2fA6ObO3sv25QkNa1O9wG8d+bvNfGVNPzUVglYIQq9GpxMzEQ9nwrJvo0etnU9TFtFIXjSFXlrdyt6p2/DNSEW5YyNAOs/LR07WnBMdbGo+roT6AFC3s703WAdxl6FrpiFJ+IXcpO/JmuwKB7yS2eOxz/v5FN8npiI= x-forefront-antispam-report: EFV:NLI; SFV:NSPM; SFS:(10019020)(98900003); DIR:OUT; SFP:1102; SCL:1; SRVR:DB5EUR03HT065; H:AM4PR0701MB2162.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; x-ms-office365-filtering-correlation-id: b4f1c174-bf9e-4298-1325-08d40433b746 x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(1601124038)(1603103081)(1601125047); SRVR:DB5EUR03HT065; x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(432015012)(102415321)(82015046); SRVR:DB5EUR03HT065; BCL:0; PCL:0; RULEID:; SRVR:DB5EUR03HT065; x-forefront-prvs: 011579F31F spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM MIME-Version: 1.0 X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Nov 2016 21:52:33.9005 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Internet X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB5EUR03HT065 Hi, I thought in preparation of the new C++ warning about wrong declarations of builtin functions it would be good to fix some of the more obvious bugs in the testsuite first. Bootstrapped and reg-tested on x86_64-pc-linux-gnu. Is it OK for trunk? Thanks Bernd. 2016-11-03 Bernd Edlinger PR c++/71973 * g++.dg/cpp1y/lambda-generic-udt.C: Fix builtin function declaration. * g++.dg/init/new15.C: Likewise. * g++.dg/ipa/inline-1.C: Likewise. * g++.dg/ipa/inline-2.C: Likewise. * g++.dg/lto/20080908-1_0.C: Likewise. * g++.dg/tc1/dr20.C: Likewise. * g++.dg/tree-ssa/inline-1.C: Likewise. * g++.dg/tree-ssa/inline-2.C: Likewise. * g++.old-deja/g++.law/except1.C: Likewise. * g++.old-deja/g++.other/vbase5.C: Likewise. * obj-c++.dg/lto/trivial-1_0.mm: Likewise. Index: gcc/testsuite/g++.dg/cpp1y/lambda-generic-udt.C =================================================================== --- gcc/testsuite/g++.dg/cpp1y/lambda-generic-udt.C (revision 241831) +++ gcc/testsuite/g++.dg/cpp1y/lambda-generic-udt.C (working copy) @@ -14,7 +14,7 @@ bool shadow = false; }; -extern "C" void printf(...); +extern "C" int printf(const char*, ...); #define assert(e) if (e); else \ printf ("%s:%d: !(%s)\n", __FILE__, __LINE__, #e), __builtin_abort (); Index: gcc/testsuite/g++.dg/init/new15.C =================================================================== --- gcc/testsuite/g++.dg/init/new15.C (revision 241831) +++ gcc/testsuite/g++.dg/init/new15.C (working copy) @@ -1,6 +1,6 @@ // PR c++/9782 -extern "C" void printf(const char*, ...); +extern "C" int printf(const char*, ...); template struct A { Index: gcc/testsuite/g++.dg/ipa/inline-1.C =================================================================== --- gcc/testsuite/g++.dg/ipa/inline-1.C (revision 241831) +++ gcc/testsuite/g++.dg/ipa/inline-1.C (working copy) @@ -3,7 +3,7 @@ /* { dg-add-options bind_pic_locally } */ namespace std { - extern "C" void puts(const char *s); + extern "C" int puts(const char *s); } template void Index: gcc/testsuite/g++.dg/ipa/inline-2.C =================================================================== --- gcc/testsuite/g++.dg/ipa/inline-2.C (revision 241831) +++ gcc/testsuite/g++.dg/ipa/inline-2.C (working copy) @@ -3,7 +3,7 @@ /* { dg-add-options bind_pic_locally } */ namespace std { - extern "C" void puts(const char *s); + extern "C" int puts(const char *s); } template void Index: gcc/testsuite/g++.dg/lto/20080908-1_0.C =================================================================== --- gcc/testsuite/g++.dg/lto/20080908-1_0.C (revision 241831) +++ gcc/testsuite/g++.dg/lto/20080908-1_0.C (working copy) @@ -1,5 +1,5 @@ /* { dg-lto-do run } */ -extern "C" { extern void *memcpy (void *, const void *, unsigned); } +extern "C" { extern void *memcpy (void *, const void *, __SIZE_TYPE__); } inline int bci (const float &source) Index: gcc/testsuite/g++.dg/tc1/dr20.C =================================================================== --- gcc/testsuite/g++.dg/tc1/dr20.C (revision 241831) +++ gcc/testsuite/g++.dg/tc1/dr20.C (working copy) @@ -2,7 +2,7 @@ // Origin: Giovanni Bajo // DR20: Some clarifications needed for 12.8 para 15 -extern "C" void printf(const char*, ...); +extern "C" int printf(const char*, ...); extern "C" void abort(void); int count = 0; Index: gcc/testsuite/g++.dg/tree-ssa/inline-1.C =================================================================== --- gcc/testsuite/g++.dg/tree-ssa/inline-1.C (revision 241831) +++ gcc/testsuite/g++.dg/tree-ssa/inline-1.C (working copy) @@ -3,7 +3,7 @@ /* { dg-add-options bind_pic_locally } */ namespace std { - extern "C" void puts(const char *s); + extern "C" int puts(const char *s); } template void Index: gcc/testsuite/g++.dg/tree-ssa/inline-2.C =================================================================== --- gcc/testsuite/g++.dg/tree-ssa/inline-2.C (revision 241831) +++ gcc/testsuite/g++.dg/tree-ssa/inline-2.C (working copy) @@ -3,7 +3,7 @@ /* { dg-add-options bind_pic_locally } */ namespace std { - extern "C" void puts(const char *s); + extern "C" int puts(const char *s); } template void Index: gcc/testsuite/g++.old-deja/g++.law/except1.C =================================================================== --- gcc/testsuite/g++.old-deja/g++.law/except1.C (revision 241831) +++ gcc/testsuite/g++.old-deja/g++.law/except1.C (working copy) @@ -7,7 +7,7 @@ // Subject: Bugs // Date: Wed, 22 Jul 92 08:29:30 EDT -extern "C" void puts(const char *); +extern "C" int puts(const char *); class foo { public: Index: gcc/testsuite/g++.old-deja/g++.other/vbase5.C =================================================================== --- gcc/testsuite/g++.old-deja/g++.other/vbase5.C (revision 241831) +++ gcc/testsuite/g++.old-deja/g++.other/vbase5.C (working copy) @@ -6,7 +6,7 @@ // vbases. Normally that's just a pessimization, unfortunately during // constructoring it leads to uninitialized reads. -extern "C" int printf (...); +extern "C" int printf (const char*,...); int fail = 0; Index: gcc/testsuite/obj-c++.dg/lto/trivial-1_0.mm =================================================================== --- gcc/testsuite/obj-c++.dg/lto/trivial-1_0.mm (revision 241831) +++ gcc/testsuite/obj-c++.dg/lto/trivial-1_0.mm (working copy) @@ -1,7 +1,7 @@ /* { dg-lto-do run } */ /* { dg-skip-if "Needs OBJC2 ABI" { "*-*-darwin*" && lp64 } { "*" } { "" } } */ extern "C" { -extern int printf (char *,...) ; +extern int printf (const char *,...) ; extern void abort (void) ; }