Bug 89639 - FAIL: gfortran.dg/ieee/ieee_9.f90 -O0 (test for excess errors)
Summary: FAIL: gfortran.dg/ieee/ieee_9.f90 -O0 (test for excess errors)
Status: RESOLVED FIXED
Alias: None
Product: gcc
Classification: Unclassified
Component: fortran (show other bugs)
Version: 9.0
: P4 normal
Target Milestone: 12.0
Assignee: Francois-Xavier Coudert
URL:
Keywords: testsuite-fail
Depends on:
Blocks:
 
Reported: 2019-03-09 00:17 UTC by John David Anglin
Modified: 2022-01-07 06:38 UTC (History)
1 user (show)

See Also:
Host: hppa-unknown-linux-gnu
Target: hppa-unknown-linux-gnu
Build: hppa-unknown-linux-gnu
Known to work:
Known to fail:
Last reconfirmed: 2019-03-10 00:00:00


Attachments
adjust testcase (685 bytes, patch)
2021-12-29 17:26 UTC, Francois-Xavier Coudert
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John David Anglin 2019-03-09 00:17:08 UTC
spawn -ignore SIGHUP /home/dave/gnu/gcc/objdir/gcc/testsuite/gfortran2/../../gfo
rtran -B/home/dave/gnu/gcc/objdir/gcc/testsuite/gfortran2/../../ -B/home/dave/gn
u/gcc/objdir/hppa-linux-gnu/./libgfortran/ /home/dave/gnu/gcc/gcc/gcc/testsuite/
gfortran.dg/ieee/ieee_9.f90 -fno-diagnostics-show-caret -fno-diagnostics-show-li
ne-numbers -fdiagnostics-color=never -O0 -pedantic-errors -fintrinsic-modules-pa
th /home/dave/gnu/gcc/objdir/hppa-linux-gnu/./libgfortran/ -fno-unsafe-math-opti
mizations -frounding-math -fsignaling-nans -B/home/dave/gnu/gcc/objdir/hppa-linu
x-gnu/./libgfortran/.libs -L/home/dave/gnu/gcc/objdir/hppa-linux-gnu/./libgfortr
an/.libs -L/home/dave/gnu/gcc/objdir/hppa-linux-gnu/./libgfortran/.libs -L/home/
dave/gnu/gcc/objdir/hppa-linux-gnu/./libatomic/.libs -lm -o ./ieee_9.exe
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:20:36: Error: I
nvalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:35:36: Error: I
nvalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:50:36: Error: I
nvalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:65:36: Error: I
nvalid kind for REAL at (1)
compiler exited with status 1
FAIL: gfortran.dg/ieee/ieee_9.f90   -O0  (test for excess errors)
Excess errors:
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:20:36: Error: Invalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:35:36: Error: Invalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:50:36: Error: Invalid kind for REAL at (1)
/home/dave/gnu/gcc/gcc/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90:65:36: Error: Invalid kind for REAL at (1)

UNRESOLVED: gfortran.dg/ieee/ieee_9.f90   -O0  compilation failed to produce executable
Comment 1 Steve Kargl 2019-03-09 01:26:28 UTC
Just XFAIL the testcase.
Comment 2 dave.anglin 2019-03-09 16:12:23 UTC
On 2019-03-08 8:26 p.m., kargl at gcc dot gnu.org wrote:
> Just XFAIL the testcase.
I'm going to skip test as it still fails when XFAIL'd as on arm.  The test fails
due to compile error.
Comment 3 John David Anglin 2019-03-09 16:32:17 UTC
Author: danglin
Date: Sat Mar  9 16:31:46 2019
New Revision: 269527

URL: https://gcc.gnu.org/viewcvs?rev=269527&root=gcc&view=rev
Log:
	PR fortran/89639
	* gfortran.dg/ieee/ieee_9.f90: Skip on hppa*-*-linux*.


Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/gfortran.dg/ieee/ieee_9.f90
Comment 4 Steve Kargl 2019-03-09 17:43:27 UTC
On Sat, Mar 09, 2019 at 04:12:23PM +0000, dave.anglin at bell dot net wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89639
> 
> --- Comment #2 from dave.anglin at bell dot net ---
> On 2019-03-08 8:26 p.m., kargl at gcc dot gnu.org wrote:
> > Just XFAIL the testcase.
> I'm going to skip test as it still fails when XFAIL'd as on arm.  The test
> fails
> due to compile error.
> 

