[Openmp-commits] [PATCH] D100810: [llvm] Use `GNUInstallDirs` to support custom installation dirs
John Ericson via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Sat Jan 1 10:22:36 PST 2022
Ericson2314 marked 2 inline comments as done.
Ericson2314 added a comment.
Mark some old threads done.
================
Comment at: llvm/cmake/modules/CMakeLists.txt:3
-set(LLVM_INSTALL_PACKAGE_DIR lib${LLVM_LIBDIR_SUFFIX}/cmake/llvm)
+set(LLVM_INSTALL_PACKAGE_DIR lib${LLVM_LIBDIR_SUFFIX}/cmake/llvm CACHE STRING "Path for CMake subdirectory (defaults to 'cmake/llvm')")
set(llvm_cmake_builddir "${LLVM_BINARY_DIR}/${LLVM_INSTALL_PACKAGE_DIR}")
----------------
Ericson2314 wrote:
> compnerd wrote:
> > Why is this variable being put into the cache now?
> I wanted all the installation dirs to be in there for consistency (c.f. Did the same for compiler-rt in D105765).
>
> FWIW I have a yet-unposted commit I want to improve a bit to make it more genuinely user-configurable too (we want the CMake with the headers so it need not be installed runtime)
I see I put this back for now, that's good.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100810/new/
https://reviews.llvm.org/D100810
More information about the Openmp-commits
mailing list