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

[Kyrill Tkachov]

Hi Ramana,

On 10/11/15 14:33, Ramana Radhakrishnan wrote:

On Thu, Nov 5, 2015 at 9:32 AM, Kyrill Tkachov <kyrylo.tkac...@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.tkac...@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