[PATCH] D65313: [llvm-readelf] Dump the stack sizes sections with --stack-sizes

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 1 14:56:14 PDT 2019


rupprecht added a comment.

In D65313#1609287 <https://reviews.llvm.org/D65313#1609287>, @wolfgangp wrote:

> Left the question about whether --stack-sizes should be include in --all open. This is still a discussion point.


Do you have a discussion going on somewhere you can link to?

Personally, I think it's neat & wouldn't mind it being included as part of `--all` for personal use, but generally we want to make `llvm-readelf` as close to GNU `readelf` as possible. (This has been feedback given at llvm dev meetings too).

Maybe it could be part of `--all` only when invoked as `llvm-readobj`? Though it doesn't make much since *now* because this only has the GNU-style dumper implemented, and in fact prints a warning for the LLVM style -- but in the future, we could do that.


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

https://reviews.llvm.org/D65313





More information about the llvm-commits mailing list