[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 10:56:15 PST 2015


pete closed this revision.
pete added a comment.

Committed in r256008.

I removed all the other dump dwarf dump options so we just have dwarf=frames.  We can add them back in as needed.

I left the conditional checks against "!= DIDT_Null" as that way we don't have to change those lines in future when we add more options to become more like objdump.

If any of that isn't quite what you were thinking, let me know and i'll be happy to change it.

Thanks again for the review, Rafael,
Pete


http://reviews.llvm.org/D15535





More information about the llvm-commits mailing list