[llvm-commits] [PATCH 01/20] Add DenseMapInfo for std::string

[David A. Greene]

Eli Friedman <eli.friedman at gmail.com> writes:

>> Well, Chris suggested using DenseMap with std::string and this is all I
>> could come up with.  Originally I used a FoldingSet but several people
>> objected due to performance concerns.  So I'm a bit stuck.  :(
>
> If you must, make a custom TableGenStringKey class (my name not really
> suggested) containing an std::string, and implement DenseMapInfo on
> that.

Ok, that's safer.  Good idea.

                                    -Dave