[PATCH] D46855: [pdb] Change ghash section to use 8-byte ghashes instead of 20 byte

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 15:53:00 PDT 2018


zturner created this revision.
zturner added reviewers: rnk, ruiu.
Herald added a subscriber: hiraditya.

The previous algorithm used 20-byte SHA1 hashes.  This new version computes the same 20-byte SHA1, but only uses the last 8 bytes as a hash.

This speeds up /DEBUG:GHASH links by ~10% in my testing while also making object files smaller.


https://reviews.llvm.org/D46855

Files:
  lld/COFF/PDB.cpp
  lld/test/COFF/Inputs/pdb-hashes-1.yaml
  lld/test/COFF/Inputs/pdb-hashes-2.yaml
  llvm/include/llvm/DebugInfo/CodeView/TypeHashing.h
  llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
  llvm/lib/DebugInfo/CodeView/TypeHashing.cpp
  llvm/lib/ObjectYAML/CodeViewYAMLTypeHashing.cpp
  llvm/test/DebugInfo/COFF/global-type-hashes.ll
  llvm/test/DebugInfo/PDB/obj-globalhash.test
  llvm/test/ObjectYAML/CodeView/sections.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D46855.146712.patch
Type: text/x-patch
Size: 31068 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180514/579ba44a/attachment.bin>


More information about the llvm-commits mailing list