[PATCH][AArch64][cleanup] Remove uses of CONST_DOUBLE_HIGH, CONST_DOUBLE_LOW

Kyrill Tkachov kyrylo.tkachov@arm.com
Mon Nov 9 11:34:00 GMT 2015

Hi all,

 From what I understand, we shouldn't encounter the case where CONST_DOUBLE is used to hold
a large integer value, accessed through CONST_DOUBLE_LOW and CONST_DOUBLE_HIGH.
The aarch64 backend doesn't generate any such rtxes and the midend shouldn't be passing
any such rtxes either.  This allows for a bit of cleanup in aarch64_simd_valid_immediate.

Bootstrapped and tested on aarch64.

Ok for trunk?

P.S. I propose this patch separate from my other patch that defines TARGET_SUPPORTS_WIDE_INT for aarch64
as this is not needed to fix the ICE in that case and just removes some dead code and is thus not
appropriate for backporting in my opinion.

2015-11-09  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * config/aarch64/aarch64.c (aarch64_simd_valid_immediate):
     Remove integer CONST_DOUBLE handling.  It should never occur.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: aarch64-const-double_high-low.patch
Type: text/x-patch
Size: 1607 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20151109/c1e7a584/attachment.bin>

More information about the Gcc-patches mailing list