[PATCH,V2 0/5] sframe: use data word instead of offset

[Jens Remus]

Hello Indu!

On 1/23/2026 11:35 AM, Indu Bhagat via Binutils wrote:
> This version is rebased on latest master and addresses Jens' reviews.
> 
> V1 posting: https://inbox.sourceware.org/binutils/20260120102824.2522876-1-indu.bhagat@oracle.com/
> 
> I will take one more look at these before committing.

I just realized that there are a few leftovers in include/sframe.h:

#define SFRAME_V3_FLEX_FDE_OFFSET_REG_NUM(data)       ((data) >> 3)
#define SFRAME_V3_FLEX_FDE_OFFSET_REG_DEREF_P(data)   (((data) >> 1) & 0x1)
#define SFRAME_V3_FLEX_FDE_OFFSET_REG_P(data)         ((data) & 0x1)

What about the following?

#define SFRAME_V3_FLEX_FDE_DATAWORD_REGNUM(data)      ((data) >> 3)
#define SFRAME_V3_FLEX_FDE_DATAWORD_DEREF_P(data)     (((data) >> 1) & 0x1)
#define SFRAME_V3_FLEX_FDE_DATAWORD_REG_P(data)       ((data) & 0x1)

Also a V3 version of FRE_RA_UNDEFINED_P() is missing.  What about:

#define SFRAME_V3_FRE_RA_UNDEFINED_P(data)        (SFRAME_V2_FRE_RA_UNDEFINED_P(data))

Regards,
Jens
-- 
Jens Remus
Linux on Z Development (D3303)
jremus@de.ibm.com / jremus@linux.ibm.com

IBM Deutschland Research & Development GmbH; Vorsitzender des Aufsichtsrats: Wolfgang Wendt; Geschäftsführung: David Faller; Sitz der Gesellschaft: Ehningen; Registergericht: Amtsgericht Stuttgart, HRB 243294
IBM Data Privacy Statement: https://www.ibm.com/privacy/