[PATCH] D20930: Adding reserve and capacity methods to FoldingSet

Anna Zaks via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 17:21:18 PDT 2016


zaks.anna added a subscriber: zaks.anna.
zaks.anna added a comment.

Ben,

It would be a bit easier to review if you upload the diff with full context.

The patch LGTM.


================
Comment at: include/llvm/ADT/FoldingSet.h:184
@@ +183,3 @@
+  void reserve(unsigned EltCount);
+  unsigned capacity() {
+    // We allow a load factor of up to 2.0,
----------------
Please, add a comment describing the capacity() API.


http://reviews.llvm.org/D20930





More information about the llvm-commits mailing list