[all-commits] [llvm/llvm-project] 22afe1: DebugInfo: Rebuild dwp debug_info index column fro...
David Blaikie via All-commits
all-commits at lists.llvm.org
Tue Mar 28 13:47:35 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 22afe19ac03f5b5db642cbb8ba7022c2ffc09710
https://github.com/llvm/llvm-project/commit/22afe19ac03f5b5db642cbb8ba7022c2ffc09710
Author: David Blaikie <dblaikie at gmail.com>
Date: 2023-03-28 (Tue, 28 Mar 2023)
Changed paths:
M llvm/lib/DebugInfo/DWARF/DWARFContext.cpp
A llvm/test/tools/llvm-dwp/X86/cu_tu_units_manual_v4.s
Log Message:
-----------
DebugInfo: Rebuild dwp debug_info index column from v5 indexes more robustly
the v4 rebuilding is a best-effort because it's not possible to reliably
parse the DWO ID as it requires the abbrev section (& if the index isn't
trustworthy then there's no way to find the associated abbrev section
contribution for a given info section contribution)
But in v5 the DWO ID/type signature is in the header and can be rebuilt
losslessly (only at the cost of performance of rescanning/parsing the
headers of all the units), so let's implement that.
the testing isn't /ideal/ - I think the testing should've been
implemented as a hardcoded dwp file with a corrupted/incorrect index,
then the test could've demonstrated that reparsing the index produces
the right answer - but this is a quick port of the existing v5 test back
to v4 so that we don't lose coverage on the v4 codepath now that it's
separated from the v5 codepath.
Differential Revision: https://reviews.llvm.org/D146662
More information about the All-commits
mailing list