[PATCH] D63826: [docs][llvm-readelf] Expand llvm-readelf documentation

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 27 01:40:30 PDT 2019


jhenderson added a comment.

In D63826#1559403 <https://reviews.llvm.org/D63826#1559403>, @peter.smith wrote:

> Overall no objections. Only thing I can think of that could be surprising is the presence of mach-o and coff options in llvm-readelf. A case could be made for leaving these out as people migrating won't need to use them, but I don't have a strong opinion.


I'd be okay with this, but in that case, I'd recommend hiding the options in the help text too, so that's possibly tangential to this change? I'll post something on the mailing list to see if anybody else has any thoughts.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63826





More information about the llvm-commits mailing list