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

Adding Vulkan support? #12

Open
Ulfgard opened this issue May 7, 2018 · 2 comments
Open

Adding Vulkan support? #12

Ulfgard opened this issue May 7, 2018 · 2 comments

Comments

@Ulfgard
Copy link
Member

Ulfgard commented May 7, 2018

Is possible. The computational model fits quite well for compute queues and the interop with graphics is naturally quite a lot better. It could maybe give better performance as one can write really gpu dependent code.

@TGlas
Copy link
Contributor

TGlas commented May 8, 2018

Sure, but it would be a huge overhead. Imho we should better wait for an abstraction layer for the compute functionality to pop up somewhere.

@Ulfgard
Copy link
Member Author

Ulfgard commented May 8, 2018

the overhead is not as big as it might sound. Remora is pretty extensible. And it would be a strong selling point.

here are the steps needed

  1. rename gpu_tag to opencl_tag, add vulkan_tag
  2. move all ///gpu folders -> opencl folder, fix all includes
  3. add the kernels/vulkan implementation (around 2k loc?), include them in kernels/*.hpp
  4. implement the remora/vulkan folder (around 3k loc maybe some wrapper code is more expensive idk,).

this also holds for cuda support, etc.

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

2 participants