From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 105868 invoked by alias); 3 Mar 2017 16:27:36 -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 105853 invoked by uid 89); 3 Mar 2017 16:27:35 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-25.9 required=5.0 tests=BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,KAM_LAZY_DOMAIN_SECURITY,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=lie, our, interest X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 03 Mar 2017 16:27:34 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CB9B91424 for ; Fri, 3 Mar 2017 08:27:32 -0800 (PST) Received: from [10.2.207.77] (e100706-lin.cambridge.arm.com [10.2.207.77]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 755F93F2E5 for ; Fri, 3 Mar 2017 08:27:32 -0800 (PST) Message-ID: <58B99972.9070906@foss.arm.com> Date: Fri, 03 Mar 2017 16:27:00 -0000 From: Kyrill Tkachov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: GCC Patches Subject: [PATCH][AArch64] Fix bootstrap due to wide_int .elt (0) uninit warning Content-Type: multipart/mixed; boundary="------------070008000702050905040607" X-SW-Source: 2017-03/txt/msg00182.txt.bz2 This is a multi-part message in MIME format. --------------070008000702050905040607 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-length: 510 Hi all, This patch fixes the aarch64 bootstrap failure I've encountered. Richi suggested on IRC that we should be using .ulow () on the wide_int insetad of accessing elt (0) as that doesn't play well with the uninit analysis. Bootstrapped and tested on aarch64-none-linux-gnu. Committing to trunk in the interest of fixing the build 2017-03-03 Kyrylo Tkachov * config/aarch64/aarch64.c (aarch64_float_const_representable_p): Use wide_int::ulow () instead of .elt (0). --------------070008000702050905040607 Content-Type: text/x-patch; name="aarch64-welt.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="aarch64-welt.patch" Content-length: 523 diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index ab1bdc0..714bb79 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -12496,7 +12496,7 @@ aarch64_float_const_representable_p (rtx x) /* If the low part of the mantissa has bits set we cannot represent the value. */ - if (w.elt (0) != 0) + if (w.ulow () != 0) return false; /* We have rejected the lower HOST_WIDE_INT, so update our understanding of how many bits lie in the mantissa and --------------070008000702050905040607--