From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 36272 invoked by alias); 10 Nov 2015 00:09:47 -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 36263 invoked by uid 89); 10 Nov 2015 00:09:46 -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_HELO_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 10 Nov 2015 00:09:46 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 159AD461DA; Tue, 10 Nov 2015 00:09:45 +0000 (UTC) Received: from localhost.localdomain (vpn1-6-175.ams2.redhat.com [10.36.6.175]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tAA09hS2022320; Mon, 9 Nov 2015 19:09:44 -0500 Subject: Re: Use combined_fn in tree-vrp.c To: gcc-patches@gcc.gnu.org, richard.sandiford@arm.com References: <87io5eq8nv.fsf@e105548-lin.cambridge.arm.com> From: Bernd Schmidt Message-ID: <564135C7.6030405@redhat.com> Date: Tue, 10 Nov 2015 00:09:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <87io5eq8nv.fsf@e105548-lin.cambridge.arm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2015-11/txt/msg01138.txt.bz2 On 11/07/2015 01:46 PM, Richard Sandiford wrote: > @@ -3814,8 +3817,8 @@ extract_range_basic (value_range *vr, gimple *stmt) > break; > /* Both __builtin_ffs* and __builtin_popcount return > [0, prec]. */ > - CASE_INT_FN (BUILT_IN_FFS): > - CASE_INT_FN (BUILT_IN_POPCOUNT): > + CASE_CFN_FFS: > + CASE_CFN_POPCOUNT: > arg = gimple_call_arg (stmt, 0); > prec = TYPE_PRECISION (TREE_TYPE (arg)); > mini = 0; So let me see if I understood this. From what we discussed the purpose of these new internal functions is that they can have vector types. If so, isn't this code (here and elsewhere) which expects integers potentially going to be confused? Bernd