commit 940c8fbf3d52bd8cc722a75fd3210e7239ac7f0b
author Martin Storsjö <martin@martin.st> Wed Jan 10 11:24:19 2024 +0200
committer Copybara-Service <copybara-worker@google.com> Wed Jan 10 01:25:37 2024 -0800
tree 4f0cf1d25712c625b0961c598b25a4e24cf3bcb6
parent d0aacea95ee8b566f977a50b3893470d00e91207

[OpenMP] Allow setting OPENMP_INSTALL_LIBDIR (#77533)

The comment indicate that it should be possible, but as long as it
wasn't a cache variable, the cmake script overwrote whatever variable
the user had set.

GitOrigin-RevId: 14435a28cd144f157ec4e6022d8c0ff0926e549f

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index c1c79f8..03068af 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -29,7 +29,8 @@
   set(OPENMP_LIBDIR_SUFFIX "" CACHE STRING
     "Suffix of lib installation directory, e.g. 64 => lib64")
   # Do not use OPENMP_LIBDIR_SUFFIX directly, use OPENMP_INSTALL_LIBDIR.
-  set(OPENMP_INSTALL_LIBDIR "lib${OPENMP_LIBDIR_SUFFIX}")
+  set(OPENMP_INSTALL_LIBDIR "lib${OPENMP_LIBDIR_SUFFIX}" CACHE STRING
+      "Path where built OpenMP libraries should be installed.")
 
   # Group test settings.
   set(OPENMP_TEST_C_COMPILER ${CMAKE_C_COMPILER} CACHE STRING
@@ -46,7 +47,8 @@
 else()
   set(OPENMP_ENABLE_WERROR ${LLVM_ENABLE_WERROR})
   # If building in tree, we honor the same install suffix LLVM uses.
-  set(OPENMP_INSTALL_LIBDIR "lib${LLVM_LIBDIR_SUFFIX}")
+  set(OPENMP_INSTALL_LIBDIR "lib${LLVM_LIBDIR_SUFFIX}" CACHE STRING
+      "Path where built OpenMP libraries should be installed.")
 
   if (NOT MSVC)
     set(OPENMP_TEST_C_COMPILER ${LLVM_RUNTIME_OUTPUT_INTDIR}/clang)