[PATCH] D65384: [Docs][llvm-strip] Add help text to llvm-strip rst doc

Mike Pozulp via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 29 23:01:06 PDT 2019


mmpozulp added a comment.

In D65384#1604328 <https://reviews.llvm.org/D65384#1604328>, @jhenderson wrote:

> Thanks for working on this! You're missing a number of aliases from the list. Please make sure they are included in the doc.


Happy to help :). Will do.

> Also, please use the text from the llvm-objcopy doc where the switch is common to the two tools.

Yes, good idea. In fact, I'm going to move the common switches into a new file, say CommonOpts.td, and `include "CommonOpts.td"` at the top of ObjcopyOpts.td and StripOpts.td. That way we don't have the same switch in two different places each with different HelpText.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65384





More information about the llvm-commits mailing list