From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10746 invoked by alias); 28 Nov 2017 08:14:10 -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 10689 invoked by uid 89); 28 Nov 2017 08:14:10 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-11.2 required=5.0 tests=BAYES_00,GIT_PATCH_2,GIT_PATCH_3,KAM_NUMSUBJECT,KB_WAM_FROM_NAME_SINGLEWORD,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy= X-HELO: mx2.suse.de Received: from mx2.suse.de (HELO mx2.suse.de) (195.135.220.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 28 Nov 2017 08:14:08 +0000 Received: from relay1.suse.de (charybdis-ext.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 766C0AAB5 for ; Tue, 28 Nov 2017 08:14:06 +0000 (UTC) Date: Tue, 28 Nov 2017 08:31:00 -0000 From: Richard Biener To: gcc-patches@gcc.gnu.org Subject: [PATCH] Fix PR83158 Message-ID: User-Agent: Alpine 2.20 (LSU 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-SW-Source: 2017-11/txt/msg02367.txt.bz2 Recent changes caused VRP to create [-2147483646, +INF] from merging ~[0, 0] and [-2147483646, +INF]. This causes i386 specific folding of lznct to no longer trigger. The following extends the existing special-casing of ~[0, 0] from pointer-like types to pointer types and integer types > int (covering integral arguments). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2017-11-28 Richard Biener PR tree-optimization/83158 * tree-vrp.c (intersect_ranges): Prefer ~[0, 0] in a few more cases. Index: gcc/tree-vrp.c =================================================================== --- gcc/tree-vrp.c (revision 255173) +++ gcc/tree-vrp.c (working copy) @@ -6021,11 +6021,14 @@ intersect_ranges (enum value_range_type && vrp_val_is_max (vr1max)) ; /* Choose the anti-range if it is ~[0,0], that range is special - enough to special case when vr1's range is relatively wide. */ + enough to special case when vr1's range is relatively wide. + At least for types bigger than int - this covers pointers + and arguments to functions like ctz. */ else if (*vr0min == *vr0max && integer_zerop (*vr0min) - && (TYPE_PRECISION (TREE_TYPE (*vr0min)) - == TYPE_PRECISION (ptr_type_node)) + && ((TYPE_PRECISION (TREE_TYPE (*vr0min)) + >= TYPE_PRECISION (integer_type_node)) + || POINTER_TYPE_P (TREE_TYPE (*vr0min))) && TREE_CODE (vr1max) == INTEGER_CST && TREE_CODE (vr1min) == INTEGER_CST && (wi::clz (wi::to_wide (vr1max) - wi::to_wide (vr1min))