[llvm] [ADT] Fix the initial size calculation of SmallDenseMap (PR #158458)

Jakub Kuderski via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 14 04:54:33 PDT 2025


================
@@ -887,11 +887,7 @@ class SmallDenseMap
   AlignedCharArrayUnion<BucketT[InlineBuckets], LargeRep> storage;
 
 public:
-  explicit SmallDenseMap(unsigned NumInitBuckets = 0) {
-    if (NumInitBuckets > InlineBuckets)
-      NumInitBuckets = llvm::bit_ceil(NumInitBuckets);
-    init(NumInitBuckets);
-  }
+  explicit SmallDenseMap(unsigned InitialReserve = 0) { init(InitialReserve); }
----------------
kuhar wrote:

Maybe make the unit explicit with `NumElementsToReservre`?

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


More information about the llvm-commits mailing list