Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Andi Kleen
> So - no, you can't simply remove it. But IMHO it should become a > target-specific flag. How about a patch to just disable it for x86? -Andi

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Richard Biener
On September 29, 2014 5:21:36 PM CEST, Jakub Jelinek wrote: >On Mon, Sep 29, 2014 at 08:20:03AM -0700, Andi Kleen wrote: >> > As we saw LTO fixes for -fshort-double it's clear that this flag >_is_ used >> > for some embedded archs. >> >> Did we? It has been ICEing since 4.5, which is before LTO.

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Jakub Jelinek
On Mon, Sep 29, 2014 at 08:20:03AM -0700, Andi Kleen wrote: > > As we saw LTO fixes for -fshort-double it's clear that this flag _is_ used > > for some embedded archs. > > Did we? It has been ICEing since 4.5, which is before LTO. Depends on which target. Aren't the ICEs i?86/x86_64 backend ICEs

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Andi Kleen
> As we saw LTO fixes for -fshort-double it's clear that this flag _is_ used > for some embedded archs. Did we? It has been ICEing since 4.5, which is before LTO. -Andi

Re: [PATCH 1/2] Remove -fshort-double

2014-09-29 Thread Richard Biener
On Sat, Sep 27, 2014 at 3:54 PM, Andi Kleen wrote: > From: Andi Kleen > > -fshort-double has crashes the compiler since 4.6 (see PR60410) > Since it's an obscure option that apparently nobody uses it the > best way to fix it seems to just remove it. > > This prevents constant ICEs when running an