Re: [PATCH] Fortran: Add location info to OpenMP tree nodes

2022-04-04 Thread Richard Biener via Fortran
On Tue, Apr 5, 2022 at 6:12 AM Sandra Loosemore wrote: > > On 3/25/22 20:03, Sandra Loosemore wrote: > > I've got another patch forthcoming (stage 1 material) that adds some new > > diagnostics for non-rectangular loops during gimplification of OMP > > nodes. When I was working on that, I discove

Re: [PATCH] Fortran: Add location info to OpenMP tree nodes

2022-04-04 Thread Sandra Loosemore
On 3/25/22 20:03, Sandra Loosemore wrote: I've got another patch forthcoming (stage 1 material) that adds some new diagnostics for non-rectangular loops during gimplification of OMP nodes.  When I was working on that, I discovered that the Fortran front end wasn't attaching location information

PING Re: [PATCH] Fortran: Fix clause splitting for OMP masked taskloop directive

2022-04-04 Thread Sandra Loosemore
On 3/25/22 20:02, Sandra Loosemore wrote: I ran into this bug in the handling of clauses on the combined "masked taskloop" OMP directive when I was working on something else.  The fix turned out to be a 1-liner.  OK for trunk? Ping! This one's borderline obvious and would be good to fix in GC

Re: *Ping* [PATCH] PR fortran/104210 - [11/12 Regression] ICE in gfc_zero_size_array, at fortran/arith.cc:1685

2022-04-04 Thread Jerry D via Fortran
On 4/4/22 12:09 PM, Harald Anlauf via Fortran wrote: Am 29.03.22 um 23:41 schrieb Harald Anlauf via Fortran: Dear all, during error recovery on invalid declarations of functions as coarrays we may hit multiple places with NULL pointer dereferences. The attached patch provides a minimal and cons

Re: [PATCH] PR fortran/105138 - Bogus error when function name does not shadow an intrinsic when RESULT clause is used

2022-04-04 Thread Jerry D via Fortran
On 4/4/22 12:04 PM, Harald Anlauf via Fortran wrote: Dear all, Steve's analysis (see PR) showed that we confused the case when a symbol refererred to a recursive procedure which was named the same as an intrinsic. The standard allows such recursive references (see e.g. F2018:19.3.1). The attac

*Ping* [PATCH] PR fortran/104210 - [11/12 Regression] ICE in gfc_zero_size_array, at fortran/arith.cc:1685

2022-04-04 Thread Harald Anlauf via Fortran
Am 29.03.22 um 23:41 schrieb Harald Anlauf via Fortran: Dear all, during error recovery on invalid declarations of functions as coarrays we may hit multiple places with NULL pointer dereferences. The attached patch provides a minimal and conservative solution. Regtested on x86_64-pc-linux-gnu.

[PATCH] PR fortran/105138 - Bogus error when function name does not shadow an intrinsic when RESULT clause is used

2022-04-04 Thread Harald Anlauf via Fortran
Dear all, Steve's analysis (see PR) showed that we confused the case when a symbol refererred to a recursive procedure which was named the same as an intrinsic. The standard allows such recursive references (see e.g. F2018:19.3.1). The attached patch is based on Steve's, but handles both functio