[PATCH] D67249: [Modules][PCH] Hash input files content

Bruno Cardoso Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 13 12:51:30 PDT 2019


bruno added a comment.



> Did you try xxHash64?

No, and I'm not planning to. I believe `hash_code` is enough here, already has low overhead on performance and size, and bunch of prior use elsewhere in LLVM/Clang. If there's a strong reason to do it I wanna know why that's the case first.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67249





More information about the cfe-commits mailing list