[PATCH] D102943: Hashing: use a 64-bit storage type on all platforms.

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 27 17:44:48 PDT 2021


dexonsmith added a comment.

In D102943#2786015 <https://reviews.llvm.org/D102943#2786015>, @arames wrote:

> This new version is an attempt to have modules not rely on `llvm::hash_code`, but on a new `llvm::stable_hash_code`.
> I understand modifying `ADT/Hashing.h` is sensitive, so maybe we need to discuss the high-level approach first.

Just seeing the update here, but we already chatted offline. @arames is going to try using llvm::MD5, which is used elsewhere for a stable hash in modules (for the AST signature), rather than introducing `stable_hash_*`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102943



More information about the llvm-commits mailing list