[PATCH] D137096: [llvm-readobj] Provide Hash Histogram for all ELFDumper implementations

[Paul Kirth via Phabricator via llvm-commits]

paulkirth added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:4972
+                                                 ArrayRef<size_t> Count,
+                                                 bool IsGnu) const {
   size_t CumulativeNonZero = 0;
----------------
jhenderson wrote:
> I have a marginal preference to make this argument `StringRef TableDescription` (with values of "" and `.gnu.hash' respectively, since there's only one place the behaviour is different between the two output styles, as far as I can tell.
Sure, but if they have the same interface, then they can be overrides from the base class, as they are now. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137096