public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask
@ 2011-05-12 16:16 nicolai.stange at zmaw dot de
  2011-05-13 15:42 ` [Bug libfortran/48982] " nicolai.stange at zmaw dot de
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: nicolai.stange at zmaw dot de @ 2011-05-12 16:16 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

           Summary: libgfortran on sysv never uses fpsetmask
           Product: gcc
           Version: 4.5.3
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libfortran
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: nicolai.stange@zmaw.de


I'm working on Solaris10/SPARC.

nic@nic-desktop:~/gcc-4.5.3$ grep -r have_fpsetmask . | grep -v libgfor_
./libgfortran/configure.host:if test "x${have_fpsetmask}" = "xyes"; then
nic@nic-desktop:~/gcc-4.5.3$

have_fpsetmask won't ever be set anywhere.
libgfor_cv_have_fpsetmask ist set and should be tested instead.

This results in set_fpu coming from fpu-generic.h.
Compiling code w. -ffpe-trap=invalid,zero,overflow results in the following
warnings when executing the code:
Fortran runtime warning: IEEE 'invalid operation' exception not supported.
Fortran runtime warning: IEEE 'division by zero' exception not supported.
Fortran runtime warning: IEEE 'overflow' exception not supported.

Best regards

Nicolai


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
@ 2011-05-13 15:42 ` nicolai.stange at zmaw dot de
  2011-05-20 13:52 ` jb at gcc dot gnu.org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: nicolai.stange at zmaw dot de @ 2011-05-13 15:42 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

--- Comment #1 from Nicolai Stange <nicolai.stange at zmaw dot de> 2011-05-13 14:44:08 UTC ---
Created attachment 24240
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=24240
Against 4.5.2

The remove and add of the comment line in fpu-sysv.h within this patch is
intentional: It seems to contain stray '\303' and gcc complains about this.


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
  2011-05-13 15:42 ` [Bug libfortran/48982] " nicolai.stange at zmaw dot de
@ 2011-05-20 13:52 ` jb at gcc dot gnu.org
  2011-05-20 14:02 ` jb at gcc dot gnu.org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: jb at gcc dot gnu.org @ 2011-05-20 13:52 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jb at gcc dot gnu.org

--- Comment #2 from Janne Blomqvist <jb at gcc dot gnu.org> 2011-05-20 13:32:21 UTC ---
Are you sure you need the 

#include <ieeefp.h>

in fpu-sysv.h? It should be included already in libgfortran.h, which is
included in runtime/fpu.c before fpu-target.h.


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
  2011-05-13 15:42 ` [Bug libfortran/48982] " nicolai.stange at zmaw dot de
  2011-05-20 13:52 ` jb at gcc dot gnu.org
@ 2011-05-20 14:02 ` jb at gcc dot gnu.org
  2011-05-20 14:17 ` jb at gcc dot gnu.org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: jb at gcc dot gnu.org @ 2011-05-20 14:02 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

--- Comment #3 from Janne Blomqvist <jb at gcc dot gnu.org> 2011-05-20 13:40:47 UTC ---
Actually, in the version control log for acinclude.m4 we have

commit b00fa2cfbfd81cbdc31d5a44e4ce8140cc153776
Author: fxcoudert <fxcoudert@138bc75d-0d04-0410-961f-82ee72b054a4>
Date:   Fri Feb 25 17:56:04 2011 +0000

        * acinclude.m4 (LIBGFOR_CHECK_FPSETMASK): Set shell variable
        tested in configure.host.
        * configure: Regenerate.


    git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@170500
138bc75d-0d04-0410-961f-82ee72b054a4

And indeed the test is now

