[llvm-commits] [PATCH 01/20] Add DenseMapInfo for std::string
Eli Friedman
eli.friedman at gmail.com
Tue Jul 19 15:08:23 PDT 2011
On Tue, Jul 19, 2011 at 2:11 PM, David A. Greene <greened at obbligato.org> wrote:
> Eli Friedman <eli.friedman at gmail.com> writes:
>
>> On Tue, Jul 19, 2011 at 1:11 PM, David Greene <dag at cray.com> wrote:
>> This is unsafe in general; TableGen might never see the string
>> "<<<EMPTY KEY>>>", but you can't assume arbitrary code using this
>> header will not see it.
>
> True, but it's highly unlikely. Do you have a better alternative? I
> certainly would like to do this the Right Way.
Use std::map? It's not like performance matters here.
-Eli
More information about the llvm-commits
mailing list