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: "Richard Guenther" <richard dot guenther at gmail dot com>
- To: "Kenneth Zadeck" <zadeck at naturalbridge 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 21:56:13 +0100
- Subject: Re: [Bug rtl-optimization/34302] [4.3 regression] Invalid code reordering
- References: <475DA5AA.7010004@naturalbridge.com>
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.
>
>