Re: Remove -fshort-double (PR60410)

2016-02-05 Thread Jeff Law
On 02/05/2016 12:31 PM, Bernd Schmidt wrote: This patch fixes PR60410 by removing -fshort-double. Nick earlier propsed a fix for the crash, but Richard B suggested removing the option entirely, and I'd agree with that. It's a pointless ABI-changing option on most targets, and if a port really nee

RE: Remove -fshort-double (PR60410)

2016-02-06 Thread Matthew Fortune
Jeff Law writes: > On 02/05/2016 12:31 PM, Bernd Schmidt wrote: > > This patch fixes PR60410 by removing -fshort-double. Nick earlier > > propsed a fix for the crash, but Richard B suggested removing the option > > entirely, and I'd agree with that. It's a pointless ABI-changing option > > on most

Re: Remove -fshort-double (PR60410)

2016-02-07 Thread Joseph Myers
On Fri, 5 Feb 2016, Jeff Law wrote: > Shouldn't c-opt.def be changed to note the option is deprecated and ignored > rather than totally removing it? No. That's appropriate with optimization options which can safely be ignored, not with options such as this that have an incompatible effect on A

Re: Remove -fshort-double (PR60410)

2016-02-07 Thread Jeff Law
On 02/07/2016 07:30 AM, Joseph Myers wrote: On Fri, 5 Feb 2016, Jeff Law wrote: Shouldn't c-opt.def be changed to note the option is deprecated and ignored rather than totally removing it? No. That's appropriate with optimization options which can safely be ignored, not with options such as