[PATCH] D25644: [ADT] Move CachedHashString to a own header in ADT, and rename to CachedHashStringRef.

[Rafael Avila de Espindola via llvm-commits]

This also changes some bits of the implementation, like comparing the hash in isequal. Could  you make this do just the move and rename?


LGTM if the implementation is not changed in this patch.

On October 15, 2016 6:07:22 PM EDT, 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.
>
>Reclaiming the name 'CachedHashString' will let us add a type with that
>name that owns its value.
>
>
>https://reviews.llvm.org/D25644
>
>Files:
>  lld/ELF/InputSection.cpp
>  lld/ELF/OutputSections.cpp
>  lld/ELF/OutputSections.h
>  llvm/include/llvm/ADT/CachedHashString.h
>  llvm/include/llvm/MC/StringTableBuilder.h
>  llvm/lib/MC/StringTableBuilder.cpp
>  llvm/unittests/MC/StringTableBuilderTest.cpp
>
>
>
>------------------------------------------------------------------------
>
>_______________________________________________
>llvm-commits mailing list
>llvm-commits at lists.llvm.org
>http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161016/5b40ec15/attachment.html>