Thanks.  When I wrote the test, I tried to skip the
REAL128 case with the "if (real128 >) then" ... 
blocks, and completely overlooked that real128 is 
a named constant.  This means the conversion functions
real(..., real128) will try ti real128 = -1, which 
of fails.
Comment 5 dave.anglin 2019-03-09 17:50:43 UTC
On 2019-03-09 12:43 p.m., sgk at troutmask dot apl.washington.edu wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89639
>
> --- Comment #4 from Steve Kargl <sgk at troutmask dot apl.washington.edu> ---
> On Sat, Mar 09, 2019 at 04:12:23PM +0000, dave.anglin at bell dot net wrote:
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89639
>>
>> --- Comment #2 from dave.anglin at bell dot net ---
>> On 2019-03-08 8:26 p.m., kargl at gcc dot gnu.org wrote:
>>> Just XFAIL the testcase.
>> I'm going to skip test as it still fails when XFAIL'd as on arm.  The test
>> fails
>> due to compile error.
>>
> Thanks.  When I wrote the test, I tried to skip the
> REAL128 case with the "if (real128 >) then" ... 
> blocks, and completely overlooked that real128 is 
> a named constant.  This means the conversion functions
> real(..., real128) will try ti real128 = -1, which 
> of fails.
Now I understand why this just fails on hppa-linux.  hppa-hpux has REAL128.  I probably
should change skip reason to "No REAL128".

Dave
Comment 6 Steve Kargl 2019-03-09 18:53:37 UTC
On Sat, Mar 09, 2019 at 05:50:43PM +0000, dave.anglin at bell dot net wrote:
> >>
> > Thanks.  When I wrote the test, I tried to skip the
> > REAL128 case with the "if (real128 >) then" ... 
> > blocks, and completely overlooked that real128 is 
> > a named constant.  This means the conversion functions
> > real(..., real128) will try ti real128 = -1, which 

s/will try ti/will try to use

> > of fails.
> Now I understand why this just fails on hppa-linux.i
>  hppa-hpux has REAL128.  I probably should change skip
> reason to "No REAL128".
> 

I added predefine macros for cpp.  I'll someday move
this testcase to ieee_9.F90, and use #if __GFC_REAL16__
to guard againt the issue.  "Someday" may not be too
soon.
Comment 7 Dominique d'Humieres 2019-03-10 14:15:49 UTC
Is it fixed by r269527 or not?
Comment 8 dave.anglin 2019-03-10 14:33:34 UTC
On 2019-03-10 10:15 a.m., dominiq at lps dot ens.fr wrote:
> Is it fixed by r269527 or not?
It's not a good fix.  r269527 simply avoids the problem on hppa-linux.
Comment 9 Dominique d'Humieres 2019-03-10 14:45:13 UTC
> > Is it fixed by r269527 or not?
>
> It's not a good fix.  r269527 simply avoids the problem on hppa-linux.

So let move the status to NEW.
Comment 10 Francois-Xavier Coudert 2021-12-29 17:26:28 UTC
Created attachment 52086 [details]
adjust testcase

David, could you kindly test the attached patch, to see if it fixes things?
Comment 11 dave.anglin 2021-12-29 18:03:16 UTC
On 2021-12-29 12:26 p.m., fxcoudert at gcc dot gnu.org wrote:
> David, could you kindly test the attached patch, to see if it fixes things?
Added patch to my build tree.
Comment 12 dave.anglin 2021-12-31 20:08:46 UTC
On 2021-12-29 12:26 p.m., fxcoudert at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89639
>
> --- Comment #10 from Francois-Xavier Coudert <fxcoudert at gcc dot gnu.org> ---
> Created attachment 52086 [details]
>    --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=52086&action=edit
> adjust testcase
>
> David, could you kindly test the attached patch, to see if it fixes things?
Tests pass with patch on hppa-unknown-linux-gnu.
Comment 13 Francois-Xavier Coudert 2021-12-31 22:23:59 UTC
Fixed at cb48166e52c0f159eb80a0666c4847825e294ec0