------- 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 <[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. >> >> >> committed as revision 130751 Kenny -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34302
