This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [fortran, patch] IEEE intrinsic modules (ping)
- From: Steve Kargl <sgk at troutmask dot apl dot washington dot edu>
- To: FX <fxcoudert at gmail dot com>
- Cc: Tobias Burnus <burnus at net-b dot de>, Fortran List <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Janne Blomqvist <blomqvist dot janne at gmail dot com>
- Date: Wed, 25 Jun 2014 07:24:06 -0700
- Subject: Re: [fortran, patch] IEEE intrinsic modules (ping)
- Authentication-results: sourceware.org; auth=none
- References: <53A9C4AF dot 6020700 at net-b dot de> <20140624191947 dot GA70297 at troutmask dot apl dot washington dot edu> <DB1AFC53-68AF-495C-9083-135F73FBD0AC at gmail dot com> <20140624202327 dot GA81301 at troutmask dot apl dot washington dot edu> <4006CA91-3464-4A9A-A228-66FA6C02673B at gmail dot com> <20140624203658 dot GA81560 at troutmask dot apl dot washington dot edu> <20140624205518 dot GA81619 at troutmask dot apl dot washington dot edu> <852C0E47-A1AE-4D64-98B5-4338D2EE4CC2 at gmail dot com> <20140624215016 dot GA81800 at troutmask dot apl dot washington dot edu> <E04A163E-5CC2-4D58-9815-FBBA448B91D7 at gmail dot com>
On Wed, Jun 25, 2014 at 01:41:02AM +0200, FX wrote:
> > If I remove the previously installed gcc, the failure again occurs.
> > So, it looks like the testsuite is picking up installed *.mod files
> > over the freshly built *.mod files. This is not a showstopper.
>
> And this is not all the testsuite, but only ieee_1.F90. After a long
> chase, it turns out it?s because it has a dg-options, which overrides
> my IEEE options in ieee.exp. This is because I used dg-options when I
> should have used dg-additional-options (phew!).
Testcase now passes.
>
> > I also read through the config/fpu-sysv.h diff. It looks correct
> > to me. I do note that I suspect that there is a problem in
> > config/fpu-sysv.h.
>
> I concur with your diagnosis. I?ve fixed this, and full new patch
> is attached (including regenerated files).
>
> > Both Janne and Tobias have stated that they took a quick glance
> > over the patch, and with my testing I think you should commit.
> > Although you'll probably need to deal with other odd architectures.
>
> I?ll wait a few more days to commit, so others can comment/review
> and I am sure to be around if there is fallout.
>
I tested the new diff and ieee_1.F90 on x86_64-*-freebsd.
'gmake check-gfortran' passed without any new issues.
Thanks for taking on the IEEE features of Fortran.
--
Steve