Re: drop -fdump-fortran-global ? [was: Re: [PATCH, FORTRAN] Fix memory leak of gsymbol]

2023-02-25 Thread Thomas König
On 25.02.23 22:55, Bernhard Reutner-Fischer wrote: but should we really keep the option, if we don't even test basics and if it was more a specific debug dump, from the looks? It is always possible to add test cases if testing is required, but I don't think this is necessary. We also do not

drop -fdump-fortran-global ? [was: Re: [PATCH,FORTRAN] Fix memory leak of gsymbol]

2023-02-25 Thread Bernhard Reutner-Fischer via Fortran
On Sun, 31 Oct 2021 21:25:44 +0100 Bernhard Reutner-Fischer wrote: > On Sun, 31 Oct 2021 20:46:07 +0100 > Harald Anlauf wrote: > > > Am 30.10.21 um 23:51 schrieb Bernhard Reutner-Fischer via Fortran: > > >>> The only caller is translate_all_program_units. > > >>> Since we free only module

[PATCH, committed] Fortran: fix memory leak with real to integer conversion warning

2023-02-25 Thread Harald Anlauf via Fortran
Dear all, while checking f951 for memory leaks on testcases that appeared relevant during work on pr108924, I found that the conversion warning triggered by do_subscript_6.f90 uses a code path that forgot to mpfr_clear a used variable. The attached obvious patch fixes this - verified by

Re: fortran: Reuse associated_dummy memory if previously allocated [PR108923]

2023-02-25 Thread Mikael Morin
Le 25/02/2023 à 18:20, Harald Anlauf a écrit : Hi Mikael, Am 25.02.23 um 17:35 schrieb Mikael Morin: Hello, Harald found a testcase with memory still leaking despite my previous patch for PR108923. That patch was fixing a leak caused by absence of memory release, the attached patch fixes a

Re: fortran: Reuse associated_dummy memory if previously allocated [PR108923]

2023-02-25 Thread Harald Anlauf via Fortran
Hi Mikael, Am 25.02.23 um 17:35 schrieb Mikael Morin: Hello, Harald found a testcase with memory still leaking despite my previous patch for PR108923. That patch was fixing a leak caused by absence of memory release, the attached patch fixes a leak caused by pointer overwrite. I haven't

fortran: Reuse associated_dummy memory if previously allocated [PR108923]

2023-02-25 Thread Mikael Morin
Hello, Harald found a testcase with memory still leaking despite my previous patch for PR108923. That patch was fixing a leak caused by absence of memory release, the attached patch fixes a leak caused by pointer overwrite. I haven't investigated why sort_actual is called several times(

Re: [Patch] Fortran/OpenMP: Fix mapping of array descriptors and deferred-length strings

2023-02-25 Thread Thomas Schwinge
Hi Tobias! On 2023-02-23T17:42:08+0100, Tobias Burnus wrote: > (Side note: this patch has been committed to OG12 as > http://gcc.gnu.org/g:55a18d47442 ) I see og12 commit 55a18d4744258e3909568e425f9f473c49f9d13f "Fortran/OpenMP: Fix mapping of array descriptors and deferred-length strings"