[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