Re: [PATCH,Fortran 1/2] Add uop/name helpers

2021-10-29 Thread Jakub Jelinek via Fortran
On Fri, Oct 29, 2021 at 01:52:58AM +0200, Bernhard Reutner-Fischer wrote: > From: Bernhard Reutner-Fischer > > Introduce a helper to construct a user operator from a name and the > reverse operation, i.e. a helper to construct a name from a user > operator. > > Cc: Jakub Jelinek > > gcc/fortra

[PATCH] Fortran: adjust column sizes in intrinsic.texi

2021-10-29 Thread Manfred Schwarb via Fortran
Hi, in intrinsic.texi, a lot of tables wrap lines when watching the resulting info file in a 80char terminal. Adjust the @columnfractions items to fit screen. Some minor white space changes are added as well to help saving space. Signed-off-by Manfred Schwarb [Note: I do not have commit acces

[PATCH] Fortran: adjust error message for SHORT and LONG intrinsics

2021-10-29 Thread Manfred Schwarb via Fortran
Hi, on 2019-07-23, support for SHORT and LONG intrinsics were removed be Steve Kargl by adding an error message in check.c. However, the error message Error: 'long' intrinsic subprogram at (1) has been deprecated is misleading, as support has been disabled by this patch. Adjust the error mess

[PATCH] Fortran: Remove documentation for SHORT and LONG intrinics

2021-10-29 Thread Manfred Schwarb via Fortran
Hi, on 2019-07-23, support for SHORT and LONG intrinsics was removed be Steve Kargl by adding an error message in check.c. As far as I can see code support is still there, though. Remove documentation for these intrinsics. Signed-off-by Manfred Schwarb [Note: I do not have commit access] -

[PATCH] Fortran: Correct documentation for REAL intrinsic

2021-10-29 Thread Manfred Schwarb via Fortran
Hi, documentation for REAL intrinsic is slightly wrong. Fix it. Patch is done on top of the column adjustment patch. Signed-off-by Manfred Schwarb [Note: I do not have commit access] --- gcc/gcc/fortran/intrinsic.texi.2 2021-10-29 15:08:38.302188947 +0200 +++ gcc/gcc/fortran/intrinsic.texi 202

[PATCH] Fortran: recognize Gerhard Steinmetz

2021-10-29 Thread Manfred Schwarb via Fortran
Hi, there were really a lot of test cases provided by Gerhard Steinmetz lately. Although I'm not really in the position to suggest this, I would appreciate it, if one could recognize him by adding an entry to gfortran.texi. As e.g. in the proposed patch. Such a patch should probably be signed-off

Re: [PATCH,FORTRAN 28/29] Free type-bound procedure structs

2021-10-29 Thread Jerry D via Fortran
Looks good and simple. Proceed. Thanks Jerry On 10/28/21 5:05 PM, Bernhard Reutner-Fischer via Fortran wrote: ping [Rebased, re-regtested cleanly. Ok for trunk?] On Wed, 5 Sep 2018 14:57:31 + Bernhard Reutner-Fischer wrote: From: Bernhard Reutner-Fischer compiling gfortran.dg/typeboun

Re: [PATCH,FORTRAN 28/29] Free type-bound procedure structs

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Fri, 29 Oct 2021 07:54:21 -0700 Jerry D wrote: > Looks good and simple. Proceed. Thanks Committed as 7883a7f07c1ad9c8aaccc5bbd96e0ae1fa230c89 Thanks! Maybe somebody could eyeball and ACK "Fix memory leak in finalization wrappers" https://gcc.gnu.org/pipermail/fortran/2021-October/056838.html

Re: [PATCH,Fortran 1/2] Add uop/name helpers

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Fri, 29 Oct 2021 13:13:52 +0200 Jakub Jelinek wrote: > On Fri, Oct 29, 2021 at 01:52:58AM +0200, Bernhard Reutner-Fischer wrote: > > From: Bernhard Reutner-Fischer > > > > Introduce a helper to construct a user operator from a name and the > > reverse operation, i.e. a helper to construct a

missing simplify of CAF get_team [was: Re: [PATCH,Fortran 0/7] delete some unused decls, make static]

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Mon, 25 Oct 2021 00:30:16 +0200 Bernhard Reutner-Fischer wrote: > - delete unused gfc_simplify_get_team or wire it up in intrinsics, > get_team handling (instead of the NULL..) > > Anyone who does coarrays might want to fill in the missing get_team() > simplify and add an appropriate test.

Re: [PATCH] PR fortran/99853 - ICE: Cannot convert 'LOGICAL(4)' to 'INTEGER(8)' (etc.)

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Thu, 28 Oct 2021 23:03:05 +0200 Harald Anlauf via Fortran wrote: > Dear Fortranners, > > the original fix by Steve was lingering in the PR. > > We did ICE in situations where in a SELECT CASE a kind conversion > was deemed necessary, but it did involve different types. > The check gfc_conver

Re: [PATCH,FORTRAN 01/29] gdbinit: break on gfc_internal_error

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
ping On Wed, 5 Sep 2018 14:57:04 + Bernhard Reutner-Fischer wrote: > From: Bernhard Reutner-Fischer > > Aids debugging the fortran FE. > > gcc/ChangeLog: > > 2017-11-12 Bernhard Reutner-Fischer > > * gdbinit.in: Break on gfc_internal_error. > --- > gcc/gdbinit.in | 1 + > 1 f

Re: libgfortran.so SONAME and powerpc64le-linux ABI changes (work in progress patches)

2021-10-29 Thread Thomas Koenig via Fortran
Hi Michael, I tried this out on the one POWER machine where I can get something installed :-) It runs Ubuntu 20.04, but does not appear to have the right glibc version; it has $ lsb_release -a No LSB modules are available. Distributor ID: Ubuntu Description:Ubuntu 20.04.1 LTS Release:

