diff options
author | Spyridoula Gravani <sgravani@apple.com> | 2017-06-16 22:03:21 +0000 |
---|---|---|
committer | Spyridoula Gravani <sgravani@apple.com> | 2017-06-16 22:03:21 +0000 |
commit | 32614fcf421a8e812cde1218c9271d51a9fee166 (patch) | |
tree | ae567f2491a610504d7f1e863d0a10b30ca4feb3 /llvm/lib | |
parent | 7687f04672b10c01be1a75d33d0c103fec4e4100 (diff) | |
download | bcm5719-llvm-32614fcf421a8e812cde1218c9271d51a9fee166.tar.gz bcm5719-llvm-32614fcf421a8e812cde1218c9271d51a9fee166.zip |
[DWARF] Corrected behavior for when no .apple_names section is present in the object.
The verifier should not output any message in such a case.
Added test case with no .apple_name section in the file to verify new functionality.
Made existing test case more specific.
llvm-svn: 305597
Diffstat (limited to 'llvm/lib')
-rw-r--r-- | llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp index a6240fb6014..41907e57056 100644 --- a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp +++ b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp @@ -279,7 +279,6 @@ bool DWARFVerifier::handleDebugLine() { bool DWARFVerifier::handleAppleNames() { NumAppleNamesErrors = 0; - OS << "Verifying .apple_names...\n"; DataExtractor AppleNamesSection(DCtx.getAppleNamesSection().Data, DCtx.isLittleEndian(), 0); @@ -288,10 +287,11 @@ bool DWARFVerifier::handleAppleNames() { DCtx.getAppleNamesSection().Relocs); if (!AppleNames.extract()) { - OS << "error: cannot extract .apple_names accelerator table\n"; - return false; + return true; } + OS << "Verifying .apple_names...\n"; + // Verify that all buckets have a valid hash index or are empty uint32_t NumBuckets = AppleNames.getNumBuckets(); uint32_t NumHashes = AppleNames.getNumHashes(); |