[V2 21/36] [SFrame-V3] sframe: gas: s390: aarch64: x86: add new backend hooks for FLEX FDE

[Indu Bhagat]

On 1/7/26 5:44 AM, Jens Remus wrote:
> On 1/7/2026 9:42 AM, Indu Bhagat wrote:
>> For the newly added SFrame FDE type SFRAME_FDE_TYPE_FLEX, add a new
>> backend hook so that the respective ABI/arch can opt out of the
>> generation of SFRAME_FDE_TYPE_FLEX in GAS.
>>
>> AArch64 is an example of an ABI for which SFrame is supported, but one
>> that does not need the flexible FDE representation for any of the
>> current usecases currently.
>>
>> TBD:
>>    - split out into arch specific patches ?
> 
> Wouldn't that cause compile errors unless you default
> sframe_support_flex_fde_p to false if it is undefined?
> 

No, unless I misunderstood you.  It will effectively be this patch split 
into three patches, one each for x86_64, s390x, aarch64.

But on second thought, I think splitting into three is not quite 
necessary for review/approval.

> Maybe change the commit title to:
> 
> gas: sframe: add new backend hook sframe_support_flex_fde_p for FLEX FDEs
> 

Sure.

>>
>> gas/
>> 	* config/tc-aarch64.c (aarch64_support_flex_fde_p): New
>> 	definition.
>> 	* config/tc-aarch64.h (aarch64_support_flex_fde_p): New
>> 	declaration.
>> 	(sframe_support_flex_fde_p): Define.
>> 	* config/tc-i386.c (x86_support_flex_fde_p): New definition.
>> 	* config/tc-i386.h (x86_support_flex_fde_p): New declaration.
>> 	(sframe_support_flex_fde_p): Define.
>> 	* config/tc-s390.c (s390_support_flex_fde_p): New definition.
>> 	* config/tc-s390.h (s390_support_flex_fde_p): New declaration.
>> 	(sframe_support_flex_fde_p): Define.
> 
> Reviewed-by: Jens Remus <jremus@linux.ibm.com>
> 
> Regards,
> Jens