[Openmp-commits] [PATCH] D93738: [OpenMP] Not set OPENMP_STANDALONE_BUILD=ON when building OpenMP along with LLVM

Mon Jan 4 01:17:14 PST 2021

protze.joachim added a comment.

In D93738#2476692 <https://reviews.llvm.org/D93738#2476692>, @tianshilei1992 wrote:

> Building runtimes is not part of building LLVM. It is different from building OpenMP using CMake argument `LLVM_ENABLE_PROJECTS `.

It might be different, but your change still breaks the OpenMP tests, when OpenMP is built using `LLVM_ENABLE_PROJECTS`.

Can you test for the clang target and only set the dependency, if the target exists in the current configuration?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93738