From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29685 invoked by alias); 16 Jul 2019 09:30:17 -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 29653 invoked by uid 89); 16 Jul 2019 09:30:16 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-9.7 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,GIT_PATCH_3,KAM_ASCII_DIVIDERS autolearn=ham version=3.3.1 spammy=noise, prs X-HELO: nikam.ms.mff.cuni.cz Received: from nikam.ms.mff.cuni.cz (HELO nikam.ms.mff.cuni.cz) (195.113.20.16) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 16 Jul 2019 09:30:15 +0000 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id EC7DA2825E6; Tue, 16 Jul 2019 11:30:11 +0200 (CEST) Date: Tue, 16 Jul 2019 09:30:00 -0000 From: Jan Hubicka To: Rainer Orth Cc: Richard Biener , gcc-patches@gcc.gnu.org, d@dcepelik.cz Subject: Re: Make nonoverlapping_component_refs work with duplicated main variants Message-ID: <20190716093011.cvs55cozvayx5jr3@kam.mff.cuni.cz> References: <20190708072649.vqd5u6jxsz5ybtt7@kam.mff.cuni.cz> <20190709114917.qva4nb2h7j5vzdur@kam.mff.cuni.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170113 (1.7.2) X-SW-Source: 2019-07/txt/msg01098.txt.bz2 > Hi Jan, > > > * g++.dg/lto/alias-3_0.C: New file. > > * g++.dg/lto/alias-3_1.c: New file. > > the new test has a couple of problems: DejaGnu warns everywhere: > > WARNING: lto.exp does not support dg-lto-do in secondary source files > WARNING: lto.exp does not support dg-lto-options in secondary source files > > This would have been prominent in either mail-report.log or the runtest > output. > > Besides, the test FAILs in the same way as its companions lto/alias-[12] > on Solaris (PRs ipa/90720 and lto/91028). Your fix for the latter two > didn't change anything, btw., neither on Solaris nor on Linux/x86_64 > with -fno-use-linker-plugin. Hi, sorry for the noise. This patch fixes -fno-use-linker-plugin for me and also removes the extra dg-do I forgot in alias_3-1.c * alias-1_0.C: Use -O3. * alias-2_0.C: Use -O3. * alias-3_0.C: Add loop to enable inlining with -fno-use-linker-plugin. * alias-3_1.C: Remove dg-lto-do and dg-lto-options. Index: g++.dg/lto/alias-1_0.C =================================================================== --- g++.dg/lto/alias-1_0.C (revision 273478) +++ g++.dg/lto/alias-1_0.C (working copy) @@ -1,5 +1,5 @@ /* { dg-lto-do run } */ -/* { dg-lto-options { { -O2 -flto } } } */ +/* { dg-lto-options { { -O3 -flto } } } */ /* With LTO we consider all pointers to incomplete types to be possibly aliasing. This makes *bptr to alias with aptr. Index: g++.dg/lto/alias-2_0.C =================================================================== --- g++.dg/lto/alias-2_0.C (revision 273478) +++ g++.dg/lto/alias-2_0.C (working copy) @@ -1,5 +1,5 @@ /* { dg-lto-do run } */ -/* { dg-lto-options { { -O2 -flto } } } */ +/* { dg-lto-options { { -O3 -flto } } } */ /* With LTO we consider all pointers to incomplete types to be possibly aliasing. This makes *bptr to alias with aptr. Index: g++.dg/lto/alias-3_0.C =================================================================== --- g++.dg/lto/alias-3_0.C (revision 273478) +++ g++.dg/lto/alias-3_0.C (working copy) @@ -14,13 +14,15 @@ __attribute__ ((used)) struct b b, *bptr __attribute__ ((used)) int i,j; extern "C" void inline_me_late (void); +int n=1; int main (void) { int jj=j; bptr2->a[jj].bar = 0; - inline_me_late (); + for (int i=0; ia[jj].bar == 0)) __builtin_abort (); return 0; Index: g++.dg/lto/alias-3_1.c =================================================================== --- g++.dg/lto/alias-3_1.c (revision 273478) +++ g++.dg/lto/alias-3_1.c (working copy) @@ -1,5 +1,3 @@ -/* { dg-lto-do run } */ -/* { dg-lto-options { { -O3 -flto -fno-early-inlining } } } */ struct a { int foo,bar;