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

[PATCH 1/2] Remove -fshort-double

2014-09-27 Thread Andi Kleen
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 gcc optimization flags autotuner. gcc/testsuite/: 2014-09-26