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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

The tabelgen option changes should be in their own review. I haven't tried to review them here.



================
Comment at: llvm/docs/CommandGuide/llvm-strip.rst:15
+:program:`llvm-strip` is a tool to strip sections and symbols from object files.
+In basic usage it removes any data that is not needed to run the code in the
+file.
----------------
Perhaps rephrase this to make the behaviour clearer. E.g. "If no other stripping or remove options are specified, --strip-all will be enabled by default."


================
Comment at: llvm/docs/CommandGuide/llvm-strip.rst:87
+
+ Remove debug sections only.
+
----------------
Change this to "remove all debug sections". Specifying other switches could mean that the "only" is invalid.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65384