[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