[clang] [AArch64][PAC] Support ptrauth builtins and -fptrauth-intrinsics. (PR #65996)

Chuanqi Xu via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 12 18:47:32 PDT 2023


================
@@ -303,6 +303,10 @@ bool Module::directlyUses(const Module *Requested) {
   if (!Requested->Parent && Requested->Name == "_Builtin_stddef_max_align_t")
     return true;
 
+  // Anyone is allowed to use our builtin ptrauth.h and its accompanying module.
+  if (!Requested->Parent && Requested->Name == "ptrauth")
----------------
ChuanqiXu9 wrote:

But this is still dirty and not good. We should try to find a better way to implement the semantics. Also I am wondering if this change is really necessary to the current patch. I mean, what if we drop the modules related change in the current patch and add it later in another patch?

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


More information about the cfe-commits mailing list