[Openmp-commits] [PATCH] D123177: [libomptarget] Add device RTL to regression test dependencies.

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

I think there is a problem here. Openmp as a llvm_project as opposed to a llvm_runtime can be built with GCC to get the host side of openmp working without the target offloading part. However GCC can't build the devicertl.

That probably means we only want a dependency from the plugins to the devicertl when we're building for offloading, which in practice means when building with the same llvm version as the openmp runtime code. Perhaps the right fix is to only build the gpu plugins when we can also build the gpu devicertl, as well as having the new dependency edge this patch proposed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123177