diff options
| author | Pavel Labath <labath@google.com> | 2018-05-14 14:13:20 +0000 |
|---|---|---|
| committer | Pavel Labath <labath@google.com> | 2018-05-14 14:13:20 +0000 |
| commit | 2a6afe5f87a08e1df0bed97edf8c4ab497dd9bff (patch) | |
| tree | c923b01eebfe155bb14573c336b5a1b3da3ffb12 /llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp | |
| parent | ded3dae331b8344677225de77e5ac0bfcd96438a (diff) | |
| download | bcm5719-llvm-2a6afe5f87a08e1df0bed97edf8c4ab497dd9bff.tar.gz bcm5719-llvm-2a6afe5f87a08e1df0bed97edf8c4ab497dd9bff.zip | |
[CodeGen/AccelTable]: Handle -dwarf-linkage-names=Abstract correctly
Summary:
If we are not emitting a linkage name in the .debug_info sections, we
should not add it into the index either. This makes sure our index is
consistent with the actual debug info.
I am also explicitly setting the --dwarf-linkage-names=All in the
name-collsions test as that one would now fail on targets where this
defaults to "Abstract" (in fact, it would have failed already if there
wasn't a bug in the DWARF verifier, which I fix as well).
Reviewers: probinson, aprantl, JDevlieghere
Subscribers: llvm-commits
Differential Revision: https://reviews.llvm.org/D46748
llvm-svn: 332246
Diffstat (limited to 'llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp')
| -rw-r--r-- | llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp index ba7aef37eac..b3b8d7d9146 100644 --- a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp +++ b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp @@ -1122,6 +1122,7 @@ DWARFVerifier::verifyNameIndexEntries(const DWARFDebugNames::NameIndex &NI, "of DIE @ {2:x}: index - {3}; debug_info - {4}.\n", NI.getUnitOffset(), EntryID, DIEOffset, Str, make_range(EntryNames.begin(), EntryNames.end())); + ++NumErrors; } } handleAllErrors(EntryOr.takeError(), |
