[Openmp-commits] [PATCH] D102883: [OpenMP] Define named constants for interop's foreign runtime ID

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a subscriber: sriharikrishna.
jdoerfert added a comment.

In D102883#2791123 <https://reviews.llvm.org/D102883#2791123>, @JonChesterfield wrote:

> Does anyone have a reference for what we're supposed to do with one of these things?
>
> Amdgpu openmp isn't implemented in terms of any of those (it is a layer on HSA), but it's possible we can cobble together enough of a HIP or opencl 'instance' on the fly to satisfy the interface. I can't find any information on what that interface is meant to be though.

It's pretty much up to the impl what this means. The extra document lists what the user should expect for those foreign runtimes. You can ask them to add HSA without problems, defining what the results of various calls should be (in terms of type).

Also @sriharikrishna is working on getting the early prototype for interop up and running. We'll get some patches hopefully soon. The plugins can then implement their own getters eventually.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102883