Re: [Ping, Fortran, Patch, PR120637, v1] Ensure expression in finalizer creation is freed only when unused.

2025-07-08 Thread Andre Vehreschild
Hi Jerry, thanks for the review. Pushed as gcc-16-2086-gd1f05661fa6. Thanks again, Andre On Mon, 7 Jul 2025 10:49:50 -0700 Jerry D wrote: > On 7/7/25 8:39 AM, Andre Vehreschild wrote: > > Ping! > > > OK for mainline. > > Thanks, > > Jerry > > > > On Thu, 26 Jun 2025 15:32:47 +0

Re: [Fortran, Patch, PR120711, v1] 1/(3) Fix out of bounds access in cleanup of array constructor

2025-07-08 Thread Andre Vehreschild
HI Harald, hi Mikael, why shall the testcase be invalid? The `list` is empty. Concatenating with it is valid in Fortran like in most other programming languages. The mapping of an empty list in gfortran is to have the array's data pointer set to NULL and the ubound below the lbound. Current gfortr

Re: [Fortran, Patch, PR120711, v1] 1/(3) Fix out of bounds access in cleanup of array constructor

2025-07-08 Thread Harald Anlauf
Am 09.07.25 um 08:50 schrieb Andre Vehreschild: HI Harald, hi Mikael, why shall the testcase be invalid? The `list` is empty. Concatenating with it is valid in Fortran like in most other programming languages. The mapping of an empty list in gfortran is to have the array's data pointer set to NU

Re: [PATCH] libgfortran: add fallback for trigonometric pi-based functions

2025-07-08 Thread Yuao Ma
On 7/8/2025 2:03 AM, Steve Kargl wrote: On Sun, Jul 06, 2025 at 09:09:53PM +0800, Yuao Ma wrote: Hi Tobias, On 7/6/2025 6:34 PM, Tobias Burnus wrote: As that commit is from 2020 and 2.69 in from 2012, it seems as if your autoconf is too new. Can you re-check that the right version is at the

Re: Coarray shared memory testing

2025-07-08 Thread Jerry D
On the tsunami example from Curcic book Modern Fortran I am getting the following: $ cafrun -np 4 ./tsunami At line 201 of file mod_field.f90 Fortran runtime error: Attempting to allocate already allocated variable 'edge' At line 201 of file mod_field.f90 Fortran runtime error: Attempting to all

Re: [Fortran, Patch, PR120711, v1] 1/(3) Fix out of bounds access in cleanup of array constructor

2025-07-08 Thread Harald Anlauf
Am 05.07.25 um 14:55 schrieb Mikael Morin: Hello Andre, I get a regression on this testcase with a patch that is otherwise regression-free. I think the testcase is invalid. It does:     type(container), allocatable :: list(:)     list = [list, new_elem(5)] so it's using the variable 'list