[PATCH] D63100: [llvm-lipo] Update llvm-lipo docs for -archs flag

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 10 15:38:03 PDT 2019


alexshap accepted this revision.
alexshap added a comment.
This revision is now accepted and ready to land.

I've added one inline comment, other than that - LGTM, I'd wait for @mtrent as well )



================
Comment at: llvm/docs/CommandGuide/llvm-lipo.rst:32
+
+  Take a single input file and display the architectures present in the file.
+
----------------
maybe add a bit more details similar to what the man page says:
"Each architecture name is a  single  word,  making  this  option
suitable for shell scripting. Unknown architectures will be represented by "unknown" along with the numeric CPU type and CPU subtype values as a single word.", though personally I don't insist.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63100





More information about the llvm-commits mailing list