From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15777 invoked by alias); 13 Sep 2011 10:09:39 -0000 Received: (qmail 15763 invoked by uid 22791); 13 Sep 2011 10:09:37 -0000 X-SWARE-Spam-Status: No, hits=-2.9 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 13 Sep 2011 10:09:24 +0000 From: "hubicka at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug other/49533] [4.7 regression] Revision 174989 (ipa-inline-transform.c) regressions Date: Tue, 13 Sep 2011 10:15:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: other X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: hubicka at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 4.7.0 X-Bugzilla-Changed-Fields: Attachment #24602 is obsolete Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2011-09/txt/msg00890.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49533 Jan Hubicka changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #24602|0 |1 is obsolete| | --- Comment #11 from Jan Hubicka 2011-09-13 10:09:17 UTC --- Created attachment 25261 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=25261 Proposed fix Hi, there seems to be 3 bugs cooperating on this problem. 1) we sometimes forget to output alias in comdat group since assemble_alias gets into wrong code path when called from cgraphunit 2) I made pasto while walking aliases testing always the master node instead of alias 3) test checking aliases looks only for unremovable function that are called, instead for functions that are either unremovable or called. Honza