[PATCH] D62706: [llvm-lipo] Add docs for llvm-lipo
Alexander Shaposhnikov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 6 16:29:34 PDT 2019
alexshap marked an inline comment as done.
alexshap added inline comments.
================
Comment at: docs/CommandGuide/llvm-lipo.rst:28
+
+ Verify that the specified architectures are present in the input file.
+
----------------
mtrent wrote:
> Verify that the specified architecture is present by doing what? Best to call out the command's exit status here.
>
> Is it obvious what "architecture" means here? How would you describe that to someone?
>
> The -verify_arch command only works on a single file (the Apple man page suggests otherwise, but AFAICT the code has never supported that). Do you need to mention that here?
>Is it obvious what "architecture" means here?
no, it's not, honestly, I don't have a good explanation appropriate for docs other than "supported architecture name" with some popular examples, would you like me to add these bits here ?
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62706/new/
https://reviews.llvm.org/D62706
More information about the llvm-commits
mailing list