This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Switch the rs6000 port over to LRA
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Peter Bergner <peter at bergner dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, Michael Meissner <meissner at linux dot vnet dot ibm dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Tue, 2 Aug 2016 15:20:46 -0500
- Subject: Re: [PATCH, rs6000] Switch the rs6000 port over to LRA
- Authentication-results: sourceware.org; auth=none
- References: <d4245e94-6a1e-93ad-0d13-8eb016f04029@bergner.org>
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