[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-06 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 Eric Botcazou changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-06 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #17 from Eric Botcazou --- Author: ebotcazou Date: Fri Dec 6 11:31:56 2013 New Revision: 205735 URL: http://gcc.gnu.org/viewcvs?rev=205735&root=gcc&view=rev Log: PR target/59316 * config/sparc/sparc.h (SPARC_LOW_FE_EXCEPT_VAL

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-03 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #16 from Eric Botcazou --- > Properly it should cause traps if those are enabled (although enabling > traps is outside the scope of ISO C, and my guess is that when TS 18661-5 > provides C bindings for IEEE 754-2008 alternate excepti

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-03 Thread joseph at codesourcery dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #15 from joseph at codesourcery dot com --- On Tue, 3 Dec 2013, ebotcazou at gcc dot gnu.org wrote: > > In any case, c11-atomic-exec-5.c does not test anything relating to enabled > > traps, although the feholdexcept code sequence fro

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-03 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #14 from Eric Botcazou --- > Hopefully the other OSes use one of these two settings. At least FreeBSD uses the Linux setting.

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-03 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #13 from Eric Botcazou --- > x86/x86-64 now have their own fenv.c file that defines the FE_* macros > itself and so is immune to that issue. I was hoping that generally the > macros would be defined to correspond to the appropriate

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-02 Thread joseph at codesourcery dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #12 from joseph at codesourcery dot com --- One possibility would be to change libatomic/fenv.c to include a local atomic-fenv.h (for example) header instead of (which would no longer need a configure check), so that if the generic

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-02 Thread joseph at codesourcery dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #11 from joseph at codesourcery dot com --- On Mon, 2 Dec 2013, ebotcazou at gcc dot gnu.org wrote: > Ugh. Linux and Solaris disagree on the values of the FE_* macros so we will > need to have OS-dependent code in the sparc_atomic_as

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-02 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #10 from Eric Botcazou --- > Fixing on SPARC. Ugh. Linux and Solaris disagree on the values of the FE_* macros so we will need to have OS-dependent code in the sparc_atomic_assign_expand_fenv hook if we call __atomic_feraiseexcept (I

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-12-01 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 Eric Botcazou changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-11-28 Thread joseph at codesourcery dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #8 from joseph at codesourcery dot com --- It fails everywhere that (a) supports floating-point exceptions, (b) has not had the target hook added and (c) supports pthreads ((c) is a requirement for the test to run - the feature in qu

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-11-28 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 Eric Botcazou changed: What|Removed |Added Target|sparc*-sun-solaris2.*, |sparc*-sun-solaris2.* |

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-11-28 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 --- Comment #6 from Eric Botcazou --- > Btw, for the release I consider this a testsuite bug (the test should be > disabled/skipped for archs not supporting the feature). Yes, but only immediately before the release, as that's a good reminder.

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-11-28 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 Richard Biener changed: What|Removed |Added Severity|normal |enhancement --- Comment #5 from Richard

[Bug target/59316] gcc.dg/atomic/c11-atomic-exec-5.c FAILs on Solaris/SPARC

2013-11-27 Thread ebotcazou at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59316 Eric Botcazou changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|