Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 03:44:37PM -0400, Michael Meissner wrote: > > I hope you mean for ppc only, otherwise you're breaking a lot of ports... > > Yes I mean for PowerPC only. I can change the switches name to: > > --enable-powerpc-lra > --enable-powerpc-float128 > > if it would

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Michael Meissner
On Tue, Jul 12, 2016 at 01:21:47PM +0200, Bernd Schmidt wrote: > On 07/12/2016 12:29 PM, Richard Biener wrote: > > >Instead of adding more configury can we please enable LRA on trunk by default > >_now_? Otherwise the amount of testing it recieves won't really increase. > > I hope you mean for

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Bernd Schmidt
On 07/12/2016 06:31 PM, Segher Boessenkool wrote: Do you have a testcase? This sounds like fun :-) This is bfin-elf, gcc.c-torture/compile/pr59417.c, -O3 -fomit-frame-pointer. Bernd

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 05:17:04PM +0200, Bernd Schmidt wrote: > The Blackfin thing happens frequently with -fomit-frame-pointer when we have > > (insn 66 64 70 8 (set (reg:SI 96 [ ivtmp.32 ]) > (reg/f:SI 15 FP)) 19 {*movsi_insn} > > Which LRA transforms to an invalid insn: > > (insn 66

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Bernd Schmidt
On 07/12/2016 02:48 PM, Bernd Schmidt wrote: No. You can reproduce issues with Blackfin easily by enabling LRA for it, and I described the C6X issues when the LRA patches were posted for review. That was here: https://gcc.gnu.org/ml/gcc-patches/2012-10/msg00235.html The Blackfin thing

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 02:40:28PM +0200, Bernd Schmidt wrote: > On 07/12/2016 02:15 PM, Segher Boessenkool wrote: > >How can LRA ever be made default for all targets without breaking those > >that do not want to change? > > LRA advocates would have to fix the issues with it that prevent it from

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Bernd Schmidt
On 07/12/2016 02:45 PM, Segher Boessenkool wrote: On Tue, Jul 12, 2016 at 02:40:28PM +0200, Bernd Schmidt wrote: On 07/12/2016 02:15 PM, Segher Boessenkool wrote: How can LRA ever be made default for all targets without breaking those that do not want to change? LRA advocates would have to

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 02:40:28PM +0200, Bernd Schmidt wrote: > On 07/12/2016 02:15 PM, Segher Boessenkool wrote: > >How can LRA ever be made default for all targets without breaking those > >that do not want to change? > > LRA advocates would have to fix the issues with it that prevent it from

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Bernd Schmidt
On 07/12/2016 02:15 PM, Segher Boessenkool wrote: How can LRA ever be made default for all targets without breaking those that do not want to change? LRA advocates would have to fix the issues with it that prevent it from being usable on certain targets. Based on my initial experiments with

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 01:31:35PM +0200, Richard Biener wrote: > On Tue, Jul 12, 2016 at 1:21 PM, Bernd Schmidt wrote: > > On 07/12/2016 12:29 PM, Richard Biener wrote: > > > >> Instead of adding more configury can we please enable LRA on trunk by > >> default > >> _now_?

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Richard Biener
On Tue, Jul 12, 2016 at 1:21 PM, Bernd Schmidt wrote: > On 07/12/2016 12:29 PM, Richard Biener wrote: > >> Instead of adding more configury can we please enable LRA on trunk by >> default >> _now_? Otherwise the amount of testing it recieves won't really increase. > > > I

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Bernd Schmidt
On 07/12/2016 12:29 PM, Richard Biener wrote: Instead of adding more configury can we please enable LRA on trunk by default _now_? Otherwise the amount of testing it recieves won't really increase. I hope you mean for ppc only, otherwise you're breaking a lot of ports... Bernd

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Segher Boessenkool
On Tue, Jul 12, 2016 at 12:29:04PM +0200, Richard Biener wrote: > Instead of adding more configury can we please enable LRA on trunk by default > _now_? Otherwise the amount of testing it recieves won't really increase. I agree we should change default_lra_p to return true instead of false.

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-12 Thread Richard Biener
On Mon, Jul 11, 2016 at 10:07 PM, Michael Meissner wrote: > These configuration switches will allow the PowerPC GCC developers to switch > defaults in the compiler to debug the code, before making the decision to flip > the default permanently. In the future, when

Re: [PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-11 Thread Michael Meissner
Sigh, I keep forgetting to attach the patch. 2016-07-11 Michael Meissner * doc/install.texi (Configuration): Document PowerPC specific configuration options --enable-lra and --enable-float128. * configure.ac: Add --enable-lra and

[PATCH], PowerPC support to enable -mlra and/or -mfloat128

2016-07-11 Thread Michael Meissner
These configuration switches will allow the PowerPC GCC developers to switch defaults in the compiler to debug the code, before making the decision to flip the default permanently. In the future, when the defaults have been changed, these configuration options would allow developers to go back to