[PATCH] D55091: Add --analyze option to llvm-dwarfdump

[David Blaikie via llvm-commits]

On Fri, Nov 30, 2018 at 4:09 PM Greg Clayton <clayborg at gmail.com> wrote:

>
>
> > On Nov 30, 2018, at 4:02 PM, David Blaikie <dblaikie at gmail.com> wrote:
> >
> > I'd vote for it going in libDebugInfo and being visible from both tools
> - to improve discoverability. (at least I think a section-level breakdown
> starts to feel like an objdump feature - and it'd be weird to have to
> switch between the two tools, etc).
> >
> > I'm wind Vedant also on having diff support, same as bloaty (looks like
> we're going down the road of basically implementing that sort of
> functionality in LLVM) - but that's a feature request, not something that
> blocks development on other features, etc.
>
> So basically we can move Analyze.cpp over into libDebugInfo and generalize
> it a bit to have the analyzeObjectFile function take a options parameter
> for any options that would be set by tools options?


I haven't looked at the code in detail, but yeah, that's about what I'd
think. Others might have more nuanced ideas about how to do it.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181130/7e0ead8e/attachment.html>