Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-04 Thread Peter Bergner
On 8/3/16 6:03 PM, David Edelsohn wrote: Please open a Bugzilla for the rs6000 backend about the vsx-timode performance regression. The vsx-timode regression needs to be fixed for GCC 7. Ok, I opened https://gcc.gnu.org/PR72804 and will start debugging the problem. Peter

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-03 Thread Peter Bergner
On 8/3/16 6:03 PM, David Edelsohn wrote: On Wed, Aug 3, 2016 at 6:59 PM, Peter Bergner wrote: My question, is since these failures are not due to LRA, do we want to consider the switch to LRA ok to commit or do we want to wait until the -mvsx-timode performance bug is

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-03 Thread David Edelsohn
On Wed, Aug 3, 2016 at 6:59 PM, Peter Bergner wrote: > On 8/2/16 3:17 PM, 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

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-03 Thread Peter Bergner
On 8/2/16 3:17 PM, 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?

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-02 Thread Segher Boessenkool
On Tue, Aug 02, 2016 at 03:38:10PM -0500, Peter Bergner wrote: > On 8/2/16 3:30 PM, David Edelsohn wrote: > >>> * config/rs6000/rs6000.c (rs6000_option_override_internal): Make > >>> LRA > >>> the default for the rs6000 port. > > > >Okay. > > > >Do we eventually want to remove the

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-02 Thread Peter Bergner
On 8/2/16 3:30 PM, David Edelsohn wrote: * config/rs6000/rs6000.c (rs6000_option_override_internal): Make LRA the default for the rs6000 port. Okay. Do we eventually want to remove the switch? I think we want to keep it for at least one release, so we can fall back to reload in

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-02 Thread David Edelsohn
On Tue, Aug 2, 2016 at 4:20 PM, Segher Boessenkool wrote: > 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

Re: [PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-02 Thread Segher Boessenkool
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 &

[PATCH, rs6000] Switch the rs6000 port over to LRA

2016-08-02 Thread Peter Bergner
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? Peter * config/rs6000/rs6000.c