Hello Mikael,

I am very glad to see this resubmission. Equally, I am happy with your
assurance that tis version is only trivially different from the
previous one.

>From me, OK for mainline.

Thanks to you and TF for this work.

Paul

On Sun, 28 Jun 2026 at 17:32, Mikael Morin <[email protected]> wrote:
>
> From: Mikael Morin <[email protected]>
>
> This is an update of the series posted just before the end of stage 1 in
> october:
> https://gcc.gnu.org/pipermail/fortran/2025-October/063053.html
> https://gcc.gnu.org/pipermail/gcc-patches/2025-October/699114.html
>
> I have added references to the PR opened to track the full patch series,
> rebased on current master, and made a couple of whitespace or ChangeLog
> fixes.  The patches are otherwise unchanged.
>
> The originally posted patches have been approved last year already; I plan
> to push this series next week under that approval.
>
> --
>
> This starts a series of patches (funded by STF) to refactor initialisation
> and updates of array descriptors.
>
> The current API touching array descriptors is mostly limited to a set of
> getters to pick the value of individual array descriptor fields, and setters
> to update them.  This requires every area of the compiler touching array
> descriptors to have knowledge of their contents.
>
> The final goal of the patches is to remove (most of) the setters from the
> public API, and replace them with more high level functions providing
> complete array descriptor updates (say copy descriptor, or shift array
> bounds) without leaking the detail of the fields that have their values
> changed under the hood.
>
> This first series moves the existing array descriptor functions to a
> separate file.
>
> Mikael Morin (7):
>   fortran: array descriptor: Move accessor functions to a separate file
>     [PR122521]
>   fortran: array descriptor: Move debug info generation function
>     [PR122521]
>   fortran: array descriptor: Move null factory function [PR122521]
>   fortran: array descriptor: Move size and cosize functions [PR122521]
>   fortran: array descriptor: Move bound shift utility function
>     [PR122521]
>   fortran: array descriptor: Move descriptor copy function [PR122521]
>   fortran: array descriptor: Move array growth function [PR122521]
>
>  gcc/fortran/Make-lang.in        |   7 +-
>  gcc/fortran/trans-array.cc      | 504 +--------------------------
>  gcc/fortran/trans-array.h       |  33 --
>  gcc/fortran/trans-decl.cc       |   1 +
>  gcc/fortran/trans-descriptor.cc | 580 ++++++++++++++++++++++++++++++++
>  gcc/fortran/trans-descriptor.h  |  67 ++++
>  gcc/fortran/trans-expr.cc       |   1 +
>  gcc/fortran/trans-intrinsic.cc  |   1 +
>  gcc/fortran/trans-io.cc         |   1 +
>  gcc/fortran/trans-openmp.cc     |   1 +
>  gcc/fortran/trans-stmt.cc       |  47 +--
>  gcc/fortran/trans-types.cc      |   1 +
>  gcc/fortran/trans.cc            |   1 +
>  13 files changed, 662 insertions(+), 583 deletions(-)
>  create mode 100644 gcc/fortran/trans-descriptor.cc
>  create mode 100644 gcc/fortran/trans-descriptor.h
>
> --
> 2.53.0
>

Reply via email to