[PATCH] D125008: [llvm-objdump] Print Mnemonic Histogram

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

Thanks all for looking at this!

I understand the concerns about adding/removing options. What I will do is leave a message on discourse to possibly draw some attention to this and check if anyone else has objections/preferences.

In the mean time I will work on a new revision of this which will distinguish between narrow and wide instruction encodings, and see how that fits in. That will serve two purposes. First, it's what I would really need for my use case, although in its current form it's already useful. Second, although not impossible, that would be more difficult to achieve with a one-liner on the command line. That would require a script, and that could be the alternative of this.

Personally, I prefer the built-in option because of its convenience. I could never remember that one-liner command line, would need to save it to a file, which I then don't have available on the different systems I work on etc.  A python script contributed to `llvm/utils` for example would have this problem a lot less, but still wouldn't be as convenient as a built-in option.


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

https://reviews.llvm.org/D125008