Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HIP support for NVRTC? #268

Closed
markehammons opened this issue Nov 19, 2017 · 8 comments
Closed

HIP support for NVRTC? #268

markehammons opened this issue Nov 19, 2017 · 8 comments
Assignees

Comments

@markehammons
Copy link

As far as I've seen so far, HIP does not support runtime compilation like opencl does or NVRTC allows cuda to do. Will this be added soon? I'm using java and looking into writing a jnr-ffi binding to HIP, but I wouldn't be able to test it much on my work system since nvcc requires a lower GCC version than I have, and I rely on nvrtc for compiling .cu kernels.

@markehammons
Copy link
Author

Apparently, HIP is just a shim over nvcc/opencl libs, and not an actual library. Unfortunately that means I can't make a jnr-ffi binding to it, but if there's a hiprtc library released, i could possibly create a java lib that mimics hip and uses the hip kernel language.

@bensander
Copy link
Contributor

HIP uses NVCC on the CUDA path and HCC/ROCR runtime for the ROCm side. ROCR is an implementation of the HSA runtime APIs, plus some extensions.
To answer your original question : HIP does not provide a fully integrated runtime compilation. Depending on use case, you might be able to approximate this with a process-level call to hipcc?

@shoshijak
Copy link

In the meantime, hiprtc has been added to HIP, thus making it possible to compile kernels at runtime on AMD GPUs 🎉

Unfortunately however, it isn't possible to compile any of the NVRTC functionalities with HIP targeting NVIDIA GPUs at the moment because of a missing header: hip/nvcc_detail/nvrtc.h.

Are you planning on adding this feature anytime soon?

@yashkgp
Copy link

yashkgp commented Feb 25, 2021

Hey Guys,
Is there any update on the missing header file : hip/nvcc_detail/nvrtc.h ?

@sunway513
Copy link
Contributor

Hi @gargrahul , could HIP add that missing header? Thanks..

@gargrahul
Copy link
Contributor

@sunway513 Thanks for following. We are working on fixing it.

@nicholasmalaya
Copy link
Contributor

This issue was fixed by AMD in ROCm 4.3. The missing header file is available and we believe this github issue can be closed.

@nartmada
Copy link

Closing the ticket as the issue has been fixed in ROCm 4.3. @markehammons, please re-open if your issue is still not addressed with latest ROCm 6.0.2. Thanks.

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

No branches or pull requests

9 participants