[Openmp-commits] [llvm] [openmp] [OpenMP][Offload] Support standalone build (PR #88957)

via Openmp-commits openmp-commits at lists.llvm.org
Wed Apr 17 07:02:35 PDT 2024


================
@@ -35,11 +42,12 @@ endif()
 # TODO: Leftover from the move, could probably be just LLVM_LIBDIR_SUFFIX everywhere.
 set(OFFLOAD_INSTALL_LIBDIR "lib${LLVM_LIBDIR_SUFFIX}")
 
-set(LLVM_COMMON_CMAKE_UTILS ${CMAKE_CURRENT_SOURCE_DIR}/cmake)
+set(LLVM_COMMON_CMAKE_UTILS ${CMAKE_CURRENT_SOURCE_DIR}/../cmake)
----------------
estewart08 wrote:

I think it's ok to assume the openmp sources are still inside the llvm-project, right? It is standalone in the sense that it will not be built during the same build as llvm. This change was more so a correction on the current offload patch `${CMAKE_CURRENT_SOURCE_DIR}/cmake` == `offload/cmake`, which is not what `LLVM_COMMON_CMAKE_UTILS` is supposed to reference.

I was just keeping the autodetection in for OMPT as the system configuration still needs checked to verify whether it can be supported, regardless if the user passes it in or not.

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


More information about the Openmp-commits mailing list