This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [PATCH, fortran ieee]: PR 88678, Many gfortran.dg/ieee/ieee_X.f90 test cases fail starting with r267465
- From: Steve Ellcey <sellcey at marvell dot com>
- To: "ubizjak at gmail dot com" <ubizjak at gmail dot com>, "blomqvist dot janne at gmail dot com" <blomqvist dot janne at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, "bergner at linux dot ibm dot com" <bergner at linux dot ibm dot com>, "sgk at troutmask dot apl dot washington dot edu" <sgk at troutmask dot apl dot washington dot edu>
- Date: Thu, 7 Feb 2019 23:53:10 +0000
- Subject: Re: [PATCH, fortran ieee]: PR 88678, Many gfortran.dg/ieee/ieee_X.f90 test cases fail starting with r267465
- References: <CAFULd4Y-UWxMQvRfB67znSDAnPDZp2OWQkQVJsSNKbUmP1NdKw@mail.gmail.com>
On Thu, 2019-01-31 at 08:46 +0100, Uros Bizjak wrote:
> On Wed, Jan 30, 2019 at 9:51 PM Janne Blomqvist
>
> > This seems to change the only user of support_fpu_trap() that is
> > different from support_fpu_flag(), so with this change one could
> > remove support_fpu_trap() entirely and modify all callers (since
> > it's an internal function it's not used outside libgfortran) to
> > call support_fpu_flag() directly. Otherwise Ok.
>
> Some targets only support IEEE flags (so, flags in some FP status
> register), but not exceptions (traps) based on these flags that halt
> the program. Currently, fpu-glibc.h assumes that all flags can
> generate exceptions (that is true for the current set of gfortran
> targets), but some future target wants to return 0 from
> support_fpu_trap.
>
> Uros.
Is this actually true for all existing gfortran targets? Specifically
I am wondering about Arm and Aarch64. PR 78314 says that ARM trapping
FPU exceptions are optional.
I am currently seeing gfortran.dg/ieee/ieee_6.f90 fail on my Aarch64
ThunderX box. I wonder if we should have an Arm/Aarch64 specific
version of the fpu header file (fpu-arm.h) that would use the previous
version of support_fpu_trap.
Steve Ellcey
sellcey@marvell.com