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

Reply via email to