From 686f121a5de3864ed0343b5753e60e1641c2c4a7 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Fri, 21 Jul 2017 18:28:55 +0000 Subject: [PDB] Dump extra info about the publics stream This includes the hash table, the address map, and the thunk table and section offset table. The last two are only used for incremental linking, which LLD doesn't support, so they are less interesting. The hash table is particularly important to get right, since this is the one of the streams that debuggers use to translate addresses to symbols. llvm-svn: 308764 --- llvm/lib/DebugInfo/PDB/Native/GSI.cpp | 6 +++--- llvm/lib/DebugInfo/PDB/Native/GSI.h | 2 +- llvm/lib/DebugInfo/PDB/Native/PublicsStream.cpp | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) (limited to 'llvm/lib/DebugInfo/PDB') diff --git a/llvm/lib/DebugInfo/PDB/Native/GSI.cpp b/llvm/lib/DebugInfo/PDB/Native/GSI.cpp index b219fe275f7..d77676dbd1c 100644 --- a/llvm/lib/DebugInfo/PDB/Native/GSI.cpp +++ b/llvm/lib/DebugInfo/PDB/Native/GSI.cpp @@ -29,6 +29,7 @@ static Error checkHashHdrVersion(const GSIHashHeader *HashHdr) { } Error readGSIHashBuckets(FixedStreamArray &HashBuckets, + ArrayRef &HashBitmap, const GSIHashHeader *HashHdr, BinaryStreamReader &Reader) { if (auto EC = checkHashHdrVersion(HashHdr)) @@ -36,15 +37,14 @@ Error readGSIHashBuckets(FixedStreamArray &HashBuckets, // Before the actual hash buckets, there is a bitmap of length determined by // IPHR_HASH. - ArrayRef Bitmap; size_t BitmapSizeInBits = alignTo(IPHR_HASH + 1, 32); uint32_t NumBitmapEntries = BitmapSizeInBits / 8; - if (auto EC = Reader.readBytes(Bitmap, NumBitmapEntries)) + if (auto EC = Reader.readBytes(HashBitmap, NumBitmapEntries)) return joinErrors(std::move(EC), make_error(raw_error_code::corrupt_file, "Could not read a bitmap.")); uint32_t NumBuckets = 0; - for (uint8_t B : Bitmap) + for (uint8_t B : HashBitmap) NumBuckets += countPopulation(B); // Hash buckets follow. diff --git a/llvm/lib/DebugInfo/PDB/Native/GSI.h b/llvm/lib/DebugInfo/PDB/Native/GSI.h index 9e63bc83548..ce2b301b3cc 100644 --- a/llvm/lib/DebugInfo/PDB/Native/GSI.h +++ b/llvm/lib/DebugInfo/PDB/Native/GSI.h @@ -41,7 +41,7 @@ namespace pdb { static const unsigned IPHR_HASH = 4096; /// Header of the hash tables found in the globals and publics sections. -/// Based on GSIHashHeader in +/// Based on GSIHashHdr in /// https://github.com/Microsoft/microsoft-pdb/blob/master/PDB/dbi/gsi.h struct GSIHashHeader { enum : unsigned { diff --git a/llvm/lib/DebugInfo/PDB/Native/PublicsStream.cpp b/llvm/lib/DebugInfo/PDB/Native/PublicsStream.cpp index 9c3e654f808..3b4e18de8f9 100644 --- a/llvm/lib/DebugInfo/PDB/Native/PublicsStream.cpp +++ b/llvm/lib/DebugInfo/PDB/Native/PublicsStream.cpp @@ -75,7 +75,7 @@ Error PublicsStream::reload() { if (auto EC = readGSIHashRecords(HashRecords, HashHdr, Reader)) return EC; - if (auto EC = readGSIHashBuckets(HashBuckets, HashHdr, Reader)) + if (auto EC = readGSIHashBuckets(HashBuckets, HashBitmap, HashHdr, Reader)) return EC; NumBuckets = HashBuckets.size(); -- cgit v1.2.3