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

[Jeff Law <law at redhat dot com>]

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