[PATCH] D37714: llvm-dwarfdump: Replace -debug-dump=sect option with individual options.

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 11 16:14:07 PDT 2017


On Mon, Sep 11, 2017 at 4:10 PM Adrian Prantl via Phabricator <
reviews at reviews.llvm.org> wrote:

> aprantl added a comment.
>
> In https://reviews.llvm.org/D37714#867252, @probinson wrote:
>
> > I would kind of like one option to dump whichever of .debug_info or
> .debug_info.dwo (etc) there is, so I don't have to know up front which
> section to be asking for.  Would also be more user-friendly to people who
> kind of know DWARF but not at that level of detail.
>
>
> Since the DWO and non-DWO variants rarely (never) appear in the same file,
> is there even a point in having a separate options?
> What about supporting both options, but --debug-info also implies
> --debug-info-dwo?
>

I'd mostly be OK merging them, though they do come up in the same file as
it comes out of llc (since llc doesn't run objcopy to split out the dwo
file, etc) - might involve tweaking some tests.


>
> - adrian
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D37714
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170911/89edc59c/attachment.html>


More information about the llvm-commits mailing list