[binutils-gdb] libsframe: bfd: use uint32_t for return type of sframe_calc_fre_type

Indu Bhagat ibhagat@sourceware.org
Tue Jun 27 19:42:36 GMT 2023


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=49e4485cba2fa322a649b39c8a05dae8b421331d

commit 49e4485cba2fa322a649b39c8a05dae8b421331d
Author: Indu Bhagat <indu.bhagat@oracle.com>
Date:   Tue Jun 27 11:55:38 2023 -0700

    libsframe: bfd: use uint32_t for return type of sframe_calc_fre_type
    
    Use uint32_t type alias consistently for all APIs in libsframe.
    
    bfd/
            * elfxx-x86.c (_bfd_x86_elf_create_sframe_plt): Adjust for the
            changed return type.
    libsframe/
            * sframe.c (sframe_calc_fre_type): Use uint32_t for return type.
    include/
            * sframe-api.h (sframe_calc_fre_type): Likewise.

Diff:
---
 bfd/elfxx-x86.c      | 2 +-
 include/sframe-api.h | 2 +-
 libsframe/sframe.c   | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/bfd/elfxx-x86.c b/bfd/elfxx-x86.c
index 8e13a92e7f9..ffd02f137d1 100644
--- a/bfd/elfxx-x86.c
+++ b/bfd/elfxx-x86.c
@@ -1832,7 +1832,7 @@ _bfd_x86_elf_create_sframe_plt (bfd *output_bfd,
   bool plt0_generated_p;
   unsigned int plt0_entry_size;
   unsigned char func_info;
-  unsigned int fre_type;
+  uint32_t fre_type;
   /* The dynamic plt section for which .sframe stack trace information is being
      created.  */
   asection *dpltsec;
diff --git a/include/sframe-api.h b/include/sframe-api.h
index d04ebe85d11..70829ec87cb 100644
--- a/include/sframe-api.h
+++ b/include/sframe-api.h
@@ -98,7 +98,7 @@ sframe_fde_create_func_info (uint32_t fre_type, uint32_t fde_type);
 
 /* Gather the FRE type given the function size.  */
 
-extern unsigned int
+extern uint32_t
 sframe_calc_fre_type (size_t func_size);
 
 /* The SFrame Decoder.  */
diff --git a/libsframe/sframe.c b/libsframe/sframe.c
index 7391df0d3b5..8e9dc0a9d2c 100644
--- a/libsframe/sframe.c
+++ b/libsframe/sframe.c
@@ -601,7 +601,7 @@ sframe_fde_create_func_info (uint32_t fre_type,
 /* Get the FRE type given the function size.  */
 /* FIXME API for linker.  Revisit if its better placed somewhere else?  */
 
-unsigned int
+uint32_t
 sframe_calc_fre_type (size_t func_size)
 {
   uint32_t fre_type = 0;


More information about the Binutils-cvs mailing list