Re: [PATCH] Fortran: adjust column sizes in intrinsic.texi

2021-10-29 Thread Harald Anlauf via Fortran
Hi Manfred, Am 29.10.21 um 16:05 schrieb Manfred Schwarb via Fortran: Hi, in intrinsic.texi, a lot of tables wrap lines when watching the resulting info file in a 80char terminal. Adjust the @columnfractions items to fit screen. Some minor white space changes are added as well to help saving s

Re: [PATCH] Fortran: adjust error message for SHORT and LONG intrinsics

2021-10-29 Thread Harald Anlauf via Fortran
Hi Manfred, Am 29.10.21 um 16:12 schrieb Manfred Schwarb via Fortran: Hi, on 2019-07-23, support for SHORT and LONG intrinsics were removed be Steve Kargl by adding an error message in check.c. However, the error message Error: 'long' intrinsic subprogram at (1) has been deprecated is misl

Re: [PATCH] Fortran: Remove documentation for SHORT and LONG intrinics

2021-10-29 Thread Harald Anlauf via Fortran
Hi Manfred, Am 29.10.21 um 16:13 schrieb Manfred Schwarb via Gcc-patches: Hi, on 2019-07-23, support for SHORT and LONG intrinsics was removed be Steve Kargl by adding an error message in check.c. As far as I can see code support is still there, though. Remove documentation for these intrin

Re: [PATCH] Fortran: recognize Gerhard Steinmetz

2021-10-29 Thread Harald Anlauf via Fortran
Hi Manfred, Am 29.10.21 um 16:33 schrieb Manfred Schwarb via Fortran: Hi, there were really a lot of test cases provided by Gerhard Steinmetz lately. Although I'm not really in the position to suggest this, I would appreciate it, if one could recognize him by adding an entry to gfortran.texi.

Re: [PATCH] Fortran: Correct documentation for REAL intrinsic

2021-10-29 Thread Harald Anlauf via Fortran
Hi Manfred, Am 29.10.21 um 16:18 schrieb Manfred Schwarb via Gcc-patches: Hi, documentation for REAL intrinsic is slightly wrong. Fix it. Patch is done on top of the column adjustment patch. the patch looks fine, but it would help a lot to have a ChangeLog entry. Thanks, Harald Signed-off-

Re: [PATCH,FORTRAN 28/29] Free type-bound procedure structs

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Fri, 29 Oct 2021 21:36:26 +0200 Harald Anlauf via Gcc-patches wrote: > Dear Bernhard, all, > > Am 29.10.21 um 02:05 schrieb Bernhard Reutner-Fischer via Gcc-patches: > > >> diff --git a/gcc/fortran/symbol.c b/gcc/fortran/symbol.c > >> index 53c760a6c38..cde34c67482 100644 > >> --- a/gcc/fort

Re: libgfortran.so SONAME and powerpc64le-linux ABI changes (work in progress patches)

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
Michael, On Thu, 28 Oct 2021 23:36:20 -0400 Michael Meissner via Fortran wrote: ISTM the second @defmac FORTRAN_USE_LONG_DOUBLE in tm.texi.in should be FORTRAN_USE_FLOAT128 thanks,

Re: libgfortran.so SONAME and powerpc64le-linux ABI changes (work in progress patches)

2021-10-29 Thread Michael Meissner via Fortran
On Fri, Oct 29, 2021 at 09:07:38PM +0200, Thomas Koenig wrote: > Hi Michael, > > I tried this out on the one POWER machine where I can get something > installed :-) It runs Ubuntu 20.04, but does not appear to have the > right glibc version; it has > > $ lsb_release -a > No LSB modules are availa

