commit 5842a56cb02050d32708940db1476d58c5609e0b
author Nico Weber <nicolasweber@gmx.de> Sat Sep 15 18:37:22 2018 +0000
committer Nico Weber <nicolasweber@gmx.de> Sat Sep 15 18:37:22 2018 +0000
tree c713a6c2c19f68c14d6a01d38583c95f63213f2d
parent dda930c4ebce41d12ec8bc148b8189f28595deca

lld-link: Set PDB GUID to hash of PDB contents instead of to a random byte sequence.

Previously, lld-link would use a random byte sequence as the PDB GUID. Instead,
use a hash of the PDB file contents.

To not disturb llvm-pdbutil pdb2yaml, the hash generation is an opt-in feature
on InfoStreamBuilder and ldb/COFF/PDB.cpp always sets it.

Since writing the PDB computes this ID which also goes in the exe, the PDB
writing code now must be called before writeBuildId(). writeBuildId() for that
reason is no longer included in the "Code Layout" timer.

Since the PDB GUID is now a function of the PDB contents, the PDB Age is always
set to 1. There was a long comment above loadExistingBuildId (now gone) about
how not changing the GUID and only incrementing the age was important, but
according to the discussion in PR35914 that comment was incorrect.

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


git-svn-id: https://llvm.org/svn/llvm-project/lld/trunk@342334 91177308-0d34-0410-b5e6-96231b3b80d8