[clang] 3487533 - [PAC][compiler-rt] Use `__ptrauth` qualifier instead of builtins (#188109)
via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 24 02:24:26 PDT 2026
Author: Daniil Kovalev
Date: 2026-03-24T09:24:19Z
New Revision: 34875331d956862d6f5b750c0bcf026d40564671
URL: https://github.com/llvm/llvm-project/commit/34875331d956862d6f5b750c0bcf026d40564671
DIFF: https://github.com/llvm/llvm-project/commit/34875331d956862d6f5b750c0bcf026d40564671.diff
LOG: [PAC][compiler-rt] Use `__ptrauth` qualifier instead of builtins (#188109)
Since #100830 has landed, we no longer need to rely on builtins
Added:
Modified:
clang/lib/Headers/ptrauth.h
compiler-rt/lib/builtins/crtbegin.c
Removed:
################################################################################
diff --git a/clang/lib/Headers/ptrauth.h b/clang/lib/Headers/ptrauth.h
index af94168c35fb7..cde1b3c4ebbe2 100644
--- a/clang/lib/Headers/ptrauth.h
+++ b/clang/lib/Headers/ptrauth.h
@@ -349,6 +349,14 @@ typedef __UINTPTR_TYPE__ ptrauth_generic_signature_t;
__ptrauth(ptrauth_key_objc_class_ro_pointer, 1, \
__ptrauth_objc_class_ro_discriminator)
+#if __has_feature(ptrauth_init_fini_address_discrimination)
+#define __ptrauth_init_fini_pointer \
+ __ptrauth(ptrauth_key_init_fini_pointer, 1, __ptrauth_init_fini_discriminator)
+#else
+#define __ptrauth_init_fini_pointer \
+ __ptrauth(ptrauth_key_init_fini_pointer, 0, __ptrauth_init_fini_discriminator)
+#endif
+
#else
#define ptrauth_strip(__value, __key) \
diff --git a/compiler-rt/lib/builtins/crtbegin.c b/compiler-rt/lib/builtins/crtbegin.c
index 8b5f98fdd04ed..74becdbe5dd0e 100644
--- a/compiler-rt/lib/builtins/crtbegin.c
+++ b/compiler-rt/lib/builtins/crtbegin.c
@@ -55,17 +55,9 @@ static void __attribute__((used)) __do_init(void) {
#ifdef CRT_HAS_INITFINI_ARRAY
# if __has_feature(ptrauth_init_fini)
-// TODO: use __ptrauth-qualified pointers when they are supported on clang side
-# if __has_feature(ptrauth_init_fini_address_discrimination)
-__attribute__((section(".init_array"), used)) static void *__init =
- ptrauth_sign_constant(&__do_init, ptrauth_key_init_fini_pointer,
- ptrauth_blend_discriminator(
- &__init, __ptrauth_init_fini_discriminator));
-# else
-__attribute__((section(".init_array"), used)) static void *__init =
- ptrauth_sign_constant(&__do_init, ptrauth_key_init_fini_pointer,
- __ptrauth_init_fini_discriminator);
-# endif
+__attribute__((section(".init_array"),
+ used)) static void *__ptrauth_init_fini_pointer __init =
+ __do_init;
# elif __has_feature(ptrauth_calls)
# ifdef __aarch64__
// If ptrauth_init_fini feature is not present, compiler emits raw unsigned
@@ -137,17 +129,9 @@ static void __attribute__((used)) __do_fini(void) {
#ifdef CRT_HAS_INITFINI_ARRAY
# if __has_feature(ptrauth_init_fini)
-// TODO: use __ptrauth-qualified pointers when they are supported on clang side
-# if __has_feature(ptrauth_init_fini_address_discrimination)
-__attribute__((section(".fini_array"), used)) static void *__fini =
- ptrauth_sign_constant(&__do_fini, ptrauth_key_init_fini_pointer,
- ptrauth_blend_discriminator(
- &__fini, __ptrauth_init_fini_discriminator));
-# else
-__attribute__((section(".fini_array"), used)) static void *__fini =
- ptrauth_sign_constant(&__do_fini, ptrauth_key_init_fini_pointer,
- __ptrauth_init_fini_discriminator);
-# endif
+__attribute__((section(".fini_array"),
+ used)) static void *__ptrauth_init_fini_pointer __fini =
+ __do_fini;
# elif __has_feature(ptrauth_calls)
# ifdef __aarch64__
// If ptrauth_init_fini feature is not present, compiler emits raw unsigned
More information about the cfe-commits
mailing list