[Fortran] OG11 backports

2021-08-19 Thread Sandra Loosemore
I've backported several patches having to do with Fortran/C interoperability from mainline to the OG11 branch. See attached log for details. -Sandra commit d554155c07771935778f557e9ef649cc3624d1ce Author: Sandra Loosemore Date: Wed Aug 11 19:24:17 2021 -0700 Fortran: Fix c_float128

Re: [PATCH] PR fortran/100950 - ICE in output_constructor_regular_field, at varasm.c:5514

2021-08-19 Thread H.J. Lu via Fortran
On Thu, Aug 19, 2021 at 12:12 PM Harald Anlauf via Gcc-patches wrote: > > Hi Tobias, > > > I am inclined to say that the Intel compiler has a bug by not > > accepting it – but as written before, I regard sub-string length > > (esp. with const expr) inquiries as an odd corner case which > > is

[PATCH] libgfortran : Use the libtool macro to determine libm availability.

2021-08-19 Thread Iain Sandoe
Hi, A while ago had a report of build failure against a Darwin branch on the latest OS release. This was because (temporarily) the symlink from libm.dylib => libSystem.dylib had been removed/omitted. libm is not needed on Darwin, and should not be added unconditionally even if that is (mostly)

Re: [PATCH] PR fortran/100950 - ICE in output_constructor_regular_field, at varasm.c:5514

2021-08-19 Thread Harald Anlauf via Fortran
Hi Tobias, > I am inclined to say that the Intel compiler has a bug by not > accepting it – but as written before, I regard sub-string length > (esp. with const expr) inquiries as an odd corner case which > is unlikely to occur in real-world code. ok. > Still does not work – or rather:

Re: [PATCH v3, Fortran] TS 29113 testsuite

2021-08-19 Thread Sandra Loosemore
On 7/27/21 5:07 AM, Tobias Burnus wrote: 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

Re: F2018 C937

2021-08-19 Thread Steve Kargl via Fortran
Arjen, If this is your first go around with patching gfortran, I'll suggest running the testsuite (if you haven't, see step 5. below; if you have see step 6.). I suspect you already know much of what I enumerate below, but it may help others. With a bug report, the workflow for me is/was 1.

Re: F2018 C937

2021-08-19 Thread Arjen Markus via Fortran
I have applied the patches locally (take care to restore the tabs ;)). Should I now commit these changes or is there a more formal procedure involved? Regards, Arjen Op do 19 aug. 2021 om 08:59 schreef Arjen Markus : > Hi Steve, > > I am willing to take up this challenge ;), as well as the

Re: [PATCH] PR fortran/100950 - ICE in output_constructor_regular_field, at varasm.c:5514

2021-08-19 Thread Tobias Burnus
Hi Harald, On 18.08.21 23:01, Harald Anlauf wrote: Von: "Tobias Burnus" Note, however, that gfc_simplify_len still won't handle neither deferred strings nor their substrings. Obviously, nonsubstrings cannot be simplified but I do not see why len(str(1:2)) cannot or should not be simplified.

Re: F2018 C937

2021-08-19 Thread Arjen Markus via Fortran
Hi Steve, I am willing to take up this challenge ;), as well as the patch for C949. It would be my next attempt to get acquainted with the source code (a first step hopefully to actively contribute). Regards, Arjen Op di 17 aug. 2021 om 21:02 schreef Steve Kargl via Fortran <