[Openmp-commits] [PATCH] D103545: [NFC][libomptarget] Reduce the dependency on libelf

Fri Jun 18 08:48:24 PDT 2021

ronlieb added a comment.

> Thank you, @ronlieb!  I am able to reproduce it.  It seems that `LLVM_ENABLE_RUNTIMES` behavior is quite different from the `-DLLVM_ENABLE_PROJECTS="openmp"` that I use (so I did not see this issue before).  It seems to me that in `LLVM_ENABLE_RUNTIMES` mode, `openmp` is configured/built as a standalone project using `clang` built by the "main" cmake/build step, but for some reason `OPENMP_STANDALONE_BUILD` is `false`.  I am not sure about the right solution yet.  Can you please add `openmp` to your `-DLLVM_ENABLE_PROJECTS="clang;lld"` and remove `"-DLLVM_ENABLE_RUNTIMES="openmp"` as a temporary workaround?

i added openmp to PROJECTS, and removed        -DLLVM_ENABLE_RUNTIMES="openmp"

results in error

CMake Error at cmake/modules/LLVMExternalProjectUtils.cmake:358 (add_custom_target):

  add_custom_target cannot create target "check-openmp" because another
  target with the same name already exists.  The existing target is a custom
  target created in source directory
  "/work/rlieberm/mono-repo/llvm-project/openmp".  See documentation for
  policy CMP0002 for more details.

Call Stack (most recent call first):

  runtimes/CMakeLists.txt:226 (llvm_ExternalProject_Add)
  runtimes/CMakeLists.txt:353 (runtime_default_target)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D103545/new/

https://reviews.llvm.org/D103545