On 09/20/2012 07:42 PM, H.J. Lu wrote:
> On Fri, Aug 3, 2012 at 5:05 AM, Bernd Schmidt wrote:
>> This patch allows us to change
>>
>> rn++
>> rm=[rn]
>>
>> into
>>
>> rm=[rn + 4]
>> rn++
>>
> This caused:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54645
The jury is still out on that, but it
On Fri, Aug 3, 2012 at 5:05 AM, Bernd Schmidt wrote:
> This patch allows us to change
>
> rn++
> rm=[rn]
>
> into
>
> rm=[rn + 4]
> rn++
>
> Opportunities to do this are discovered by a mini-pass over the
> instructions after generating dependencies and before scheduling a
> block. At that point w
On 12/09/2012, at 4:34 AM, Vladimir Makarov wrote:
> On 08/03/2012 08:05 AM, Bernd Schmidt wrote:
>>
...
> Ok, thanks. The changes are pretty straightforward. Only just a few
> comments.
>
...
>
> Thanks for the patch, Bernd. Sorry for the delay with the review. I thought
> that Maxim wri
On 08/03/2012 08:05 AM, Bernd Schmidt wrote:
This patch allows us to change
rn++
rm=[rn]
into
rm=[rn + 4]
rn++
That is an interesting optimization. I think analogous optimization
could be done for INC/DEC addressing (probably it might be beneficial
for ppc which has such addressing and disp
On 08/03/2012 02:05 PM, Bernd Schmidt wrote:
> This patch allows us to change
>
> rn++
> rm=[rn]
>
> into
>
> rm=[rn + 4]
> rn++
Ping.
Bernd
On 4/08/2012, at 12:05 AM, Bernd Schmidt wrote:
> This patch allows us to change
>
> rn++
> rm=[rn]
>
> into
>
> rm=[rn + 4]
> rn++
The patch is OK with the following nitpicks.
[BTW, if anyone else wants to review this patch, it helps to read it from the
end.]
>
> Opportunities to do this
On 4/08/2012, at 12:05 AM, Bernd Schmidt wrote:
> This patch allows us to change
>
> rn++
> rm=[rn]
>
> into
>
> rm=[rn + 4]
> rn++
This is a good scheduler optimization that I wanted to have for quite a while.
Bernd, kudos for implementing it!
I am going to review this patch, and then the