| Message ID | 20240220102427.1512739-3-cupertino.miranda@oracle.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/5] btf: fixed type id in BTF_KIND_FUNC struct data. | expand |
On 2/20/24 02:24, Cupertino Miranda wrote: > Added a traversal function to traverse all BTF_KIND_FUNC nodes with a > callback function. Used for .BTF.ext section content creation. > > gcc/ChangeLog > > * btfout.cc (output_btf_func_types): use FOR_EACH_VEC_ELT. > (traverse_btf_func_types): Defined function. > * ctfc.h (funcs_traverse_callback): typedef for function > prototype. > (traverse_btf_func_types): Added prototype. OK, thanks. > --- > gcc/btfout.cc | 22 ++++++++++++++++++++-- > gcc/ctfc.h | 3 +++ > 2 files changed, 23 insertions(+), 2 deletions(-) > > diff --git a/gcc/btfout.cc b/gcc/btfout.cc > index 7e114e224449..7aabd99f3e7c 100644 > --- a/gcc/btfout.cc > +++ b/gcc/btfout.cc > @@ -1276,8 +1276,10 @@ output_btf_types (ctf_container_ref ctfc) > static void > output_btf_func_types (ctf_container_ref ctfc) > { > - for (size_t i = 0; i < vec_safe_length (funcs); i++) > - btf_asm_func_type (ctfc, (*funcs)[i], i); > + ctf_dtdef_ref ref; > + unsigned i; > + FOR_EACH_VEC_ELT (*funcs, i, ref) > + btf_asm_func_type (ctfc, ref, i); > } > > /* Output all BTF_KIND_DATASEC records. */ > @@ -1452,4 +1454,20 @@ btf_finalize (void) > tu_ctfc = NULL; > } > > +/* Traversal function for all BTF_KIND_FUNC type records. */ > + > +bool > +traverse_btf_func_types (funcs_traverse_callback callback, void *data) > +{ > + ctf_dtdef_ref ref; > + unsigned i; > + FOR_EACH_VEC_ELT (*funcs, i, ref) > + { > + bool stop = callback (ref, data); > + if (stop == true) > + return true; > + } > + return false; > +} > + > #include "gt-btfout.h" > diff --git a/gcc/ctfc.h b/gcc/ctfc.h > index 7aac57edac55..fa188bf2f5a4 100644 > --- a/gcc/ctfc.h > +++ b/gcc/ctfc.h > @@ -441,6 +441,9 @@ extern int ctf_add_variable (ctf_container_ref, const char *, ctf_id_t, > extern ctf_id_t ctf_lookup_tree_type (ctf_container_ref, const tree); > extern ctf_id_t get_btf_id (ctf_id_t); > > +typedef bool (*funcs_traverse_callback) (ctf_dtdef_ref, void *); > +bool traverse_btf_func_types (funcs_traverse_callback, void *); > + > /* CTF section does not emit location information; at this time, location > information is needed for BTF CO-RE use-cases. */ >
diff --git a/gcc/btfout.cc b/gcc/btfout.cc index 7e114e224449..7aabd99f3e7c 100644 --- a/gcc/btfout.cc +++ b/gcc/btfout.cc @@ -1276,8 +1276,10 @@ output_btf_types (ctf_container_ref ctfc) static void output_btf_func_types (ctf_container_ref ctfc) { - for (size_t i = 0; i < vec_safe_length (funcs); i++) - btf_asm_func_type (ctfc, (*funcs)[i], i); + ctf_dtdef_ref ref; + unsigned i; + FOR_EACH_VEC_ELT (*funcs, i, ref) + btf_asm_func_type (ctfc, ref, i); } /* Output all BTF_KIND_DATASEC records. */ @@ -1452,4 +1454,20 @@ btf_finalize (void) tu_ctfc = NULL; } +/* Traversal function for all BTF_KIND_FUNC type records. */ + +bool +traverse_btf_func_types (funcs_traverse_callback callback, void *data) +{ + ctf_dtdef_ref ref; + unsigned i; + FOR_EACH_VEC_ELT (*funcs, i, ref) + { + bool stop = callback (ref, data); + if (stop == true) + return true; + } + return false; +} + #include "gt-btfout.h" diff --git a/gcc/ctfc.h b/gcc/ctfc.h index 7aac57edac55..fa188bf2f5a4 100644 --- a/gcc/ctfc.h +++ b/gcc/ctfc.h @@ -441,6 +441,9 @@ extern int ctf_add_variable (ctf_container_ref, const char *, ctf_id_t, extern ctf_id_t ctf_lookup_tree_type (ctf_container_ref, const tree); extern ctf_id_t get_btf_id (ctf_id_t); +typedef bool (*funcs_traverse_callback) (ctf_dtdef_ref, void *); +bool traverse_btf_func_types (funcs_traverse_callback, void *); + /* CTF section does not emit location information; at this time, location information is needed for BTF CO-RE use-cases. */