From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5612 invoked by alias); 19 Jun 2011 12:45:19 -0000 Received: (qmail 5604 invoked by uid 22791); 19 Jun 2011 12:45:18 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from nikam.ms.mff.cuni.cz (HELO nikam.ms.mff.cuni.cz) (195.113.20.16) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sun, 19 Jun 2011 12:45:04 +0000 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id DCD889AC7F3; Sun, 19 Jun 2011 14:45:02 +0200 (CEST) Date: Sun, 19 Jun 2011 12:55:00 -0000 From: Jan Hubicka To: Hans-Peter Nilsson Cc: Jan Hubicka , gcc-patches@gcc.gnu.org Subject: Re: Cgraph alias reorg 15/14 (New infrastructure for same body aliases) Message-ID: <20110619124502.GA1199@kam.mff.cuni.cz> References: <20110611125552.GA25068@kam.mff.cuni.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.18 (2008-05-17) 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 X-SW-Source: 2011-06/txt/msg01410.txt.bz2 > On Sat, 11 Jun 2011, Jan Hubicka wrote: > > > Hi, > > this patch complettes the same body alias rework by removing the old same body > > alias code and adding new representation. Same body aliases are now separate > > function nodes that have IPA_REF_ALIAS reference to the node they are alias of. > > > > I am still getting one failure: > > FAIL: g++.dg/torture/pr43879-1_1.C > > > > It tests IPA PTA in presence of same body aliases. I honestly have no idea > > what is wrong there. I decided to go ahead with the patch anyway, given the > > current state of affair of aliases and IPA-PTA. Hope Richard will help me > > fixing this on Monday. > > Still there, see again PR49373. Anything better than a xfail in the works? Yes, we discussed the problem with Richi and the issue is that ipa-pta gets confused by inliner redirecting edge from alias to the real inline clone. The proper fix is to teach passmanager to do small ipa passes before final compilation. I am currently at the GCC gathering, so I plan to implement this day after tomorrow after returning. Honza > > brgds, H-P