[PATCH,V2 2/2] gas: sframe: partially process DWARF expressions in CFI_escape

[Jens Remus]

On 05.02.2025 20:59, Indu Bhagat wrote:
> On 2/5/25 8:31 AM, Jens Remus wrote:
>> On 05.02.2025 00:10, Indu Bhagat wrote:
>>
>>>   - Restructure functionality to accommodate the fact that a valid
>>>     unwind info may be split across multiple .cfi_escape. If this is the
>>>     case, the current implementation bails out.  Handling this case does
>>>     not appear to be worth the effort.
>>
>> Is that worth to be mentioned in the commit message?  How is multiple
>> unwind info within one .cfi_escape handled?
>>
> 
> I remove the stubs between [Changes in VN] and [End of changes in VN] from the commit log before pushing. Admittedly this has caused confusion previously too...

Aww, sorry, I meant it would make sense to mention above and below in
the commit message.  That is that neither CFI instructions split across
multiple .cfi_escape nor multiple CFI instructions encoded in a single
.cfi_escape are handled and the SFrame FDE is skipped.

> Multiple unwind info within one .cfi_escape is not handled.  We skip generating FDE and warn the user.

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/