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

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added reviewers: rnk, beanz.
chandlerc added a comment.

(Will likely need more eyes than just mine -- RPATH is mostly a mystery to me...)



================
Comment at: llvm/cmake/modules/AddLLVM.cmake:758
+    set(USE_SHARED USE_SHARED)
+    if(NOT ARG_NO_INSTALL_RPATH)
+      llvm_setup_rpath(${name})
----------------
Just to make sure I understand correctly, *any* time we build a shared library of some form, we'll get here? Even if its `libclang.so` or some such?

Provided I've understood correctly, then this make sense. However, I wonder, won't this still fail for things like c-index-test?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63637