[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?
Thanks,
Kyrill

P.S. I propose this patch separate from my other patch that defines TARGET_SUPPORTS_WIDE_INT for aarch64
(https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00934.html)
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