[ADT] Remove DenseMapBase::getHashValue (NFC)

[kazutakahirata]

This patch removes:

static unsigned getHashValue(const KeyT &Val) {
return KeyInfoT::getHashValue(Val);
}

This function is redundant given the templated overload:

template
static unsigned getHashValue(const LookupKeyT &Val) {
return KeyInfoT::getHashValue(Val);
}

Note that the callers doFind and LookupBucketFor are themselves
templated on LookupKeyT.

[llvmbot]

@llvm/pr-subscribers-llvm-adt

Author: Kazu Hirata (kazutakahirata)

Changes

This patch removes:

static unsigned getHashValue(const KeyT &Val) {
return KeyInfoT::getHashValue(Val);
}

This function is redundant given the templated overload:

template <typename LookupKeyT>
static unsigned getHashValue(const LookupKeyT &Val) {
return KeyInfoT::getHashValue(Val);
}

Note that the callers doFind and LookupBucketFor are themselves
templated on LookupKeyT.

---

Full diff: https://github.com/llvm/llvm-project/pull/161123.diff

1 Files Affected:

• (modified) llvm/include/llvm/ADT/DenseMap.h (-4)

diff --git a/llvm/include/llvm/ADT/DenseMap.h b/llvm/include/llvm/ADT/DenseMap.h
index bcf3e9676a7b5..4bda50f5a5cc0 100644
--- a/llvm/include/llvm/ADT/DenseMap.h
+++ b/llvm/include/llvm/ADT/DenseMap.h
@@ -440,10 +440,6 @@ class DenseMapBase : public DebugEpochBase {
     }
   }
 
-  static unsigned getHashValue(const KeyT &Val) {
-    return KeyInfoT::getHashValue(Val);
-  }
-
   template <typename LookupKeyT>
   static unsigned getHashValue(const LookupKeyT &Val) {
     return KeyInfoT::getHashValue(Val);