]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
aarch64: s390: x86_64: sframe: adjust flex fde hook implementation
authorIndu Bhagat <indu.bhagat@oracle.com>
Tue, 20 Jan 2026 18:25:49 +0000 (10:25 -0800)
committerIndu Bhagat <indu.bhagat@oracle.com>
Wed, 21 Jan 2026 21:15:27 +0000 (13:15 -0800)
A previous commit added new backend hook sframe_support_flex_fde_p by
defining a new function for each backend that supports SFrame stack
trace format:

  commit 2f9b3987db53d7e0606f89bfe2527dd3d0915568
  [SFrame-V3] gas: sframe: add new backend hook
  sframe_support_flex_fde_p for FLEX FDEs

As pointed out in a review comment, simply providing the definition to
use true/false (as applicable) is sufficient for the purpose, and helps
generate better code.  So do that.
https://inbox.sourceware.org/binutils/80404871-53b4-4f5c-9d86-7a3a4d0a920e@suse.com/

ChangeLog:
* gas/config/tc-aarch64.c (aarch64_support_flex_fde_p): Remove.
* gas/config/tc-aarch64.h (aarch64_support_flex_fde_p): Remove.
(sframe_support_flex_fde_p): Define to false.
* gas/config/tc-i386.c (x86_support_flex_fde_p): Remove.
* gas/config/tc-i386.h (x86_support_flex_fde_p): Remove.
(sframe_support_flex_fde_p): Define to true.
* gas/config/tc-s390.c (s390_support_flex_fde_p): Remove.
* gas/config/tc-s390.h (s390_support_flex_fde_p): Remove.
(sframe_support_flex_fde_p): Define to true.

gas/config/tc-aarch64.c
gas/config/tc-aarch64.h
gas/config/tc-i386.c
gas/config/tc-i386.h
gas/config/tc-s390.c
gas/config/tc-s390.h

index 669e9620a78ec091968a34b55dc5cf41845100e6..9bde2250eeb12cb18db46b320c4371af9536b6a2 100644 (file)
@@ -9292,14 +9292,6 @@ aarch64_sframe_ra_tracking_p (void)
   return true;
 }
 
-/* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-
-bool
-aarch64_support_flex_fde_p (void)
-{
-  return false;
-}
-
 /* The fixed offset from CFA for SFrame to recover the return address.
    (useful only when SFrame RA tracking is not needed).  */
 
index 1b7cde747ac676bb69bbe0fce256f391ee694161..133604a8e0775a0f8e9346274e1b8e843add7d0a 100644 (file)
@@ -340,8 +340,7 @@ unsigned char aarch64_sframe_get_abi_arch (void);
 #define sframe_get_abi_arch aarch64_sframe_get_abi_arch
 
 /* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-extern bool aarch64_support_flex_fde_p (void);
-#define sframe_support_flex_fde_p aarch64_support_flex_fde_p
+#define sframe_support_flex_fde_p() false
 
 #endif /* OBJ_ELF  */
 
index 98f32049ffb8122ba5351751f81957d4aa4ecf6c..b317536cc602e6bf11aeb5666da92eb6d71d2a33 100644 (file)
@@ -12172,13 +12172,6 @@ x86_sframe_ra_tracking_p (void)
   return false;
 }
 
-/* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-bool
-x86_support_flex_fde_p (void)
-{
-  return true;
-}
-
 /* The fixed offset from CFA for SFrame to recover the return address.
    (useful only when SFrame RA tracking is not needed).  */
 offsetT
index 544064380df0f842e7442be31a76cfacd735e335..a362851a9eb10dd5d55992fb1937f7a3b513a1ee 100644 (file)
@@ -479,8 +479,7 @@ extern unsigned char x86_sframe_get_abi_arch (void);
 #define sframe_get_abi_arch x86_sframe_get_abi_arch
 
 /* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-extern bool x86_support_flex_fde_p (void);
-#define sframe_support_flex_fde_p x86_support_flex_fde_p
+#define sframe_support_flex_fde_p() true
 
 #endif
 
index 26a782baaa36cb30e32427924eebe5952ab0629c..4286574ce6ed5f704f2aae43a19b6b17ef6a26a4 100644 (file)
@@ -2917,14 +2917,6 @@ s390_sframe_ra_tracking_p (void)
   return true;
 }
 
-/* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-
-bool
-s390_support_flex_fde_p (void)
-{
-  return true;
-}
-
 /* Specify the fixed offset to recover RA from CFA.
    (useful only when RA tracking is not needed).  */
 
index 1511596a330f49117bee3d43ddc9a8d451e11b06..54f51237e8ac076303d595cc78ed5e6ab88bff51 100644 (file)
@@ -132,8 +132,7 @@ unsigned char s390_sframe_get_abi_arch (void);
 #define sframe_get_abi_arch s390_sframe_get_abi_arch
 
 /* Whether SFrame FDE of type SFRAME_FDE_TYPE_FLEX be generated.  */
-extern bool s390_support_flex_fde_p (void);
-#define sframe_support_flex_fde_p s390_support_flex_fde_p
+#define sframe_support_flex_fde_p() true
 
 /* The target supports Object Attributes.  */
 #define TC_OBJ_ATTR 1