[PATCH] D81944: [llvm-install-name-tool] Adds docs for install-name-tool

Sameer Arora via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 16 12:05:40 PDT 2020


sameerarora101 marked 2 inline comments as done.
sameerarora101 added inline comments.


================
Comment at: llvm/docs/CommandGuide/llvm-install-name-tool.rst:28
+ Add an rpath named ``<rpath>`` to the specified binary. Can be specified multiple
+ times to add multiple rpaths. Throws an error if ``<rpath>`` is already listed in
+ the binary.
----------------
sameerarora101 wrote:
> Ktwu wrote:
> > Does `install-name-tool` throw different error codes for different error situations? It might be good to document what those error codes are.
> it does throw different error "messages" based on different error situations. But i'll take a look if there is a difference in error codes and update it soon.
`install-name-tool` has exit code 1 for all the errors. I have now updated that in the `EXIT STATUS` section below.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81944





More information about the llvm-commits mailing list