dnl Checks for fpsetmask function.
AC_DEFUN([LIBGFOR_CHECK_FPSETMASK], [
  AC_CACHE_CHECK([whether fpsetmask is present], libgfor_cv_have_fpsetmask, [
    AC_LINK_IFELSE([AC_LANG_PROGRAM([[
#if HAVE_FLOATINGPOINT_H
# include <floatingpoint.h>
#endif /* HAVE_FLOATINGPOINT_H */
#if HAVE_IEEEFP_H
# include <ieeefp.h>
#endif /* HAVE_IEEEFP_H */]],[[fpsetmask(0);]])],
    eval "libgfor_cv_have_fpsetmask=yes", eval "libgfor_cv_have_fpsetmask=no")
  ])
  if test x"$libgfor_cv_have_fpsetmask" = xyes; then
    have_fpsetmask=yes
    AC_DEFINE(HAVE_FPSETMASK, 1, [Define if you have fpsetmask.])
  fi
])


Can you test whether this works on 4.6?


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
                   ` (2 preceding siblings ...)
  2011-05-20 14:02 ` jb at gcc dot gnu.org
@ 2011-05-20 14:17 ` jb at gcc dot gnu.org
  2011-05-20 14:43 ` nicolai.stange at zmaw dot de
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: jb at gcc dot gnu.org @ 2011-05-20 14:17 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2011.05.20 13:41:33
     Ever Confirmed|0                           |1


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
                   ` (3 preceding siblings ...)
  2011-05-20 14:17 ` jb at gcc dot gnu.org
@ 2011-05-20 14:43 ` nicolai.stange at zmaw dot de
  2011-05-20 14:45 ` nicolai.stange at zmaw dot de
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: nicolai.stange at zmaw dot de @ 2011-05-20 14:43 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

--- Comment #4 from Nicolai Stange <nicolai.stange at zmaw dot de> 2011-05-20 14:14:05 UTC ---
(In reply to comment #2)
> Are you sure you need the 
> 
> #include <ieeefp.h>
Definitely, i've got compile errors when it isn't there.


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
                   ` (4 preceding siblings ...)
  2011-05-20 14:43 ` nicolai.stange at zmaw dot de
@ 2011-05-20 14:45 ` nicolai.stange at zmaw dot de
  2011-05-28 16:28 ` nicolai.stange at zmaw dot de
  2011-05-29  9:03 ` jb at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: nicolai.stange at zmaw dot de @ 2011-05-20 14:45 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

--- Comment #5 from Nicolai Stange <nicolai.stange at zmaw dot de> 2011-05-20 14:15:44 UTC ---
(In reply to comment #3)
> Can you test whether this works on 4.6?
Sure, but this might take some time.


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
                   ` (5 preceding siblings ...)
  2011-05-20 14:45 ` nicolai.stange at zmaw dot de
@ 2011-05-28 16:28 ` nicolai.stange at zmaw dot de
  2011-05-29  9:03 ` jb at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: nicolai.stange at zmaw dot de @ 2011-05-28 16:28 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

--- Comment #6 from Nicolai Stange <nicolai.stange at zmaw dot de> 2011-05-28 15:48:41 UTC ---
> Can you test whether this works on 4.6?
Works like a charm with gcc-4.6.0 release (without any diffs applied).


^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Bug libfortran/48982] libgfortran on sysv never uses fpsetmask
  2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
                   ` (6 preceding siblings ...)
  2011-05-28 16:28 ` nicolai.stange at zmaw dot de
@ 2011-05-29  9:03 ` jb at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: jb at gcc dot gnu.org @ 2011-05-29  9:03 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48982

Janne Blomqvist <jb at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|                            |FIXED

--- Comment #7 from Janne Blomqvist <jb at gcc dot gnu.org> 2011-05-29 09:01:49 UTC ---
Thanks for verifying, closing as fixed.


^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2011-05-29  9:03 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-12 16:16 [Bug libfortran/48982] New: libgfortran on sysv never uses fpsetmask nicolai.stange at zmaw dot de
2011-05-13 15:42 ` [Bug libfortran/48982] " nicolai.stange at zmaw dot de
2011-05-20 13:52 ` jb at gcc dot gnu.org
2011-05-20 14:02 ` jb at gcc dot gnu.org
2011-05-20 14:17 ` jb at gcc dot gnu.org
2011-05-20 14:43 ` nicolai.stange at zmaw dot de
2011-05-20 14:45 ` nicolai.stange at zmaw dot de
2011-05-28 16:28 ` nicolai.stange at zmaw dot de
2011-05-29  9:03 ` jb at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).