[clang] [PAC][clang] Define `PointerAuthQualifier` and `PointerAuthenticationMode` (PR #84384)

Daniil Kovalev via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 16 10:29:30 PDT 2024


================
@@ -607,13 +799,17 @@ class Qualifiers {
 
   void Profile(llvm::FoldingSetNodeID &ID) const {
     ID.AddInteger(Mask);
+    PtrAuth.Profile(ID);
   }
 
 private:
   // bits:     |0 1 2|3|4 .. 5|6  ..  8|9   ...   31|
   //           |C R V|U|GCAttr|Lifetime|AddressSpace|
   uint32_t Mask = 0;
 
+  PointerAuthQualifier PtrAuth;
----------------
kovdan01 wrote:

That was used for at least several months in downstream and I'm not aware of compile time issues it caused. But, as you've mentioned, the changes can lead to undesirable impact in surprising places, so it's better to test it anyway. I'll do that and let you know about results.

Do I get it right that I need to contact nikic to be able to test branches from my fork (https://llvm-compile-time-tracker.com/about.php)? When requesting smth on the compare page (https://llvm-compile-time-tracker.com/compare.php), only commits and branches from this repository seem to be allowed (while I want to use the branch from my fork).


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


More information about the cfe-commits mailing list