[Openmp-commits] [PATCH] D101213: [libomptarget] Enable AMDGPU devicertl

[Jon Chesterfield via Phabricator via Openmp-commits]

JonChesterfield added a comment.

There are some interesting configuration choices.

In particular, we can build the nvptx or amdgcn device bitcode with a (recent/same-commit) clang, even if the llvm doesn't have either of those targets enabled. If that same clang is later used to build an application, it (hopefully!) errors about the missing target at that point.

We currently use the existence of cuda to control whether to run tests on nvptx. That's not strictly the right test, a machine may have cuda installed but no nvptx hardware.

I'm trying to get a simplified cmake (based on llvm-config) to run, will hold off on committing this until I can tell if that's going to work out.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101213