[llvm] [BOLT] Provide backwards compatibility for YAML profile with std::hash (PR #74253)
    Maksim Panchenko via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun Dec 10 17:09:05 PST 2023
    
    
  
================
@@ -75,6 +75,13 @@ enum IndirectCallPromotionType : char {
   ICP_ALL          /// Perform ICP on calls and jump tables.
 };
 
+/// Hash functions supported for BF/BB hashing.
+enum class HashFunction : char {
+  StdHash, /// std::hash, implementation is platform-dependent. Provided for
+           /// backwards compatibility.
+  Xxh3,    /// llvm::xxh3_64bits, the default.
----------------
maksfb wrote:
Tiny nit: `s/Xxh3/XXH3/`. If the name is all caps, keep it all caps or lower all letters. Not sure if such rule is listed in LLVM docs.
https://github.com/llvm/llvm-project/pull/74253
    
    
More information about the llvm-commits
mailing list