[PATCH, ARM, LRA] Fixed bootstrap failure in Thumb mode

[Jeff Law]

On 11/27/13 03:18, Yvan Roux wrote:
> Ping
>
> On 18 November 2013 09:40, Yvan Roux <yvan.roux@linaro.org> wrote:
>> Ping.
>>
>> On 7 November 2013 15:56, Yvan Roux <yvan.roux@linaro.org> wrote:
>>> Hi,
>>>
>>> this patch fixed an LRA cycling due to secondary reload (Thumb mode).
>>> Notice that this patch is a prerequisite to turn on LRA by default on
>>> ARM.  Bootstrapped on a9 and a15 without any regression in the
>>> testsuite as LRA is off by default and with the regression reported in
>>> the thread bellow when LRA is on.
>>>
>>> http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00725.html
>>>
>>> Thanks,
>>> Yvan
>>>
>>> 2013-11-07  Yvan Roux  <yvan.roux@linaro.org>
>>>
>>>          * config/arm/arm.h (THUMB_SECONDARY_INPUT_RELOAD_CLASS): Return NO_REGS
>>>          for LRA
?

How can that be correct?

The secondary reload macros/hooks define cases where additional 
registers are needed to reload certain forms of rtl.  I doubt the use of 
LRA completely eliminates the need for secondary reloads.

Jeff