[PATCH] D37771: llvm-dwarfdump: automatically dump both regular and .dwo variant of sections

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 12 17:30:29 PDT 2017


On Tue, Sep 12, 2017 at 4:13 PM Adrian Prantl via Phabricator <
reviews at reviews.llvm.org> wrote:

> aprantl added a comment.
>
> Okay, I'll implement a "don't print anything if the section doesn't exist"
> approach, since that seems to align best with what everybody desires.
> Printing an empty "contents" only if the section was explicitly required is
> trickier to implement, so I'll leave this for a future improvement.
>

I probably-acceptable easy fix would be "if not all sections were
requested, print section headers even for empty sections" behavior might
suffice. If someone manually specifies each/every section and then finds
this weird... yeah, it could be fixed, but hardly seems too important.
Doesn't phase me, though.

*shrug*


>
>
> https://reviews.llvm.org/D37771
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170913/0a234ccf/attachment.html>


More information about the llvm-commits mailing list