[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