From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10704 invoked by alias); 9 Jan 2013 10:33:16 -0000 Received: (qmail 10688 invoked by uid 22791); 9 Jan 2013 10:33:15 -0000 X-SWARE-Spam-Status: No, hits=-5.4 required=5.0 tests=AWL,BAYES_00,KHOP_RCVD_UNTRUST,RCVD_IN_DNSWL_HI X-Spam-Check-By: sourceware.org Received: from cantor2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Wed, 09 Jan 2013 10:32:43 +0000 Received: from relay1.suse.de (unknown [195.135.220.254]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx2.suse.de (Postfix) with ESMTP id 61F7AA51B8 for ; Wed, 9 Jan 2013 11:32:42 +0100 (CET) Date: Wed, 09 Jan 2013 10:33:00 -0000 From: Martin Jambor To: GCC Patches Subject: Re: PR tree-optimization/55823 (ipa-inline-transform ICE) Message-ID: <20130109103240.GC22164@virgil.suse> Mail-Followup-To: GCC Patches References: <20130107002623.GA12714@kam.mff.cuni.cz> <20130108132900.GB22164@virgil.suse> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes 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: 2013-01/txt/msg00477.txt.bz2 Hi, On Tue, Jan 08, 2013 at 02:32:24PM +0100, Richard Biener wrote: > On Tue, Jan 8, 2013 at 2:29 PM, Martin Jambor wrote: > > Hi, > > > > On Mon, Jan 07, 2013 at 01:26:23AM +0100, Jan Hubicka wrote: > >> Hi, > >> as discused in the PR log there seems to be ordering issue in > >> update_indirect_edges_after_inlining that first updates info in call edge to > >> correspond the situation after inlining and then it tries to devirtualize that > >> is trying to look up the info prior inlining. > >> > >> Bootstrapped/regtested x86_64-linux > >> Martin, does it look sane? > > > > Yes, this is exactly what needs to be done. I'm quite surprised I had > > not already added a testcase for this. > > Is this maybe related to PR55264? No, that PR is actually a symtab/cgraph/inlining issue. > > The patch is also not yet applied btw ... Honza has applied it recently. Thanks, Martin