[clang] [PAC] Add support for __ptrauth type qualifier (PR #100830)

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 10 05:57:41 PDT 2025


================
@@ -104,6 +104,7 @@ FEATURE(thread_sanitizer, LangOpts.Sanitize.has(SanitizerKind::Thread))
 FEATURE(dataflow_sanitizer, LangOpts.Sanitize.has(SanitizerKind::DataFlow))
 FEATURE(scudo, LangOpts.Sanitize.hasOneOf(SanitizerKind::Scudo))
 FEATURE(ptrauth_intrinsics, LangOpts.PointerAuthIntrinsics)
+FEATURE(ptrauth_qualifier, LangOpts.PointerAuthIntrinsics)
----------------
AaronBallman wrote:

It does not, see the top of `Features.def`:
```
// FEATURE(...) should be used to advertise support for standard language
// features, whereas EXTENSION(...) should be used for clang extensions. Note
// that many of the identifiers in this file don't follow this rule for backward
// compatibility reasons.
```
Those backwards compatibility reasons are from a time when we only had one way to feature test macros. I think those internal projects should migrate to the correct usage instead of adding this as `FEATURE` for all uses moving forward.

https://github.com/llvm/llvm-project/pull/100830


More information about the cfe-commits mailing list