[PATCH] D115568: [lld] Use `GNUInstallDirs` to support custom installation dirs.

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 13 23:44:49 PST 2021


phosek added inline comments.


================
Comment at: lld/cmake/modules/CMakeLists.txt:29-39
+function(extend_delayed_install_path joined_path current_segment)
+  if("${current_segment}" STREQUAL "")
+    set(temp_path "\${LLD_INSTALL_PREFIX}")
+  elseif(IS_ABSOLUTE "${current_segment}")
+    message(WARNING "Since \"${current_segment}\" is absolute, it overrides LLD_INSTALL_PREFIX")
+    set(temp_path "${current_segment}")
+  else()
----------------
Could we move this to a top-level CMake module in https://github.com/llvm/llvm-project/tree/main/cmake/Modules so we can avoid duplicating this logic in every subproject?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115568



More information about the llvm-commits mailing list