[binutils-gdb/jremus/sframe-v3-s390x] TODO

Jens Remus jremus@sourceware.org
Mon Dec 15 16:26:12 GMT 2025


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

commit 34fae694f644eebd77509e010a9ecb2033aba886
Author: Jens Remus <jremus@linux.ibm.com>
Date:   Fri Dec 12 16:58:00 2025 +0100

    TODO
    
    Signed-off-by: Jens Remus <jremus@linux.ibm.com>

Diff:
---
 libsframe/sframe-dump.c | 4 +++-
 libsframe/sframe.c      | 2 ++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/libsframe/sframe-dump.c b/libsframe/sframe-dump.c
index b650720a6f9..af373123d9a 100644
--- a/libsframe/sframe-dump.c
+++ b/libsframe/sframe-dump.c
@@ -380,7 +380,9 @@ dump_sframe_func_fre_flex_topmost (const sframe_decoder_ctx *sfd_ctx,
 	}
 
       int err_ra_reg = 0;
-      int err_ra_offset = 0;
+      /* RA is not tracked, unless necessary for the FRE.  Assume no RA offset
+	 by default.  */
+      int err_ra_offset = SFRAME_ERR;
       int32_t ra_reg_data = sframe_get_fre_offset (&fre,
 						   SFRAME_FRE_RA_OFFSET_IDX * 2,
 						   &err_ra_reg);
diff --git a/libsframe/sframe.c b/libsframe/sframe.c
index a567aad9a6d..946c34590ce 100644
--- a/libsframe/sframe.c
+++ b/libsframe/sframe.c
@@ -1148,6 +1148,7 @@ sframe_fre_get_fp_offset (const sframe_decoder_ctx *dctx,
   int8_t fp_offset = sframe_decoder_get_fixed_fp_offset (dctx);
   /* If the FP offset is not being tracked, return the fixed FP offset
      from the SFrame header.  */
+  /* TODO: RA undefined support for FLEX_TOPMOST_FRAME.  */
   if (!flex_p && fp_offset != SFRAME_CFA_FIXED_FP_INVALID
       && !sframe_get_fre_ra_undefined_p (fre->fre_info))
     {
@@ -1187,6 +1188,7 @@ sframe_fre_get_ra_offset (const sframe_decoder_ctx *dctx,
 
   /* If the RA offset was not being tracked, return the fixed RA offset
      from the SFrame header.  */
+  /* TODO: RA undefined support for FLEX_TOPMOST_FRAME.  */
   if (!flex_p && ra_offset != SFRAME_CFA_FIXED_RA_INVALID
       && !sframe_get_fre_ra_undefined_p (fre->fre_info))
     {


More information about the Binutils-cvs mailing list