-
Notifications
You must be signed in to change notification settings - Fork 5
Add OpenCL runtime support #24
base: master
Are you sure you want to change the base?
Conversation
35e9965
to
d16d782
Compare
Abstract runtime functionality by HSA (TODO: OCL) Use Requires to load OpenCL bindings Allow choosing runtime via environment variables
d16d782
to
5b12d9f
Compare
Whoooo! |
So far I've gotten kernels to launch through OpenCL, however they currently segfault on the GPU because (as I understand it) we aren't extracting the correct device-side pointer from Key note for reviewers: we (and LLVM) expect our array arguments to be of type |
Note to self: If we do implement a hacky (slow) workaround to getting the device pointer, we should also provide a shortcut via |
Now I've got kernels running without segfaults (see the new |
Do you need to synchronize the memory? |
It doesn't seem like that's the issue since we wait on the kernel's event, and even adding in a |
If anyone has a working ROCm debugger setup, it would be great if we could see what instructions the GPU is actually executing (including memory addresses). I suspect we aren't writing to the correct location. |
Abstract runtime functionality by HSA (TODO: OCL)
Use Requires to load OpenCL bindings
Allow choosing runtime via environment variables
Closes #20, closes #23