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

[Greg Clayton via Phabricator via llvm-commits]

clayborg added a comment.

In D55091#1315244 <https://reviews.llvm.org/D55091#1315244>, @probinson wrote:

> In D55091#1314919 <https://reviews.llvm.org/D55091#1314919>, @vsk wrote:
>
> > Your DWARF analysis tool presumably needs to support diffing and output serialization. So would a comprehensive code size analysis tool. There's no reason why these two tools should have different diffing / serialization options. The latter shouldn't live within a debug info specific tool. However, there's no fundamental issue with adding DWARF to the long list of formats llvm-objdump already understands. Technically, it's just a matter of linking in llvm's DWARF libraries.
>
>
> Diffing?  Of what?  If we're tracking sizes etc over time, the diffing shouldn't be in the tool.  Dump stats to a database and do the normal thing with it there.


Since there is a JSON output, I figure it will be pretty easy to implement a tool that can diff two different JSON files for the same executable after it has been recompiled. So JSON should enable these tools to be made and allow progress of this feature.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55091/new/

https://reviews.llvm.org/D55091