[PATCH 0/3] sframe: Cleanups an documentation improvement

[Jens Remus]

On 04.02.2025 15:05, Indu Bhagat wrote:
> On 2/3/25 5:57 AM, Jens Remus wrote:
>> Patch 1 cleans up the SFrame generation from CFI directives to use the
>> appropriate struct cfi_insn_data union members when accessing the CFI
>> instruction data.
>>
>> Patch 2 skips SFrame FDE if .cfi_val_offset specifies the SP register
>> with a non-default value offset.  That is a value offset which does not
>> match with the implicitly assumed rule to unwind the SP register from
>> the CFA (e.g. SP = CFA + 0 on AMD64, and AArch64).
>>
>> Patch 3 clarifies the description of the SFrame FDE function start
>> address and SFrame FRE PC range start address fields.  Especially that
>> the function start address in the FDE is an offset from SFrame section.

...

>> Jens Remus (3):
>>    gas: sframe: Use appropriate struct cfi_insn_data union members
>>    gas: Skip SFrame FDE if .cfi_val_offset specifies non-default offset
>>    doc: sframe: Clarify FDE/FRE function/range start address fields
>>
> 
> Reviewed-by: Indu Bhagat <indu.bhagat@oracle.com>

...

>>   gas/gen-sframe.c               | 28 ++++++++++++++--------------
>>   libsframe/doc/sframe-spec.texi | 10 ++++++----
>>   2 files changed, 20 insertions(+), 18 deletions(-)

Thanks! Committed to mainline.

Regards,
Jens
-- 
Jens Remus
Linux on Z Development (D3303)
+49-7031-16-1128 Office
jremus@de.ibm.com

IBM

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