On Tue, Aug 02, 2016 at 03:17:45PM -0500, Peter Bergner wrote: > Now that Vlad has fixed PR69847, which was the last problem holding the > rs6000 port from switching from reload to LRA, we are ready to flip the > switch. > > Is the following ok once bootstrap/regtesting on both LE and BE > (32 & 64 regtesting) comes out clean?
Oh, I don't know. David, what do you think? Segher > * config/rs6000/rs6000.c (rs6000_option_override_internal): Make LRA > the default for the rs6000 port. > > Index: gcc/config/rs6000/rs6000.c > =================================================================== > --- gcc/config/rs6000/rs6000.c (revision 238996) > +++ gcc/config/rs6000/rs6000.c (working copy) > @@ -4323,6 +4323,10 @@ rs6000_option_override_internal (bool gl > rs6000_isa_flags &= ~OPTION_MASK_P9_DFORM_SCALAR; > } > > + /* Enable LRA by default. */ > + if ((rs6000_isa_flags_explicit & OPTION_MASK_LRA) == 0) > + rs6000_isa_flags |= OPTION_MASK_LRA; > + > /* There have been bugs with -mvsx-timode that don't show up with -mlra, > but do show up with -mno-lra. Given -mlra will become the default once > PR 69847 is fixed, turn off the options with problems by default if