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

2021-07-27 Thread Tobias Burnus
Hi Harald, On 26.07.21 23:55, Harald Anlauf wrote: I've updated this for ALLOCATE/DEALLOCATE and STAT/ERRMSG, see attached patch. This required updating the error messages of two existing files in the testsuite. Thanks. Also affected: Some I/O items, a bunch of other stat=%v and errmsg=%v. W

Re: [r12-2511 Regression] FAIL: gfortran.dg/PR93963.f90 -Os execution test on Linux/x86_64

2021-07-27 Thread Tobias Burnus
The automatic regression test of Sunil wrote: On 26.07.21 19:27, sunil.k.pandey wrote: commit 0cbf03689e3e7d9d6002b8e5d159ef3716d0404c PR fortran/93308/93963/94327/94331/97046 problems raised by descriptor handling caused FAIL: gfortran.dg/PR93963.f90 -O2 execution test ... (That's on

Re: [PATCH v2, Fortran] TS 29113 testsuite

2021-07-27 Thread Tobias Burnus
Hi Sandra, hi Thomas, hi all, @Thomas K: Comments about the following - and of course to the testsuite itself - are highly welcome. In my opinion, the testsuite LGTM and can be committed. @Sandra: - Thoughts on the directory name? (cf. below) - Give others/Thomas a chance to comment on this,

Re: [PATCH, Fortran] [PR libfortran/101310] Bind(c): Fix bugs in CFI_section

2021-07-27 Thread Tobias Burnus
On 18.07.21 06:36, Sandra Loosemore wrote: This patch fixes bugs I observed in tests for the CFI_section function -- it turns out both the function and test cases had bugs. :-( The bugs in CFI_section itself had to do with incorrect computation of the base address for the result descriptor, plus

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

2021-07-27 Thread Harald Anlauf via Fortran
Hi Tobias, > > We should rather open a separate PR on auditing the related uses > > of gfc_match. > > I concur – I just wanted to quickly check how many %v are there – > besides %v, there are also direct calls to gfc_match_variable. > > Can you open a PR? this is now https://gcc.gnu.org/bugzill

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

2021-07-27 Thread Sandra Loosemore
On 7/26/21 2:13 PM, Sandra Loosemore wrote: On 7/26/21 3:45 AM, Tobias Burnus wrote: [snip] PS: Still, it would be nice if the proper multi-lib ISO*.h could be found; while it usually does not matter, it could do so in some cases. I think I ought to fix this now instead of just sweeping it