[PATCH][ARM][cleanup] Remove uses of CONST_DOUBLE_HIGH/LOW

Kyrill Tkachov kyrylo.tkachov@arm.com
Tue Nov 10 17:28:00 GMT 2015


Hi Ramana,

On 10/11/15 14:33, Ramana Radhakrishnan wrote:
> On Thu, Nov 5, 2015 at 9:32 AM, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
>> Hi all,
>>
>> This cleanup patch removes handling of CONST_DOUBLE rtxes that carry large
>> integers.
>> These should never be passed down from the midend and the arm backend
>> doesn't create them.
>> The code has been there since 2007 but the arm backend was moved to
>> TARGET_SUPPORTS_WIDE_INT
>> in 2014, so this path should never be taken.
>>
>> Bootstrapped and tested on arm-none-linux-gnueabihf.
>>
>> Ok for trunk?
>
> This is OK -

Thanks for reviewing.
Sorry I had forgotten the ChangeLog in the initial submission.
It is
2015-11-10  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * config/arm/arm.c (neon_valid_immediate): Remove integer
     CONST_DOUBLE handling.  It should never occur.

I have committed the patch with that entry as r230115.

Thanks,
Kyrill

> Ramana
>> Thanks,
>> Kyrill
>>



More information about the Gcc-patches mailing list