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

Greg Clayton via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 30 16:09:46 PST 2018



> 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?


More information about the llvm-commits mailing list