================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to