[PATCH v4 3/4] gas: sframe: Represent .cfi_undefined RA as FRE without offsets

[Jens Remus]

Hello Indu!

On 9/2/2025 12:35 AM, Indu Bhagat via Binutils wrote:
> (Closing the open ends on this patch series..)

Thanks!  Slowly picking back up from where I left off before my vacation.

> Yes thats what I meant: single FRE with ra_undefined_p set for the
> address range from the PC where DW_CFA_undefined RA and onwards until
> the end PC of FDE.
> 
> I think we both agree that the other fields (cfa base reg id,
> RA mangled bit) are not useful when ra_undefined_p is set, and
> resetting them to a ground state is a good thing to do.  Your patch
> was already attempting to do that for the FRE (representing
> DW_CFA_undefined RA).
> 
> But in light of keeping SFrame flexible like DWARF in regard to
> representing cfi_undefined RA, your approach is more aligned.
> 
> Lets go with your V5 and move forward.

I'll incorporate your V5 review feedback and send a V6.

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/