summaryrefslogtreecommitdiffstats
path: root/clang/lib/AST/Expr.cpp
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2017-08-15 21:31:41 +0000
committerZachary Turner <zturner@google.com>2017-08-15 21:31:41 +0000
commit024323cb126f99a4261365b06c49f671d0dc78c5 (patch)
treebbb434a0c4b2c5dbcea7c60b1c94c2f4cd9103bc /clang/lib/AST/Expr.cpp
parentfe9f1e60c517e71bd819cab9e90b110457ca42b6 (diff)
downloadbcm5719-llvm-024323cb126f99a4261365b06c49f671d0dc78c5.tar.gz
bcm5719-llvm-024323cb126f99a4261365b06c49f671d0dc78c5.zip
[LLD COFF/PDB] Incrementally update the build id.
Previously, our algorithm to compute a build id involved hashing the executable and storing that as the GUID in the CV Debug Record chunk, and setting the age to 1. This breaks down in one very obvious case: a user adds some newlines to a file, rebuilds, but changes nothing else. This causes new line information and new file checksums to get written to the PDB, meaning that the debug info is different, but the generated code would be the same, so we would write the same build over again with an age of 1. Anyone using a symbol cache would have a problem now, because the debugger would open the executable, look at the age and guid, find a matching PDB in the symbol cache and then load it. It would never copy the new PDB to the symbol cache. This patch implements the canonical Windows algorithm for updating a build id, which is to check the existing executable first, and re-use an existing GUID while bumping the age if it already exists. Differential Revision: https://reviews.llvm.org/D36758 llvm-svn: 310961
Diffstat (limited to 'clang/lib/AST/Expr.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud