From patchwork Sat Nov 14 18:10:44 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Senthil Kumar Selvaraj X-Patchwork-Id: 544773 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 39C9A1409C4 for ; Sun, 15 Nov 2015 05:11:36 +1100 (AEDT) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b=Ddk8wnGv; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; q=dns; s=default; b=uwQL5aCfFPdJleu3G Fv/HsB6TG6pTyWJx2LfVmbgQE5yhV0X5JhgydilUt1/OdvTtywyBp662eyenDcSR y/9qut9xLgouo+qzxo74BGGiX3Mhz+HWzPJfT4Ae5MCGe3xhYfWAa8AgX5TwLawD LCVT55tyY3iOaMM4UK3btRRt/g= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=default; bh=qMeKHIOi/kJK7PoHQqiodpU 226M=; b=Ddk8wnGvNbwo+Bli/2F7lLiO7LuHNS/zeDUUpuIWiV3ueFQJQKczNDS pU/5JDMBNwgGnhGV89LeYIAcSmbXuHIfjmCALLpi/yy+jbGr/C2umhGhl7RDACfH 7363MC7TYCTqmJLRCdhREbEq+4AdR5hTAMRiKVMtm+oGS4771oVM= Received: (qmail 20127 invoked by alias); 14 Nov 2015 18:11:29 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 20116 invoked by uid 89); 14 Nov 2015 18:11:28 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD, SPF_PASS autolearn=ham version=3.3.2 X-HELO: eusmtp01.atmel.com Received: from eusmtp01.atmel.com (HELO eusmtp01.atmel.com) (212.144.249.242) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Sat, 14 Nov 2015 18:11:27 +0000 Received: from HNOCHT02.corp.atmel.com (10.161.30.162) by eusmtp01.atmel.com (10.161.101.30) with Microsoft SMTP Server (TLS) id 14.3.235.1; Sat, 14 Nov 2015 19:10:58 +0100 Received: from jaguar.atmel.com (10.161.30.18) by HNOCHT02.corp.atmel.com (10.161.30.162) with Microsoft SMTP Server (TLS) id 14.3.235.1; Sat, 14 Nov 2015 19:11:04 +0100 Date: Sat, 14 Nov 2015 23:40:44 +0530 From: Senthil Kumar Selvaraj To: Richard Biener CC: , Subject: Re: [Patch, vrp] Allow VRP type conversion folding only for widenings upto word mode Message-ID: <20151114181044.GA1297@jaguar.atmel.com> References: <20151114071148.GA16647@jaguar.atmel.com> <20151114084928.GA749@jaguar.atmel.com> <595F0CF8-27C1-4A63-8339-F70CB5E76CAB@suse.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <595F0CF8-27C1-4A63-8339-F70CB5E76CAB@suse.de> User-Agent: Mutt/1.5.24 (2015-08-30) X-IsSubscribed: yes On Sat, Nov 14, 2015 at 09:57:40AM +0100, Richard Biener wrote: > On November 14, 2015 9:49:28 AM GMT+01:00, Senthil Kumar Selvaraj wrote: > >On Sat, Nov 14, 2015 at 09:13:41AM +0100, Marc Glisse wrote: > >> On Sat, 14 Nov 2015, Senthil Kumar Selvaraj wrote: > >> > >> >This patch came out of a discussion held in the gcc mailing list > >> >(https://gcc.gnu.org/ml/gcc/2015-11/msg00067.html). > >> > > >> >The patch restricts folding of conditional exprs with lhs previously > >> >set by a type conversion to occur only if the source of the type > >> >conversion's mode is word mode or smaller. > >> > > >> >Bootstrapped and reg tested on x86_64 (with > >--enable-languages=c,c++). > >> > > >> >If ok, could you commit please? I don't have commit access. > >> > > >> >Regards > >> >Senthil > >> > > >> >gcc/ChangeLog > >> > > >> >2015-11-11 Senthil Kumar Selvaraj > > > >> > > >> > * tree-vrp.c (simplify_cond_using_ranges): Fold only > >> > if innerop's mode is word_mode or smaller. > >> > > >> > > >> >diff --git gcc/tree-vrp.c gcc/tree-vrp.c > >> >index e2393e4..c139bc6 100644 > >> >--- gcc/tree-vrp.c > >> >+++ gcc/tree-vrp.c > >> >@@ -9467,6 +9467,8 @@ simplify_cond_using_ranges (gcond *stmt) > >> > innerop = gimple_assign_rhs1 (def_stmt); > >> > > >> > if (TREE_CODE (innerop) == SSA_NAME > >> >+ && (GET_MODE_SIZE(TYPE_MODE(TREE_TYPE(innerop))) > >> >+ <= GET_MODE_SIZE(word_mode)) > >> > && !POINTER_TYPE_P (TREE_TYPE (innerop))) > >> > { > >> > value_range *vr = get_value_range (innerop); > >> > >> I thought the result of the discussion was that the transformation is > >ok if > >> either it is narrowing or it widens but to something no bigger than > >> word_mode. So you should have 2 comparisons, or 1 with a max. > > > >Hmm, I came to the opposite conclusion - I thought Richard only okayed > >"widening upto word-mode", not the narrowing. > > I didn't mean to suggest narrowing is not OK. In fact narrowing is always OK. My bad. Here's a revised patch that checks for both conditions, using max as Marc suggested to limit to word_mode or narrowing conversions. Bootstrapped and regtested for x86_64 with c and c++. Is this ok? If yes, would you commit it for me please? I don't have commit access. gcc/ChangeLog 2015-11-14 Senthil Kumar Selvaraj * tree-vrp.c (simplify_cond_using_ranges): Fold only if innerop's mode smaller or equal to word_mode or op0's mode. Regards Senthil > > Richard. > > >Richard? > > > >Regards > >Senthil > >> > >> -- > >> Marc Glisse > > diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index e2393e4..cfd90e7 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -9467,7 +9467,10 @@ simplify_cond_using_ranges (gcond *stmt) innerop = gimple_assign_rhs1 (def_stmt); if (TREE_CODE (innerop) == SSA_NAME - && !POINTER_TYPE_P (TREE_TYPE (innerop))) + && !POINTER_TYPE_P (TREE_TYPE (innerop)) + && (GET_MODE_SIZE (TYPE_MODE (TREE_TYPE (innerop))) + <= std::max (GET_MODE_SIZE (word_mode), + GET_MODE_SIZE (TYPE_MODE (TREE_TYPE (op0)))))) { value_range *vr = get_value_range (innerop);