<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 13, 2017, at 2:01 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Wed, Sep 13, 2017 at 1:58 PM Adrian Prantl via Phabricator <<a href="mailto:reviews@reviews.llvm.org" class="">reviews@reviews.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl updated this revision to Diff 115111.<br class="">
aprantl added a comment.<br class="">
Herald added a subscriber: javed.absar.<br class="">
<br class="">
With this update:<br class="">
<br class="">
1. llvm-dwarfdump --all prints only section headers (like `.debug_info contents:`) of non-empty sections.<br class="">
<br class="">
2a. llvm-dwarfdump --<debug-info, ...> always prints a section header<br class="">
2b. On platforms that support DWOs, llvm-dwarfdump --<debug-info ..> also always prints a section header for the corresponding .dwo section.<br class=""></blockquote><div class=""><br class="">For this one^ (& maybe this'd merge with 2a so it wouldn't need a "supported platforms" check) the thing would be to print the non-dwo header if both sections are empty, otherwise print the headers only for the non-empty ones. I mean I guess that'd be a bit weird if you ask for debug_types on a .dwo file and that file happens to not have type units enabled & it prints the non-dwo name)<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>I'm having trouble parsing this. What would you like the behavior for `llvm-dwarfdump --debug-types file.dwo` to be if (1) file.dwo has no .debug_types.dwo section and no .debug_types section, (2) if it has only the .debug_types.dwo section?</div><div><br class=""></div><div>-- adrian</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
<br class="">
<a href="https://reviews.llvm.org/D37771" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/D37771</a><br class="">
<br class="">
Files:<br class="">
include/llvm/BinaryFormat/Dwarf.def<br class="">
include/llvm/DebugInfo/DIContext.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFContext.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFDebugAbbrev.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFDebugLoc.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFDebugMacro.h<br class="">
include/llvm/DebugInfo/DWARF/DWARFUnitIndex.h<br class="">
lib/DebugInfo/DWARF/DWARFContext.cpp<br class="">
test/DebugInfo/Inputs/gmlt.ll<br class="">
test/DebugInfo/X86/fission-inline.ll<br class="">
test/DebugInfo/X86/fission-no-inlining.ll<br class="">
test/DebugInfo/X86/fission-ranges.ll<br class="">
test/DebugInfo/X86/generate-odr-hash.ll<br class="">
test/DebugInfo/X86/gnu-public-names-gmlt.ll<br class="">
test/DebugInfo/X86/live-debug-variables.ll<br class="">
test/DebugInfo/X86/split-dwarf-cross-unit-reference.ll<br class="">
test/DebugInfo/X86/split-dwarf-multiple-cu-hash.ll<br class="">
test/DebugInfo/X86/split-dwarf-omit-empty.ll<br class="">
test/DebugInfo/dwarfdump-dump-flags.test<br class="">
test/MC/ARM/dwarf-asm-multiple-sections-dwarf-2.s<br class="">
test/MC/ARM/dwarf-asm-no-code.s<br class="">
test/MC/ARM/dwarf-asm-nonstandard-section.s<br class="">
test/MC/ARM/dwarf-asm-single-section.s<br class="">
test/tools/dsymutil/X86/basic-linking-x86.test<br class="">
test/tools/llvm-dwp/X86/empty.test<br class="">
test/tools/llvm-dwp/X86/simple.test<br class="">
tools/llvm-dwarfdump/llvm-dwarfdump.cpp<br class="">
<br class="">
</blockquote></div></div>
</div></blockquote></div><br class=""></body></html>