[binutils-gdb] gas: sframe: pass SFrame FDE as argument to output_sframe_row_entry

Indu Bhagat ibhagat@sourceware.org
Wed Dec 24 08:46:39 GMT 2025


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

commit d9dd80de9add618124ff66ba34a7eedc58eba93b
Author: Indu Bhagat <indu.bhagat@oracle.com>
Date:   Wed Dec 24 00:43:30 2025 -0800

    gas: sframe: pass SFrame FDE as argument to output_sframe_row_entry
    
    Minor refactoring.  Will be needed later for supporting flexible topmost
    frames in SFrame V3.
    
    Reviewed-by: Jens Remus <jremus@linux.ibm.com>
    
    gas/
            * gen-sframe.c (output_sframe_row_entry): Use new argument.
            (output_sframe_internal): Likewise.

Diff:
---
 gas/gen-sframe.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/gas/gen-sframe.c b/gas/gen-sframe.c
index ecd56e8f164..ad865167938 100644
--- a/gas/gen-sframe.c
+++ b/gas/gen-sframe.c
@@ -568,8 +568,7 @@ sframe_fde_free (struct sframe_func_entry *sframe_fde)
 }
 
 static void
-output_sframe_row_entry (symbolS *fde_start_addr,
-			 symbolS *fde_end_addr,
+output_sframe_row_entry (const struct sframe_func_entry *sframe_fde,
 			 const struct sframe_row_entry *sframe_fre)
 {
   unsigned char fre_info;
@@ -582,6 +581,8 @@ output_sframe_row_entry (symbolS *fde_start_addr,
 
   unsigned int idx = 0;
   unsigned int fre_write_offsets = 0;
+  symbolS *fde_start_addr = get_dw_fde_start_addrS (sframe_fde->dw_fde);
+  symbolS *fde_end_addr = get_dw_fde_end_addrS (sframe_fde->dw_fde);
 
   fre_addr_size = 4; /* 4 bytes by default.   FIXME tie it to fre_type? */
 
@@ -851,9 +852,7 @@ output_sframe_internal (void)
 	   sframe_fre;
 	   sframe_fre = sframe_fre->next)
 	{
-	  output_sframe_row_entry (get_dw_fde_start_addrS (sframe_fde->dw_fde),
-				   get_dw_fde_end_addrS (sframe_fde->dw_fde),
-				   sframe_fre);
+	  output_sframe_row_entry (sframe_fde, sframe_fre);
 	}
       i++;
       sframe_fde_next = sframe_fde->next;


More information about the Binutils-cvs mailing list