https://gcc.gnu.org/g:71f157ac716618601d5fd0cb4d3d6a3ea28a897f
commit 71f157ac716618601d5fd0cb4d3d6a3ea28a897f Author: Mikael Morin <[email protected]> Date: Wed Oct 29 12:55:47 2025 +0100 fortran: array descriptor: Move null factory function Regression tested on powerpc64le-unknown-linux-gnu. OK for master? -- >8 -- Move null descriptor factory function to trans-descriptor.cc. gcc/fortran/ChangeLog: * trans-array.cc (gfc_build_null_descriptor): Move function to ... * trans-descriptor.cc (gfc_build_null_descriptor): ... this file. * trans-array.h (gfc_build_null_descriptor): Move declaration to ... * trans-descriptor.h (gfc_build_null_descriptor): ... this file. Diff: --- gcc/fortran/trans-array.cc | 21 --------------------- gcc/fortran/trans-array.h | 2 -- gcc/fortran/trans-descriptor.cc | 21 +++++++++++++++++++++ gcc/fortran/trans-descriptor.h | 3 +++ 4 files changed, 24 insertions(+), 23 deletions(-) diff --git a/gcc/fortran/trans-array.cc b/gcc/fortran/trans-array.cc index c7d86276cf90..ab419bbae0bf 100644 --- a/gcc/fortran/trans-array.cc +++ b/gcc/fortran/trans-array.cc @@ -214,27 +214,6 @@ gfc_get_cfi_dim_sm (tree desc, tree idx) #undef CFI_DIM_FIELD_SM -/* Build a null array descriptor constructor. */ - -tree -gfc_build_null_descriptor (tree type) -{ - tree field; - tree tmp; - - gcc_assert (GFC_DESCRIPTOR_TYPE_P (type)); - gcc_assert (DATA_FIELD == 0); - field = TYPE_FIELDS (type); - - /* Set a NULL data pointer. */ - tmp = build_constructor_single (type, field, null_pointer_node); - TREE_CONSTANT (tmp) = 1; - /* All other fields are ignored. */ - - return tmp; -} - - /* Modify a descriptor such that the lbound of a given dimension is the value specified. This also updates ubound and offset accordingly. */ diff --git a/gcc/fortran/trans-array.h b/gcc/fortran/trans-array.h index 6125301820f7..7eb21dbb32dd 100644 --- a/gcc/fortran/trans-array.h +++ b/gcc/fortran/trans-array.h @@ -137,8 +137,6 @@ void gfc_conv_loop_setup (gfc_loopinfo *, locus *); void gfc_set_delta (gfc_loopinfo *); /* Resolve array assignment dependencies. */ void gfc_conv_resolve_dependencies (gfc_loopinfo *, gfc_ss *, gfc_ss *); -/* Build a null array descriptor constructor. */ -tree gfc_build_null_descriptor (tree); /* Get a single array element. */ void gfc_conv_array_ref (gfc_se *, gfc_array_ref *, gfc_expr *, locus *); diff --git a/gcc/fortran/trans-descriptor.cc b/gcc/fortran/trans-descriptor.cc index b655767b7e64..4c02d4f16286 100644 --- a/gcc/fortran/trans-descriptor.cc +++ b/gcc/fortran/trans-descriptor.cc @@ -382,6 +382,27 @@ gfc_get_descriptor_offsets_for_info (const_tree desc_type, tree *data_off, } +/* Build a null array descriptor constructor. */ + +tree +gfc_build_null_descriptor (tree type) +{ + tree field; + tree tmp; + + gcc_assert (GFC_DESCRIPTOR_TYPE_P (type)); + gcc_assert (DATA_FIELD == 0); + field = TYPE_FIELDS (type); + + /* Set a NULL data pointer. */ + tmp = build_constructor_single (type, field, null_pointer_node); + TREE_CONSTANT (tmp) = 1; + /* All other fields are ignored. */ + + return tmp; +} + + /* Cleanup those #defines. */ #undef DATA_FIELD diff --git a/gcc/fortran/trans-descriptor.h b/gcc/fortran/trans-descriptor.h index db877ee402b6..44c8f6f1215d 100644 --- a/gcc/fortran/trans-descriptor.h +++ b/gcc/fortran/trans-descriptor.h @@ -51,4 +51,7 @@ void gfc_get_descriptor_offsets_for_info (const_tree, tree *, tree *, tree *, tree *, tree *, tree *, tree *, tree *); +/* Build a null array descriptor constructor. */ +tree gfc_build_null_descriptor (tree type); + #endif /* GFC_TRANS_DESCRIPTOR_H */
