[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 --- Comment #20 from M.Schewe --- Removing -fshort-double completely was not a good idea. It is required for ARM microcontrollers, e.g. ARM Cortex M4F with a FPU which can handly only single precision arithmetic operations. All double precision arithmetics must be emulated by slow integer software. Without this option it is no more possible to compile runtime libraries and application to efficiently use this FPU. Restricing resolution to single precision is acceptable for many embedded realtime applications, often runtime requirements must be met. I agree to you that on architectures with powerful FPUs like x86 and x86_64 this option is not so important and may be unsupported to solve bugs. I suggest to revert the changes from rev. 233218 thus implementing -fshort-double as before. Architectures which have implementation-bugs like x86_64-*-* simply can abort compilation if the option is activated with some error message saying "-fshort-double option is no more supported on this archeticture".
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Andrew Pinski changed: What|Removed |Added CC||der.herr at hofr dot at --- Comment #19 from Andrew Pinski --- *** Bug 70334 has been marked as a duplicate of this bug. ***
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Andrew Pinski changed: What|Removed |Added Target Milestone|4.9.4 |6.0
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Bernd Schmidt changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #18 from Bernd Schmidt --- Fixed on trunk, and I'm guessing not really important enough to backport.
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 --- Comment #17 from Bernd Schmidt --- Author: bernds Date: Mon Feb 8 15:36:16 2016 New Revision: 233218 URL: https://gcc.gnu.org/viewcvs?rev=233218&root=gcc&view=rev Log: Remove -fshort-double PR target/60410 * tree.c (build_common_tree_nodes): Remove short_double argument. All callers changed. * tree.h (build_common_tree_nodes): Adjust declaration. * doc/invoke.texi (-fshort-double): Remove documentation. * config/mips/t-img-elf (MULTILIB_OPTIONS, MULTILIB_DIRNAMES, MULTILIB_EXCEPTIONS): Remove -fshort-double variant. * lto-wrapper.c (merge_and_complain, append_compiler_options, append_linker_options): Don't handle OPT_fshort_double. c-family/ PR target/60410 * c.opt (fshort-double): Remove. testsuite/ PR target/60410 * gcc.dg/lto/pr55113_0.c: Remove test. Removed: trunk/gcc/testsuite/gcc.dg/lto/pr55113_0.c Modified: trunk/gcc/ChangeLog trunk/gcc/ada/gcc-interface/misc.c trunk/gcc/c-family/ChangeLog trunk/gcc/c-family/c-common.c trunk/gcc/c-family/c.opt trunk/gcc/config/mips/t-img-elf trunk/gcc/doc/invoke.texi trunk/gcc/fortran/f95-lang.c trunk/gcc/go/go-lang.c trunk/gcc/java/decl.c trunk/gcc/lto-wrapper.c trunk/gcc/lto/lto-lang.c trunk/gcc/testsuite/ChangeLog trunk/gcc/tree.c trunk/gcc/tree.h
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Bernd Schmidt changed: What|Removed |Added CC||bernds at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |bernds at gcc dot gnu.org --- Comment #16 from Bernd Schmidt --- Will post patch to remove -fshort-double.
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 --- Comment #15 from Nick Clifton --- Sorry I meant: I am not sure of the best way to proceed.
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Nick Clifton changed: What|Removed |Added CC||nickc at redhat dot com --- Comment #14 from Nick Clifton --- Created attachment 37275 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=37275&action=edit Use SFmode variant of DFmode builtin vector types when -fshort-doubles is enabled. Hi Guys, I recently created and posted a patch for PR 55113 which also fixes this PR. (Patch attached here for easy reference). One reviewer however has suggested that a better solution would be to disable -fshort-double option entirely: https://gcc.gnu.org/ml/gcc-patches/2015-12/msg01900.html I am not of the best way to proceed. Any suggestions ? Cheers Nick
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 Jakub Jelinek changed: What|Removed |Added Target Milestone|4.9.3 |4.9.4
[Bug target/60410] [4.9/5/6 Regression] -fshort-double ICEs x86_64
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60410 --- Comment #13 from Jakub Jelinek --- GCC 4.9.3 has been released.
