Re: reload autoinc fix

2014-01-20 Thread Bernd Schmidt
On 01/16/2014 09:53 PM, Jeff Law wrote: I happened to be going through the list of regressions and came across this again. I went ahead and installed the patch on the trunk and closed the BZ. Thanks. I was travelling and decided not to install any patches before or during that. Bernd

Re: reload autoinc fix

2014-01-16 Thread Jeff Law
On 01/07/14 09:16, Bernd Schmidt wrote: This is PR56791. The address inside of an autoinc is reloaded, and the autoinc is reloaded, but the reload insns are emitted in the wrong order. As far as I can tell, this is because find_reloads_address_1 has two methods of pushing a reload for an autoinc

Re: reload autoinc fix

2014-01-08 Thread Richard Earnshaw
On 07/01/14 21:06, Andrew Pinski wrote: > On Tue, Jan 7, 2014 at 12:55 PM, Jeff Law wrote: >> On 01/07/14 09:16, Bernd Schmidt wrote: >>> >>> This is PR56791. The address inside of an autoinc is reloaded, and the >>> autoinc is reloaded, but the reload insns are emitted in the wrong order. >>> >>>

Re: reload autoinc fix

2014-01-07 Thread Jeff Law
On 01/07/14 14:06, Andrew Pinski wrote: On Tue, Jan 7, 2014 at 12:55 PM, Jeff Law wrote: On 01/07/14 09:16, Bernd Schmidt wrote: This is PR56791. The address inside of an autoinc is reloaded, and the autoinc is reloaded, but the reload insns are emitted in the wrong order. As far as I can te

Re: reload autoinc fix

2014-01-07 Thread Andrew Pinski
On Tue, Jan 7, 2014 at 12:55 PM, Jeff Law wrote: > On 01/07/14 09:16, Bernd Schmidt wrote: >> >> This is PR56791. The address inside of an autoinc is reloaded, and the >> autoinc is reloaded, but the reload insns are emitted in the wrong order. >> >> As far as I can tell, this is because find_relo

Re: reload autoinc fix

2014-01-07 Thread Jeff Law
On 01/07/14 09:16, Bernd Schmidt wrote: This is PR56791. The address inside of an autoinc is reloaded, and the autoinc is reloaded, but the reload insns are emitted in the wrong order. As far as I can tell, this is because find_reloads_address_1 has two methods of pushing a reload for an autoinc