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

[Adrian Prantl via llvm-commits]

> On Sep 11, 2017, at 4:14 PM, David Blaikie <dblaikie at gmail.com> wrote:
> 
> On Mon, Sep 11, 2017 at 4:10 PM Adrian Prantl via Phabricator <reviews at reviews.llvm.org <mailto:reviews at reviews.llvm.org>> wrote:
> aprantl added a comment.
> 
> In https://reviews.llvm.org/D37714#867252 <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.
>  

Right, I forgot about our objcopy workflow. I'll try and implement this and then post a patch for further comments.

-- adrian

> 
> - adrian
> 
> 
> Repository:
>   rL LLVM
> 
> https://reviews.llvm.org/D37714 <https://reviews.llvm.org/D37714>
> 
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170911/78c32c92/attachment.html>