[PATCH] D18506: Add a copy constructor to StringMap

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 20:59:55 PDT 2016


hfinkel added inline comments.

================
Comment at: include/llvm/ADT/StringMap.h:94
@@ -93,2 +93,3 @@
+  /// setup the map as empty.
   void init(unsigned Size);
 
----------------
joker.eph wrote:
> private -> public is a bit rough, could it be protected maybe?
Yes, it is (all of the functions in this block are protected).


http://reviews.llvm.org/D18506





More information about the llvm-commits mailing list