[PATCH] D93177: [CMake] Fix BUILD_WITH_INSTALL_RPATH being set when LLVM_LOCAL_RPATH is empty

Raul Tambre via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 13 02:35:05 PST 2020


tambre updated this revision to Diff 311435.
tambre added a comment.

Fix commit message


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93177

Files:
  llvm/cmake/modules/AddLLVM.cmake


Index: llvm/cmake/modules/AddLLVM.cmake
===================================================================
--- llvm/cmake/modules/AddLLVM.cmake
+++ llvm/cmake/modules/AddLLVM.cmake
@@ -865,7 +865,7 @@
 
   if(NOT ARG_NO_INSTALL_RPATH)
     llvm_setup_rpath(${name})
-  elseif (LLVM_LOCAL_RPATH)
+  elseif(CMAKE_BUILD_RPATH STREQUAL "")
     set_target_properties(${name} PROPERTIES
                           BUILD_WITH_INSTALL_RPATH On
                           INSTALL_RPATH "${LLVM_LOCAL_RPATH}")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93177.311435.patch
Type: text/x-patch
Size: 502 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201213/9f12f8d4/attachment.bin>


More information about the llvm-commits mailing list