[PATCH] D106910: [Support]: Introduce the `HashBuilder` interface.
Alexandre Rames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 3 14:00:11 PDT 2021
arames updated this revision to Diff 363876.
arames marked 14 inline comments as done.
arames added a comment.
Address review comments:
Main changes:
- Switch dependency order: `HashBuilder` now depends on `ArrayRef` and `StringRef`
- Clean up `updateRange` and `updateRangeImpl`
- Fix case style
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106910/new/
https://reviews.llvm.org/D106910
Files:
llvm/include/llvm/Support/HashBuilder.h
llvm/unittests/Support/CMakeLists.txt
llvm/unittests/Support/HashBuilderTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106910.363876.patch
Type: text/x-patch
Size: 17521 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210803/47af95d8/attachment.bin>
More information about the llvm-commits
mailing list