On Mon, Nov 4, 2013 at 2:23 PM, Vladimir Makarov <vmaka...@redhat.com> wrote: > The following patch fixes > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58967 > > The removed code is too old. To be honest, I even don't remember why I > added this. LRA has been changed a lot since this change and now it > works fine without it. > > There is no test for this case as it is too big. > > The patch was successfully bootstrapped on ppc64 (with LRA) and tested > on ppc{32|64}. > > Is it ok to commit it to the trunk? > > 2013-11-04 Vladimir Makarov <vmaka...@redhat.com> > > PR rtl-optimization/58967 > * config/rs6000/rs6000.c (legitimate_lo_sum_address_p): Remove > !lra_in_progress for mode sizes bigger word. > > > Index: config/rs6000/rs6000.c > =================================================================== > --- config/rs6000/rs6000.c (revision 204305) > +++ config/rs6000/rs6000.c (working copy) > @@ -6388,7 +6388,7 @@ legitimate_lo_sum_address_p (enum machin > return false; > if (GET_MODE_NUNITS (mode) != 1) > return false; > - if (! lra_in_progress && GET_MODE_SIZE (mode) > UNITS_PER_WORD > + if (GET_MODE_SIZE > (mottp://gcc.gnu.org/bugzilla/show_bug.cgi?id=58967de) > UNITS_PER_WORD
This is okay, assuming the weird bugzilla URL paste shown in the patch is not really included. Thanks, David
