[Openmp-commits] [llvm] [openmp] [Offload] Move `/openmp/libomptarget` to `/offload` (PR #75125)

David Tenty via Openmp-commits openmp-commits at lists.llvm.org
Mon Apr 29 13:50:03 PDT 2024


================
@@ -113,7 +113,17 @@ option(OPENMP_ENABLE_LIBOMP_PROFILING "Enable time profiling for libomp." OFF)
 
 # Header install location
 if(${OPENMP_STANDALONE_BUILD})
-  set(LIBOMP_HEADERS_INSTALL_PATH "${CMAKE_INSTALL_INCLUDEDIR}")
+  if(CMAKE_CXX_COMPILER_ID MATCHES "Clang")
+    execute_process(
+      OUTPUT_STRIP_TRAILING_WHITESPACE
+      COMMAND ${CMAKE_CXX_COMPILER} --print-resource-dir
+      RESULT_VARIABLE COMMAND_RETURN_CODE
+      OUTPUT_VARIABLE COMPILER_RESOURCE_DIR
+    )
+    set(LIBOMP_HEADERS_INSTALL_PATH "${COMPILER_RESOURCE_DIR}/include")
----------------
daltenty wrote:

The addition of this change seems to have broken the stand alone build we were doing:
```
-- Installing: /nfs/build//lib/clang/19/include/omp.h
CMake Error at runtime/src/cmake_install.cmake:101 (file):
  file INSTALL cannot copy file
  "/home/daltenty/build-openmp/32/runtime/src/omp.h" to
  "/nfs/build/lib/clang/19/include/omp.h":
  Read-only file system.
Call Stack (most recent call first):
  runtime/cmake_install.cmake:42 (include)
  cmake_install.cmake:42 (include)
  
 We're now trying to install to the build compiler's resource directory for some reason, which is read-only and isn't really what we intended here.

https://github.com/llvm/llvm-project/pull/75125


More information about the Openmp-commits mailing list