This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug rtl-optimization/34302] [4.3 regression] Invalid code reordering
- From: "zadeck at naturalbridge dot com" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 10 Dec 2007 21:32:50 -0000
- Subject: [Bug rtl-optimization/34302] [4.3 regression] Invalid code reordering
- References: <bug-34302-50@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Comment #14 from zadeck at naturalbridge dot com 2007-12-10 21:32 -------
Subject: Re: [4.3 regression] Invalid code reordering
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
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34302