[lld] a3622ac - [wasm-ld] Switch to xxh3_64bits

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 10:47:53 PDT 2023


Author: Fangrui Song
Date: 2023-07-20T10:47:47-07:00
New Revision: a3622ac80be1230a96597a4efb8489d065b0ffc2

URL: https://github.com/llvm/llvm-project/commit/a3622ac80be1230a96597a4efb8489d065b0ffc2
DIFF: https://github.com/llvm/llvm-project/commit/a3622ac80be1230a96597a4efb8489d065b0ffc2.diff

LOG: [wasm-ld] Switch to xxh3_64bits

Similar to recent changes to ELF (e.g., D154813), Mach-O, and COFF to
improve hashing performance.

Reviewed By: dschuff

Differential Revision: https://reviews.llvm.org/D155752

Added: 
    

Modified: 
    lld/test/wasm/build-id.test
    lld/wasm/InputChunks.cpp
    lld/wasm/Writer.cpp

Removed: 
    


################################################################################
diff  --git a/lld/test/wasm/build-id.test b/lld/test/wasm/build-id.test
index a15f4cedcdddda..c5c3cfd737f01b 100644
--- a/lld/test/wasm/build-id.test
+++ b/lld/test/wasm/build-id.test
@@ -43,8 +43,8 @@ foo:
 
 
 # DEFAULT:      Contents of section build_id:
-# DEFAULT-NEXT: 0000 10cdbf99 f76b1f5e ebb2f36a 1bde1d6c  .....k.^...j...l
-# DEFAULT-NEXT: 0010 01
+# DEFAULT-NEXT: 0000 100e228e 4e2fa853 6393b43d ed1d4676
+# DEFAULT-NEXT: 0010 13 .
 
 # SHA1:      Contents of section build_id:
 # SHA1-NEXT: 0000 14ad22e8 54d72438 94af85de 3c5592bd  ..".T.$8....<U..

diff  --git a/lld/wasm/InputChunks.cpp b/lld/wasm/InputChunks.cpp
index a22005fe5f5a7e..f55f54a9509c79 100644
--- a/lld/wasm/InputChunks.cpp
+++ b/lld/wasm/InputChunks.cpp
@@ -451,7 +451,7 @@ void MergeInputChunk::splitStrings(ArrayRef<uint8_t> data) {
       fatal(toString(this) + ": string is not null terminated");
     size_t size = end + 1;
 
-    pieces.emplace_back(off, xxHash64(s.substr(0, size)), true);
+    pieces.emplace_back(off, xxh3_64bits(s.substr(0, size)), true);
     s = s.substr(size);
     off += size;
   }

diff  --git a/lld/wasm/Writer.cpp b/lld/wasm/Writer.cpp
index 7b33a944962a63..25a9c15957f455 100644
--- a/lld/wasm/Writer.cpp
+++ b/lld/wasm/Writer.cpp
@@ -290,7 +290,7 @@ void Writer::writeBuildId() {
   case BuildIdKind::Fast: {
     std::vector<uint8_t> fileHash(8);
     computeHash(fileHash, buf, [](uint8_t *dest, ArrayRef<uint8_t> arr) {
-      support::endian::write64le(dest, xxHash64(arr));
+      support::endian::write64le(dest, xxh3_64bits(arr));
     });
     makeUUID(5, fileHash, buildId);
     break;


        


More information about the llvm-commits mailing list