Re: [PATCH] Fortran: fix crash with bounds check writing array section [PR117791]

2024-11-27 Thread Harald Anlauf
Am 27.11.24 um 21:56 schrieb Jerry D: On 11/27/24 12:31 PM, Harald Anlauf wrote: Dear all, the attached patch fixes a wrong-code issue with bounds-checking enabled when doing I/O of an array section and an index is either an expression or a function result.  The problem does not occur without b

Re: [PATCH] Fortran: fix crash with bounds check writing array section [PR117791]

2024-11-27 Thread Jerry D
On 11/27/24 12:31 PM, Harald Anlauf wrote: Dear all, the attached patch fixes a wrong-code issue with bounds-checking enabled when doing I/O of an array section and an index is either an expression or a function result. The problem does not occur without bounds-checking. When looking at the or

[PATCH] Fortran: fix crash with bounds check writing array section [PR117791]

2024-11-27 Thread Harald Anlauf
Dear all, the attached patch fixes a wrong-code issue with bounds-checking enabled when doing I/O of an array section and an index is either an expression or a function result. The problem does not occur without bounds-checking. When looking at the original testcase, the function occuring in the

[Patch, fortran] PR117768 - [15.0 regression] ICE in diagnostic_impl (?)

2024-11-27 Thread Paul Richard Thomas
Hi All, The "fix" for PR84674 caused this regression. The diagnostics that I had used for PR117763 allowed me to find a much better fix for PR84674 and so this patch reverts the chunk in resolve.cc. The chunk in class.cc works because non_overridable typebound procedures, whose parent is abstrac

Re: [Patch, fortran] PR117768 - [15.0 regression] ICE in diagnostic_impl (?)

2024-11-27 Thread Paul Richard Thomas
Hi Andre, Yes indeed, I did regtest the patch :-) Thanks for the thumbs up. Paul On Wed, 27 Nov 2024 at 09:07, Andre Vehreschild wrote: > Hi Paul, > > the patch looks fine to me. I assume you have regtested it? (Because you > don't > state so.) > > Thanks for the patch. > > - Andre > > On We

Re: [Patch, fortran] PR117768 - [15.0 regression] ICE in diagnostic_impl (?)

2024-11-27 Thread Andre Vehreschild
Hi Paul, the patch looks fine to me. I assume you have regtested it? (Because you don't state so.) Thanks for the patch. - Andre On Wed, 27 Nov 2024 08:57:25 + Paul Richard Thomas wrote: > Hi All, > > The "fix" for PR84674 caused this regression. > > The diagnostics that I had used for PR

Re: [Patch, fortran] PR117768 - [15.0 regression] ICE in diagnostic_impl (?)

2024-11-27 Thread Paul Richard Thomas
Pushed as r15-5716. Paul On Wed, 27 Nov 2024 at 09:15, Paul Richard Thomas < paul.richard.tho...@gmail.com> wrote: > Hi Andre, > > Yes indeed, I did regtest the patch :-) > > Thanks for the thumbs up. > > Paul > > > On Wed, 27 Nov 2024 at 09:07, Andre Vehreschild wrote: > >> Hi Paul, >> >> the