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

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 18 08:17:13 PST 2015


rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM with a nit.


================
Comment at: tools/llvm-objdump/llvm-objdump.cpp:1629
@@ +1628,3 @@
+    // Dump the complete DWARF structure.
+    DICtx->dump(outs(), DwarfDumpType, true /* DumpEH */);
+  }
----------------
Shouldn't this be checking DIDT_Frames?


http://reviews.llvm.org/D15535





More information about the llvm-commits mailing list