This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Bug rtl-optimization/34302] [4.3 regression] Invalid code reordering
- From: Kenneth Zadeck <zadeck at naturalbridge dot com>
- To: Richard Guenther <richard dot guenther at gmail dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, gcc-bugzilla at gcc dot gnu dot org, Jakub Jelinek <jakub at redhat dot com>, "Park, Seongbae" <seongbae dot park at gmail dot com>, "Bonzini, Paolo" <bonzini at gnu dot org>
- Date: Mon, 10 Dec 2007 16:32:37 -0500
- Subject: Re: [Bug rtl-optimization/34302] [4.3 regression] Invalid code reordering
- References: <475DA5AA.7010004@naturalbridge.com> <84fc9c000712101256o2e004921y90480ca215508544@mail.gmail.com>
Richard Guenther wrote:
> On Dec 10, 2007 9:46 PM, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
>
>> This patch fixes where the move insn is inserted on pre increments. it
>> had been inserted before the auto inc but this is not correct. it needs
>> to replace the existing add to properly handle the case where the
>> operands of the add change between the add and the load.
>>
>> Tested on ppc-32. The patch will only effect architectures with pre
>> increment.
>>
>> Ok for commit?
>>
>
> This is ok.
>
> Thanks,
> Richard.
>
>
>> Jakub, note that i changed the name of your testcase from 20071012-1.c
>> to 20071210-1.c.
>> I believe that this is the correct.
>>
>> Kenny
>>
>>
>> 2007-12-10 Kenneth Zadeck <zadeck@naturalbridge.com>
>>
>> PR rtl-optimization/34302
>> * auto-inc-dec.c (attempt_change): Change place where move is
>> inserted.
>>
>> 2007-12-10 Jakub Jelinek <jakub@redhat.com>
>>
>> PR rtl-optimization/34302
>> * gcc.c-torture/execute/20071210-1.c: New test.
>>
>>
>>
committed as revision 130751
Kenny