Re: [PATCH,Fortran 1/2] Add uop/name helpers

2021-10-29 Thread Bernhard Reutner-Fischer via Fortran
On Fri, 29 Oct 2021 19:15:13 +0200 Bernhard Reutner-Fischer wrote: > > But most importantly, I really don't like these helpers at all, they > > unnecessarily allocate memory of the remaining duration of compilation, > > and the second one even uses heap for temporary. > I can easily switch the

Re: [PATCH,FORTRAN 01/29] gdbinit: break on gfc_internal_error

2021-10-29 Thread Jerry D via Fortran
Looks OK. Cheers On 10/29/21 11:58 AM, Bernhard Reutner-Fischer via Fortran wrote: ping On Wed, 5 Sep 2018 14:57:04 + Bernhard Reutner-Fischer wrote: From: Bernhard Reutner-Fischer Aids debugging the fortran FE. gcc/ChangeLog: 2017-11-12 Bernhard Reutner-Fischer * gdbin

Re: libgfortran.so SONAME and powerpc64le-linux ABI changes (work in progress patches)

2021-10-29 Thread Michael Meissner via Fortran
On Fri, Oct 29, 2021 at 11:21:33PM +0200, Bernhard Reutner-Fischer wrote: > Michael, > > On Thu, 28 Oct 2021 23:36:20 -0400 > Michael Meissner via Fortran wrote: > > ISTM the second > @defmac FORTRAN_USE_LONG_DOUBLE > in tm.texi.in should be FORTRAN_USE_FLOAT128 Thanks. I had noticed that afte

Re: [PATCH] Fortran: adjust column sizes in intrinsic.texi

2021-10-29 Thread Manfred Schwarb via Fortran
Am 29.10.21 um 21:44 schrieb Harald Anlauf via Fortran: > Hi Manfred, > > Am 29.10.21 um 16:05 schrieb Manfred Schwarb via Fortran: >> Hi, >> >> in intrinsic.texi, a lot of tables wrap lines when watching the >> resulting info file in a 80char terminal. >> >> Adjust the @columnfractions items to fi

Re: [PATCH] Fortran: adjust error message for SHORT and LONG intrinsics

2021-10-29 Thread Manfred Schwarb via Fortran
Am 29.10.21 um 21:51 schrieb Harald Anlauf via Fortran: > Hi Manfred, > > Am 29.10.21 um 16:12 schrieb Manfred Schwarb via Fortran: >> Hi, >> >> on 2019-07-23, support for SHORT and LONG intrinsics were removed be Steve >> Kargl by >> adding an error message in check.c. However, the error message

Re: [PATCH] Fortran: Remove documentation for SHORT and LONG intrinics

2021-10-29 Thread Manfred Schwarb via Fortran
Am 29.10.21 um 21:52 schrieb Harald Anlauf via Fortran: > Hi Manfred, > > Am 29.10.21 um 16:13 schrieb Manfred Schwarb via Gcc-patches: >> Hi, >> >> on 2019-07-23, support for SHORT and LONG intrinsics was removed be Steve >> Kargl by >> adding an error message in check.c. As far as I can see cod

Re: [PATCH] Fortran: Correct documentation for REAL intrinsic

2021-10-29 Thread Manfred Schwarb via Fortran
Am 29.10.21 um 21:56 schrieb Harald Anlauf via Fortran: > Hi Manfred, > > Am 29.10.21 um 16:18 schrieb Manfred Schwarb via Gcc-patches: >> Hi, >> >> documentation for REAL intrinsic is slightly wrong. Fix it. >> Patch is done on top of the column adjustment patch. > > the patch looks fine, but it w

Re: [PATCH] Fortran: recognize Gerhard Steinmetz

2021-10-29 Thread Manfred Schwarb via Fortran
Am 29.10.21 um 21:58 schrieb Harald Anlauf via Fortran: > Hi Manfred, > > Am 29.10.21 um 16:33 schrieb Manfred Schwarb via Fortran: >> Hi, >> there were really a lot of test cases provided by Gerhard Steinmetz lately. >> Although I'm not really in the position to suggest this, >> I would appreciate

Re: libgfortran.so SONAME and powerpc64le-linux ABI changes (2nd patch)

2021-10-29 Thread Michael Meissner via Fortran
This patch replaces the first patch. Instead of disallowing long double and only dealing with float128 types, this patch tries to accommodate the two. It adds target hooks so the back end can overwrite the kind number for types. I made the IBM long double type use KIND=15 instead of KIND=16, and