[PATCH] D107716: Correct openmp-offload-builders lld, openmp

Michael Kruse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 9 12:37:59 PDT 2021


Meinersbur reopened this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.

The `enable_runtimes=['openmp'],`openmp-offload-amdgpu-runtime` for parameters exactly sets `-DLLVM_ENABLE_RUNTIMES=openmp` (and ensures that changes to openmp triggers a build). Do not add it separately.

If you do not want support a `LLVM_ENABLE_PROJECT=openmp`, remove the `openmp-offload-amdgpu-project` builder instead. There is no point in two builders doing the same thing.


Repository:
  rZORG LLVM Github Zorg

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

https://reviews.llvm.org/D107716



More information about the llvm-commits mailing list