Re: [PATCH v2 0/2] gas: sframe: Fix for flexible FDE and new test

Jens Remus <[email protected]>
Newsgroups gmane.comp.gnu.binutils
Organization IBM Deutschland Research & Development GmbH
Message-ID <[email protected]>
On 8/11/2026 10:45 PM, Indu Bhagat wrote:
> On 2026-08-07 07:35, Jens Remus wrote:
>> For changes in v2 see individual patch notes.
>>
>> Patch 1 fixes the generation of SFrame if a target supports SFrame V3
>> flexible FDE.
>>
>> Patch 2 adds a test for the issue Nick fixed with commit fa11363bc9ad
>> ("Ensure that sframe_xlate_ctx_cleanup() doe snot leave any dangling
>> pointers").
>>
>> Regards,
>> Jens
>>
> 
> LGTM, Thanks

Thank you for the review!  Committed to mainline.

>> Jens Remus (2):
>>    gas: sframe: Fix non-SP/FP CFA base register if flexible FDE
>>    gas: sframe: Add test for signal frame with unsupported CFI
>>
>>   gas/gen-sframe.c                              | 26 +++++++------------
>>   .../gas/cfi-sframe/cfi-sframe-common-14.d     | 20 ++++++++++++++
>>   .../gas/cfi-sframe/cfi-sframe-common-14.s     |  5 ++++
>>   gas/testsuite/gas/cfi-sframe/cfi-sframe.exp   |  1 +
>>   4 files changed, 35 insertions(+), 17 deletions(-)
>>   create mode 100644 gas/testsuite/gas/cfi-sframe/cfi-sframe-common-14.d
>>   create mode 100644 gas/testsuite/gas/cfi-sframe/cfi-sframe-common-14.s

Regards,
Jens
-- 
Jens Remus
Linux on Z Development (D3303)
[email protected] / [email protected]

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/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.