[PATCH, V2 11/14] libsframe: refactor sframe_decoder_add_funcdesc for internal use

[Jens Remus]

On 12/9/2025 9:59 AM, Indu Bhagat via Binutils wrote:
> sframe_encoder_add_funcdesc () was added for SFRAME_VERSION_1.  This has
> since been obsoleted by introduction of SFRAME_VERSION_2 and its
> corresponding sframe_decoder_add_funcdesc_v2 API.
> 
> Refactor the functionality into an internal-only API:
> sframe_encoder_add_funcdesc_internal (). Ensure it returns the error
> code for the caller to take necessary action or pass to user.
> 
> Keep only two args for sframe_encoder_add_funcdesc: function size and
> function start addr.  This simple barebone API will be used in a
> subsequent commit to adjust the link-time behaviour of SFrame sections.
> 
> libsframe/
> 	* sframe.c (sframe_encoder_add_funcdesc): Refactor out into
> 	sframe_encoder_add_funcdesc_internal.  Change args.
> 	(sframe_encoder_add_funcdesc_v2): Use the new internal API.
> 
> ---
> [Changes in V2]
>   - Use only two args for the sframe_encoder_add_funcdesc_internal
> [End of changes in V2]
> ---
>  include/sframe-api.h |  8 +++----
>  libsframe/sframe.c   | 50 ++++++++++++++++++++++++++------------------
>  2 files changed, 33 insertions(+), 25 deletions(-)

Reviewed-by: Jens Remus <jremus@linux.ibm.com>

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/