Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Potential to expand support for HIP Apps? #8

Open
parallelo opened this issue Apr 21, 2017 · 1 comment
Open

Potential to expand support for HIP Apps? #8

parallelo opened this issue Apr 21, 2017 · 1 comment

Comments

@parallelo
Copy link

Hi,

Just curious how challenging it would be to add support for HIP applications. Any thoughts?

Thanks,

Jeff

@jlgreathouse
Copy link
Contributor

Sorry for the delay in getting to this.

From a design perspective, I don't think this would be too hard to support. It would mostly involve another wrapper library, like the OpenCL wrapper contained in ./src/cl_wrapper/ . However, I expect that actually implementing the checkers will lead to a number of roadblocks that would add up to a reasonable amount of work.

For instance, most of our OpenCL overflow checkers are configured to be be able to run on the GPU immediately after the user kernel runs. We take advantage of the fact that the user kernel will require a work queue and working OpenCL device, so we don't allocate our own. As such, we would need to port all of this to work in a HIP/HCC environment as well.

In any case, I believe this is a good feature request but I can't give an estimate as to when/if it would be done.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants