OpenACC 'nohost' clause

2021-07-21 Thread Thomas Schwinge
Hi! On 2018-10-02T07:11:43-0700, Cesar Philippidis wrote: > Attached is a patch that introduces support for the acc routine nohost > clause. Basically, if an acc routine function is marked as nohost, then > the compiler does not generate code for the host. This is in particular useful in combina

[PATCH] PR fortrsn/101564 - ICE in resolve_allocate_deallocate, at fortran/resolve.c:8169

2021-07-21 Thread Harald Anlauf via Fortran
I have the impression that Gerhard is a hydra: one PR down, he submits two new ones... :-( Anyway, here's a straightforward fix for a NULL pointer dereference for an invalid argument to STAT. For an alternative patch by Steve see PR. Regtested on x86_64-pc-linux-gnu. OK for mainline / 11-branch

[PATCH] PR fortran/101536 - ICE in gfc_conv_expr_descriptor, at fortran/trans-array.c:7324

2021-07-21 Thread Harald Anlauf via Fortran
Another one of Gerhard's infamous testcases. We did not properly detect and reject array elements of type CLASS as argument to an intrinsic when it should be an array. Regtested on x86_64-pc-linux-gnu. OK for mainline / 11-branch when it reopens? Thanks, Harald Fortran: extend check for array

Re: [PATCH, Fortran] [PR libfortran/101317] Bind(c): Improve error checking in CFI_* functions

2021-07-21 Thread Sandra Loosemore
On 7/21/21 11:26 AM, Tobias Burnus wrote: On 17.07.21 02:49, Sandra Loosemore wrote: This patch is for PR101317, one of the bugs uncovered by the TS29113 testsuite.  Here I'd observed that CFI_establish, etc was not diagnosing some invalid-argument situations documented in the standard, altho

Re: [PATCH, Fortran] [PR libfortran/101317] Bind(c): Improve error checking in CFI_* functions

2021-07-21 Thread Tobias Burnus
On 17.07.21 02:49, Sandra Loosemore wrote: This patch is for PR101317, one of the bugs uncovered by the TS29113 testsuite. Here I'd observed that CFI_establish, etc was not diagnosing some invalid-argument situations documented in the standard, although it was properly catching others. After f

Re: [PATCH] PR fortran/101514 - ICE: out of memory allocating 18446744073709551600 bytes

2021-07-21 Thread Tobias Burnus
On 20.07.21 21:49, Harald Anlauf via Gcc-patches wrote: While investigating one of Gerhard's latest bug reports, which was almost obvious to fix after a hint by Richard Biener, I found further variants of valid and invalid code that lead to either NULL pointer dereferences or similar OOM situati

Re: [PATCH 3/3] [PR libfortran/101305] Fix ISO_Fortran_binding.h paths in gfortran testsuite

2021-07-21 Thread Tobias Burnus
On 13.07.21 23:28, Sandra Loosemore wrote: ISO_Fortran_binding.h is now generated in the libgfortran build directory where it is on the default include path. Adjust includes in the gfortran testsuite not to include an explicit path pointing at the source directory. ... -#include "../../../libg

Re: [PATCH 1/3] [PR libfortran/101305] Bind(C): Fix type encodings in ISO_Fortran_binding.h

2021-07-21 Thread Tobias Burnus
On 13.07.21 23:28, Sandra Loosemore wrote: ISO_Fortran_binding.h had many incorrect hardwired kind encodings in the definitions of the CFI_type_* macros. Additionally, not all targets support all the defined type encodings, and the Fortran standard requires those macros to have a negative value

Re: [PATCH 2/3] [PR libfortran/101305] Bind(C): Correct sizes of some types in CFI_establish

2021-07-21 Thread Tobias Burnus
On 13.07.21 23:28, Sandra Loosemore wrote: CFI_establish was failing to set the default elem_len correctly for CFI_type_cptr, CFI_type_cfunptr, CFI_type_long_double, and CFI_type_long_double_Complex. LGTM – thanks for the patch! Tobias 2021-07-13 Sandra Loosemore libgfortran/ PR li

Re: *Ping**2 [Patch] Fortran: Fix bind(C) character length checks

2021-07-21 Thread Tobias Burnus
On 16.07.21 14:55, Jerry D wrote: Good to go Tobias. Thanks for the review! I have committed it as r12-2431-gb3d4011ba10275fbd5d6ec5a16d5aaebbdfb5d3c (+ cherry picked it to OG11). The attached and committed version has split the 'Allocatable and pointer' in two separate gfc_error and removed

Re: Pushing XFAILed test cases

2021-07-21 Thread Tobias Burnus
Hi all, hi Thomas (2x), hi Sandra, On 16.07.21 09:52, Thomas Koenig via Fortran wrote: The part of the patch to add tests for this goes on top of my base TS29113 testsuite patch, which hasn't been reviewed or committed yet. It is my understanding that it is not gcc policy to add xfailed test c

GCC 11.1.1 Status Report (2021-07-21), branch frozen for release

2021-07-21 Thread Richard Biener
Status == The GCC 11 branch is now frozen for the upcoming GCC 11.2 release. All changes require release manager approval now. Quality Data Priority # Change from last report --- --- P1 P2 260 - 12