[PATCH] D117858: [docs] Refine the description in Set-Like and Map-Like container options.

Hsiangkai Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 23 23:50:32 PST 2022


HsiangKai added inline comments.


================
Comment at: llvm/docs/ProgrammersManual.rst:2201-2205
 The STL provides several other options, such as std::multiset and the various
 "hash_set" like containers (whether from C++ TR1 or from the SGI library).  We
-never use hash_set and unordered_set because they are generally very expensive
-(each insertion requires a malloc) and very non-portable.
+never use "hash_set" like containers, e.g., unordered_set, because they are
+generally very expensive (each insertion requires a malloc) and very
+non-portable.
----------------
dexonsmith wrote:
> See the proposed edit, which:
> - Drops the references to C++ TR1 and SGI and `hash_set`.
> - Drops the worry about portability (this was a problem with `hash_set`, but `std::unordered_set` has worked portably since LLVM started depending on C++11).
It looks good. Thanks.
I will modify it according to your suggestions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117858/new/

https://reviews.llvm.org/D117858



More information about the llvm-commits mailing list