[PATCH] D63637: Do not set an RPATH on statically-linked LLVM executables.

Thu Jun 20 20:27:37 PDT 2019

mgorny requested changes to this revision.
mgorny added inline comments.
This revision now requires changes to proceed.

================
Comment at: llvm/cmake/modules/AddLLVM.cmake:756

-  if(NOT ARG_NO_INSTALL_RPATH)
-    llvm_setup_rpath(${name})
+  if (LLVM_LINK_LLVM_DYLIB AND NOT ARG_DISABLE_LLVM_LINK_LLVM_DYLIB)
+    set(USE_SHARED USE_SHARED)
----------------
I think this is also needed when building with `BUILD_SHARED_LIBS`. Possibly also when the target explicitly links to one of the shared-only libraries.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63637