public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][AArch64] Fix bootstrap due to wide_int .elt (0) uninit warning
@ 2017-03-03 16:27 Kyrill Tkachov
  0 siblings, 0 replies; only message in thread
From: Kyrill Tkachov @ 2017-03-03 16:27 UTC (permalink / raw)
  To: GCC Patches

[-- Attachment #1: Type: text/plain, Size: 510 bytes --]

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  <kyrylo.tkachov@arm.com>

     * config/aarch64/aarch64.c (aarch64_float_const_representable_p):
     Use wide_int::ulow () instead of .elt (0).

[-- Attachment #2: aarch64-welt.patch --]
[-- Type: text/x-patch, Size: 523 bytes --]

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-03-03 16:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-03 16:27 [PATCH][AArch64] Fix bootstrap due to wide_int .elt (0) uninit warning Kyrill Tkachov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).