[PATCH] D25630: [ADT] Remove CachedHash, which is unused.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 13:06:58 PDT 2016


I see CachedHashString in StringTableBuilder.h, but not CachedHash<T>.

Am I missing something?  My tree builds fine with this change, but
that doesn't really prove anything.

On Fri, Oct 14, 2016 at 12:52 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> It is used in llvm/include/llvm/MC/StringTableBuilder.h.
>
> Cheers,
> Rafael
>
>
> On 14 October 2016 at 14:06, Justin Lebar via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>> jlebar created this revision.
>> jlebar added a reviewer: timshen.
>> jlebar added a subscriber: llvm-commits.
>>
>> https://reviews.llvm.org/D25630
>>
>> Files:
>>   llvm/include/llvm/ADT/DenseMapInfo.h
>>   llvm/unittests/ADT/DenseMapTest.cpp
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>


More information about the llvm-commits mailing list