[PATCH v2 05/11] s390: Represent FP without RA saved in SFrame

Jens Remus jremus@linux.ibm.com
Tue Jun 3 14:31:10 GMT 2025


On 02.06.2025 22:36, Indu Bhagat via Binutils wrote:
> On 5/27/25 4:07 AM, Jens Remus wrote:

>> diff --git a/include/sframe.h b/include/sframe.h
>> index 393748bcd2ad..5ad007f6ccb0 100644
>> --- a/include/sframe.h
>> +++ b/include/sframe.h
>> @@ -237,6 +237,9 @@ typedef struct sframe_func_desc_entry
>>      may or may not be tracked.  */
>>   #define SFRAME_FRE_FP_OFFSET_IDX    2
>>   +/* Invalid RA offset.  Used as padding to represent FP without RA on stack.  */
> 
> How about add a followup statement/comment, something like:
>   "Currently used for s390x.  */
> 
> For documentation purposes.

Ok.

> 
>> +#define SFRAME_FRE_RA_OFFSET_INVALID 0
>> +
>>   typedef struct sframe_fre_info
>>   {
>>     /* Information about

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/



More information about the Binutils mailing list