[PATCH] D25645: [ADT] Add CachedHashString.
Rafael Espíndola via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 19 05:26:25 PDT 2016
On 17 October 2016 at 18:25, Justin Lebar <jlebar at google.com> wrote:
> jlebar added a comment.
> Rafael, Tim and I were looking for a second opinion -- does this seem to you like a sane thing to do? The immediate problem I'm trying to solve here is to convert code that's using SmallSetVector<std::string>, which doesn't work after converting SmallSetVector to use DenseSet, because you can't store an std::string in a DenseSet.
I hit that very same problem some time ago and dropped it. So yes, I
this this is sane.
Thanks for working on it.
More information about the llvm-commits