[Openmp-commits] [PATCH] D149617: [OpenMP] Make `libomptarget` link against `libomp`

Tue May 2 05:17:06 PDT 2023

aaron.ballman added a comment.

In D149617#4311697 <https://reviews.llvm.org/D149617#4311697>, @Dinistro wrote:

> We observed some change in CMake behaviour in the last 24 hours and we suspect that this revision might be the cause of it.
> Originality, we used to enable openmp with the `-DLLVM_ENABLE_PROJECTS="....;openmp"` flag, but this now produces the following error:
>
>   CMake Error: install(EXPORT "LLVMExports" ...) includes target "omptarget" which requires target "omp" that is not in any export set.
>   CMake Error in cmake/modules/CMakeLists.txt:
>      export called with target "omptarget" which requires target "omp" that is
>      not in any export set.
>
> Switching to use the `-DLLVM_ENABLE_RUNTIMES="openmp"` seem fix this issue.
>
> I was just wondering if this is intended behaviour.

This broke our ability to publish Sphinx documentation, it seems: https://lab.llvm.org/buildbot/#/builders/89/builds/40301 so hopefully we can get a quick fix or revert.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149617