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

2013-11-28 Thread Richard Earnshaw
On 27/11/13 18:27, Jeff Law wrote: > On 11/27/13 10:49, Yvan Roux wrote: >>> 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 secon

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

2013-11-27 Thread Yvan Roux
On 27 November 2013 19:27, Jeff Law wrote: > On 11/27/13 10:49, Yvan Roux wrote: >>> >>> 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 n

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

2013-11-27 Thread Vladimir Makarov
On 11/27/2013, 1:20 PM, Jeff Law wrote: On 11/27/13 10:58, Vladimir Makarov wrote: 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. When I design

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

2013-11-27 Thread Jeff Law
On 11/27/13 10:49, Yvan Roux wrote: 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. Vladimir explained me that in that cas

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

2013-11-27 Thread Jeff Law
On 11/27/13 10:58, Vladimir Makarov wrote: 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. When I designed LRA I wanted to remove as many hooks a

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

2013-11-27 Thread Yvan Roux
On 27 November 2013 18:58, Vladimir Makarov wrote: > On 11/27/2013, 12:16 PM, Jeff Law wrote: >> >> On 11/27/13 03:18, Yvan Roux wrote: >>> >>> Ping >>> >>> On 18 November 2013 09:40, Yvan Roux wrote: Ping. On 7 November 2013 15:56, Yvan Roux wrote: > > Hi, >

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

2013-11-27 Thread Vladimir Makarov
On 11/27/2013, 12:16 PM, Jeff Law wrote: On 11/27/13 03:18, Yvan Roux wrote: Ping On 18 November 2013 09:40, Yvan Roux wrote: Ping. On 7 November 2013 15:56, Yvan Roux wrote: Hi, this patch fixed an LRA cycling due to secondary reload (Thumb mode). Notice that this patch is a prerequisite

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

2013-11-27 Thread Richard Earnshaw
On 27/11/13 17:49, Yvan Roux wrote: >> 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. > > Vladimir explained me t

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

2013-11-27 Thread Yvan Roux
> 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. Vladimir explained me that in that case on arm, secondary reload hoo

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

2013-11-27 Thread Jeff Law
On 11/27/13 03:18, Yvan Roux wrote: Ping On 18 November 2013 09:40, Yvan Roux wrote: Ping. On 7 November 2013 15:56, Yvan Roux 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. Bootst

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

2013-11-27 Thread Yvan Roux
Ping On 18 November 2013 09:40, Yvan Roux wrote: > Ping. > > On 7 November 2013 15:56, Yvan Roux 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

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

2013-11-18 Thread Yvan Roux
Ping. On 7 November 2013 15:56, Yvan Roux 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 de