* [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64
@ 2022-09-05 23:54 haochen.jiang
2022-09-06 7:49 ` Jakub Jelinek
0 siblings, 1 reply; 4+ messages in thread
From: haochen.jiang @ 2022-09-05 23:54 UTC (permalink / raw)
To: haochen.jiang, gcc-patches, gcc-regression, tobias
On Linux/x86_64,
938cda536019cd6a1bc0dd2346381185b420bbf8 is the first bad commit
commit 938cda536019cd6a1bc0dd2346381185b420bbf8
Author: Tobias Burnus <tobias@codesourcery.com>
Date: Mon Sep 5 18:05:24 2022 +0200
Fortran/openmp: Partial OpenMP 5.2 doacross and omp_cur_iteration support
caused
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 28)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 42)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 48)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 55)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 70)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 82)
FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors)
with GCC configured with
../../gcc/configure --prefix=/export/users/haochenj/src/gcc-bisect/master/master/r13-2446/usr --enable-clocale=gnu --with-system-zlib --with-demangler-in-ld --with-fpmath=sse --enable-languages=c,c++,fortran --enable-cet --without-isl --enable-libmpx x86_64-linux --disable-bootstrap
To reproduce:
$ cd {build_dir}/gcc && make check RUNTESTFLAGS="gomp.exp=gfortran.dg/gomp/doacross-5.f90 --target_board='unix{-m32}'"
$ cd {build_dir}/gcc && make check RUNTESTFLAGS="gomp.exp=gfortran.dg/gomp/doacross-5.f90 --target_board='unix{-m32\ -march=cascadelake}'"
$ cd {build_dir}/gcc && make check RUNTESTFLAGS="gomp.exp=gfortran.dg/gomp/doacross-5.f90 --target_board='unix{-m64}'"
$ cd {build_dir}/gcc && make check RUNTESTFLAGS="gomp.exp=gfortran.dg/gomp/doacross-5.f90 --target_board='unix{-m64\ -march=cascadelake}'"
(Please do not reply to this email, for question about this report, contact me at haochen dot jiang at intel.com)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64
2022-09-05 23:54 [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64 haochen.jiang
@ 2022-09-06 7:49 ` Jakub Jelinek
2022-09-06 8:12 ` Tobias Burnus
0 siblings, 1 reply; 4+ messages in thread
From: Jakub Jelinek @ 2022-09-06 7:49 UTC (permalink / raw)
To: Tobias Burnus; +Cc: gcc-patches
On Tue, Sep 06, 2022 at 07:54:05AM +0800, haochen.jiang via Gcc-patches wrote:
> On Linux/x86_64,
>
> 938cda536019cd6a1bc0dd2346381185b420bbf8 is the first bad commit
> commit 938cda536019cd6a1bc0dd2346381185b420bbf8
> Author: Tobias Burnus <tobias@codesourcery.com>
> Date: Mon Sep 5 18:05:24 2022 +0200
>
> Fortran/openmp: Partial OpenMP 5.2 doacross and omp_cur_iteration support
>
> caused
>
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 28)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 42)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 48)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 55)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 70)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 82)
> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors)
Seems this is still being diagnosed in the FE when it should be deferred to
the middle-end now (only when we find out if it is doacross or not).
If you really want to diagnose it in the FE (and do the doacross discovery
there too), I think the testcase needs to be split into 2, because the FE
errors apparently result in the gimplifier or omp lowering not being done
at all.
Jakub
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64
2022-09-06 7:49 ` Jakub Jelinek
@ 2022-09-06 8:12 ` Tobias Burnus
2022-09-06 8:24 ` Jakub Jelinek
0 siblings, 1 reply; 4+ messages in thread
From: Tobias Burnus @ 2022-09-06 8:12 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 1045 bytes --]
On 06.09.22 09:49, Jakub Jelinek wrote:
> On Tue, Sep 06, 2022 at 07:54:05AM +0800, haochen.jiang via Gcc-patches wrote:
>> Fortran/openmp: Partial OpenMP 5.2 doacross and omp_cur_iteration support
>> caused
>> FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 28)
> Seems this is still being diagnosed in the FE when it should be deferred to
> the middle-end now (only when we find out if it is doacross or not).
I concur – and at some point during patch writing, I had this snippeted
removed (given that the testcase required it) – but for some reasons, it
got mangled in again. I have now removed it again/in GCC mainline –
after successful regtesting.
Committed as r13-2491-gd6582c662ca0da05c74fa3183e1bc2cadcb21424
Tobias
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955
[-- Attachment #2: committed.diff --]
[-- Type: text/x-patch, Size: 1384 bytes --]
commit d6582c662ca0da05c74fa3183e1bc2cadcb21424
Author: Tobias Burnus <tobias@codesourcery.com>
Date: Tue Sep 6 10:02:13 2022 +0200
Fix Fortran/openmp: Partial OpenMP 5.2 doacross
This removed a checking snippet which accidentally was left in in commit
r13-2446-g938cda536019cd6a1bc0dd2346381185b420bbf8 ; this caused
fails in gfortran.dg/gomp/doacross-5.f90 (added in that very commit).
Note that a similar but refined check is now done in the middle end.
(The ME version additionally checks whether doacross is present.)
gcc/fortran/
* openmp.cc (resolve_omp_clauses): Remove ordered/linear
check as it is handled now in the middle end.
---
gcc/fortran/openmp.cc | 4 ----
1 file changed, 4 deletions(-)
diff --git a/gcc/fortran/openmp.cc b/gcc/fortran/openmp.cc
index 5142fd7c608..457e983663b 100644
--- a/gcc/fortran/openmp.cc
+++ b/gcc/fortran/openmp.cc
@@ -7625,10 +7625,6 @@ resolve_omp_clauses (gfc_code *code, gfc_omp_clauses *omp_clauses,
linear_op = n->u.linear.op;
}
}
- else if (omp_clauses->orderedc)
- gfc_error ("LINEAR clause specified together with "
- "ORDERED clause with argument at %L",
- &n->where);
else if (n->u.linear.op != OMP_LINEAR_REF
&& n->sym->ts.type != BT_INTEGER)
gfc_error ("LINEAR variable %qs must be INTEGER "
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64
2022-09-06 8:12 ` Tobias Burnus
@ 2022-09-06 8:24 ` Jakub Jelinek
0 siblings, 0 replies; 4+ messages in thread
From: Jakub Jelinek @ 2022-09-06 8:24 UTC (permalink / raw)
To: Tobias Burnus; +Cc: gcc-patches
On Tue, Sep 06, 2022 at 10:12:39AM +0200, Tobias Burnus wrote:
> On 06.09.22 09:49, Jakub Jelinek wrote:
> > On Tue, Sep 06, 2022 at 07:54:05AM +0800, haochen.jiang via Gcc-patches wrote:
> > > Fortran/openmp: Partial OpenMP 5.2 doacross and omp_cur_iteration support
> > > caused
> > > FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for errors, line 28)
> > Seems this is still being diagnosed in the FE when it should be deferred to
> > the middle-end now (only when we find out if it is doacross or not).
>
> I concur – and at some point during patch writing, I had this snippeted
> removed (given that the testcase required it) – but for some reasons, it
> got mangled in again. I have now removed it again/in GCC mainline –
> after successful regtesting.
>
> Committed as r13-2491-gd6582c662ca0da05c74fa3183e1bc2cadcb21424
Thanks.
Jakub
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-09-06 8:25 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-05 23:54 [r13-2446 Regression] FAIL: gfortran.dg/gomp/doacross-5.f90 -O (test for excess errors) on Linux/x86_64 haochen.jiang
2022-09-06 7:49 ` Jakub Jelinek
2022-09-06 8:12 ` Tobias Burnus
2022-09-06 8:24 ` Jakub Jelinek
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).