[PATCH] D15535: Use dwarfdump's frame parser instead of writing a new one in llvm-obdjump

Pete Cooper via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 09:32:34 PST 2015


pete added inline comments.

================
Comment at: tools/llvm-objdump/llvm-objdump.cpp:1629
@@ +1628,3 @@
+    // Dump the complete DWARF structure.
+    DICtx->dump(outs(), DwarfDumpType, true /* DumpEH */);
+  }
----------------
rafael wrote:
> Shouldn't this be checking DIDT_Frames?
Good question.  Wasn't sure exactly how we want to handle this.

The cl::opt I added was copied from llvm-dwarfdump so supports much more than just frames.  If we only want llvm-objdump to support frames for now then I'll remove the other options from llvm-objdump until we want to add them later.

The objdump man page lists the following, which is why I thought we could just allow pretty much anything llvm-dwarfdump has in it, but i'm happy to restrict it to frames for now:

man page: "--dwarf[=rawline,=decodedline,=info,=abbrev,=pubnames,=aranges,=macro,=frames,=frames-interp,=str,=loc,=Ranges]"


http://reviews.llvm.org/D15535





More information about the llvm-commits mailing list