[PATCH] D49877: ELF: Make --print-icf-sections output deterministic.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 26 15:23:12 PDT 2018


pcc added a comment.

In https://reviews.llvm.org/D49877#1177432, @ruiu wrote:

> I think we should use a deterministic hash function such as xxHash64 instead of hash_combine to fix this issue, as I believe it's easier and more robust.


So hashing the section contents instead of contents + flags etc like we're currently doing? Seems reasonable, I'll give that a try.


Repository:
  rL LLVM

https://reviews.llvm.org/D49877





More information about the llvm-commits mailing list