-
Notifications
You must be signed in to change notification settings - Fork 31
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
Consider using gfx-rs for gpu compute #12
Comments
Interesting, but it looks like it's currently a bit of a pain to use. I'd also have to do the shader compilation manually. I'll be interested to see the project develop, though. Hopefully someone makes a higher level wrapper for it. |
Supporting DirectX would probably mean that GPU support would be easily usable by Windows users. Removing incompatibilities between compilers would also be a great benefit. I'm now fully in favor of this. |
As an added bonus, I think gfx would like having an example of being used "in production" as well :D |
I stumbled apon this repo earlier while looking to see if anyone had done ed25519 signature verification via gpu compute. If you're still interested in this issue, then a first step might be to compile the opencl kernel with clspv. When I ran
After either setting the types or deleting all the functions with Possibly related to google/clspv#261. |
I tried this out probably a year ago but gfx-rs's compute support was not ready for real usage at the time. It's very possible the situation's improved since then though. Thanks for linking clspv, I hadn't seen it before. Writing this stuff in OpenGL is a real pain given the lack of dynamic memory access. At one point I explored autogenerated OpenGL but ran into shader size limitations. It seems like the best path forward would be clspv with the current OpenCL kernel. I think I can clean up at least that error without too much trouble. |
If you want to chat about this more, I'm on Discord at |
https://github.com/gfx-rs/gfx/tree/master/examples/hal/compute
Could make it easier to compile
The text was updated successfully, but these errors were encountered: