Hi Mikael, All three patches are OK for mainline. About half of the chunks I recognise and blames should point to me :-(
Thanks for the patch. Paul On Sun, 3 Aug 2025 at 14:30, Mikael Morin <morin-mik...@orange.fr> wrote: > From: Mikael Morin <mik...@gcc.gnu.org> > > The fortran frontend has getter and setter functions to generate code > either to read array descriptor fields or to write to them. > In some places, those functions are not used, and the access to the > field is generated manually. > > The following patches replace those direct accesses with the existing > getters and setters. Is is not only an attempt to share more code. > The systematic use of setters to write and getters to read array > descriptor fields gives a clear view of which fields are accessed and > how and where, which is not possible if the generation of the field > reference is separated from its usage. > > Mikael Morin (3): > fortran: Use array descriptor data setter when possible > fortran: Remove array descriptor data address accessor function > fortran: Use array descriptor offset setter when possible > > gcc/fortran/trans-array.cc | 19 ++++--------------- > gcc/fortran/trans-array.h | 1 - > gcc/fortran/trans-decl.cc | 25 +++++++++++++++++++------ > gcc/fortran/trans-expr.cc | 2 +- > gcc/fortran/trans-stmt.cc | 7 ++++--- > gcc/fortran/trans.cc | 4 ++-- > 6 files changed, 30 insertions(+), 28 deletions(-) > > -- > 2.47.2 > >
