[PATCH] D102943: [modules] Use `HashBuilder` and `MD5` for the module hash.

Alexandre Rames via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 30 10:48:23 PDT 2021


arames added inline comments.


================
Comment at: clang/include/clang/Basic/ObjCRuntime.h:486
+  template <typename HasherT, llvm::support::endianness Endianness>
+  friend void addHash(llvm::HashBuilderImpl<HasherT, Endianness> &HBuilder,
+                      const ObjCRuntime &OCR) {
----------------
I have added these in the same line as existing `hash_value` functions. The idea being others may also need to hash those objects.
Let me know if you would rather move some/all of these locally in `CompilerInvocation.cpp`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102943/new/

https://reviews.llvm.org/D102943



More information about the cfe-commits mailing list