[PATCH] D25645: [ADT] Add CachedHashString.
Tim Shen via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 17 13:50:42 PDT 2016
timshen added inline comments.
Comment at: llvm/include/llvm/ADT/CachedHashString.h:94
+ // TODO: Use small-string optimization to avoid allocating.
> It is unfortunate to implement some of the std::string functionalities from scratch. Is there a way to implement it in terms of std::string?
More specifically, I'm asking for a member function `string& val();` that returns the underlying std::string object (it needs to exist first), so that CachedHashString compose better with other APIs that don't care about hashing.
More information about the llvm-commits