------- Comment #12 from richard dot guenther at gmail dot com 2007-12-10 20:56 ------- Subject: Re: [4.3 regression] Invalid code reordering
On Dec 10, 2007 9:46 PM, Kenneth Zadeck <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> > > PR rtl-optimization/34302 > * auto-inc-dec.c (attempt_change): Change place where move is > inserted. > > 2007-12-10 Jakub Jelinek <[EMAIL PROTECTED]> > > PR rtl-optimization/34302 > * gcc.c-torture/execute/20071210-1.c: New test. > > -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34302