[PATCH] D30555: LTO: Hash type identifier resolutions for WholeProgramDevirt.

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 18:38:21 PST 2017


mehdi_amini added inline comments.


================
Comment at: llvm/lib/LTO/LTO.cpp:104
+    Hasher.update(ArrayRef<uint8_t>{Data, 8});
+  };
   AddString(Conf.CPU);
----------------
Why do we hash bit-by-bit? Is it because of endianness? Do we care about having the same hash on different endian machine?


https://reviews.llvm.org/D30555





More information about the llvm-commits mailing list