From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 110883 invoked by alias); 5 Mar 2015 00:54:50 -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 110867 invoked by uid 89); 5 Mar 2015 00:54:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.0 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 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 (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 05 Mar 2015 00:54:48 +0000 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id 11FC55450D0; Thu, 5 Mar 2015 01:54:44 +0100 (CET) Date: Thu, 05 Mar 2015 00:54:00 -0000 From: Jan Hubicka To: Richard Biener Cc: Jeff Law , Aldy Hernandez , gcc-patches , Jan Hubicka Subject: Re: [patch/committed] PR middle-end/65233 make walk-ssa_copies handle empty PHIs Message-ID: <20150305005444.GD86051@kam.mff.cuni.cz> References: <54F3F648.8090400@redhat.com> <54F697A5.6090908@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2015-03/txt/msg00259.txt.bz2 > > > > It gets passed the valueize callback now which returns NULL_TREE for > > SSA names we can't follow. > > Btw, for match-and-simplify I had to use that as default for fold_stmt > _exactly_ because of the call to fold_stmt from replace_uses_by > via merge-blocks from cfgcleanup. This is because replace-uses-by > doesn't have all uses replaced before it folds the stmt! > > We also have the "weaker" in-place flag. > > 2015-03-04 Richard Biener > > PR middle-end/65233 > * ipa-polymorphic-call.c: Include tree-ssa-operands.h and > tree-into-ssa.h. > (walk_ssa_copies): Revert last chage. Instead do not walk > SSA names registered for SSA update. Maybe include the patch? It should not be problem to make the function to valuelize everything it looks into. Honza