[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
Fri May 21 18:20:54 PDT 2021
dexonsmith added a subscriber: jansvoboda11.
dexonsmith added a comment.
In D102943#2775115 <https://reviews.llvm.org/D102943#2775115>, @pcc wrote:
> Isn't the bug here that module hashing is using `hash_code`? So shouldn't the correct fix be to use a specific hashing algorithm for module hashes?
Yeah, I tend to agree. I thought modules already used MD5, but maybe just for the AST signature.
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