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

John Ericson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 29 08:04:12 PDT 2021


Ericson2314 added inline comments.


================
Comment at: polly/cmake/CMakeLists.txt:104-110
+    set(tgt_prefix "${CMAKE_INSTALL_FULL_BINDIR}/")
   else()
-    set(tgt_prefix "lib/")
+    set(tgt_prefix "${CMAKE_INSTALL_FULL_LIBDIR}/")
   endif()
 
-  set(tgt_path "${CMAKE_INSTALL_PREFIX}/${tgt_prefix}$<TARGET_FILE_NAME:${tgt}>")
+  set(tgt_path "${tgt_prefix}$<TARGET_FILE_NAME:${tgt}>")
   file(RELATIVE_PATH tgt_path ${POLLY_CONFIG_CMAKE_DIR} ${tgt_path})
----------------
@Meinersbur the change was here. I did indeed get an error because this stuff prior to installing did in fact need an absolute path.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99484



More information about the llvm-commits mailing list