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

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a subscriber: jakehehrlich.
vsk added a comment.

In D55091#1313752 <https://reviews.llvm.org/D55091#1313752>, @dblaikie wrote:

> What was the end result of the discussion around this and the other size analysis proposal (for objdump?) that came around the same time - and the overlap with Bloaty, for instance?


Reading the thread again (http://lists.llvm.org/pipermail/llvm-dev/2018-September/126501.html), istm that most folks were ok with having a size analysis tool in llvm. I pointed out some specific advantages (code reuse, shared maintenance) of having such a tool live in-tree. David suggested folding bloaty under the llvm umbrella -- my concern  (and IIRC @jakehehrlich's as well) was that doing so would amount to a full re-write. I've since switched teams to focus on outliner work, and haven't had time to whip my initial prototype into shape for upstreaming.

> Would be good to have a goal - for instance, one of the things about binary size analysis is also looking at the reloc sizes for the debug info sections (& ultimately, probably looking at the whole object file/executable - including headers, etc, not just the section sizes themselves - for comparison purposes (optimizing debug info size only to realize there's something else holding up the object size tent would be unfortunate)).

Agreed, I actually think this would make sense as part of a more comprehensive analysis mode in llvm-objdump.


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

https://reviews.llvm.org/D55091