[PATCH] D102943: Hashing: use a 64-bit storage type on all platforms.
Duncan P. N. Exon Smith via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list