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

Vulkan support #353

Closed
jclc opened this issue Aug 25, 2018 · 5 comments
Closed

Vulkan support #353

jclc opened this issue Aug 25, 2018 · 5 comments
Labels
enhancement This issue asks for enhancement in a specific part

Comments

@jclc
Copy link
Contributor

jclc commented Aug 25, 2018

Support for Vulkan functions. Compatibility with https://github.com/vulkan-go/vulkan would be very nice. Could Vulkan handles be transferred between packages as uintptrs? I'm not sure how two separate CGO-enabled packages will handle the same types.

@veeableful
Copy link
Contributor

I think both packages need to type alias the handles as uintptr which requires Go 1.9. Maybe there's a hack that can be used for forcing a cast between the two types but I have no clue what the hack is..

@veeableful veeableful added enhancement This issue asks for enhancement in a specific part solved? This issue might have been solved but not tested labels Aug 25, 2018
@jclc
Copy link
Contributor Author

jclc commented Aug 25, 2018

This isn't quite solved yet. The only function I couldn't test properly is VulkanCreateSurface. I'll need to make sure go-sdl2 can interface with vulkan-go first. I'll probably have to change the Go wrapper function signature to take a uintptr instead of the actual Vk handle.

@aqiank aqiank removed the solved? This issue might have been solved but not tested label Aug 26, 2018
@jclc jclc mentioned this issue Aug 27, 2018
@Pixdigit
Copy link

Doesnt SDL support Vulkan natively?

@jclc
Copy link
Contributor Author

jclc commented Aug 28, 2018

It does, but you need another library to load and use the Vulkan functions.

@jclc
Copy link
Contributor Author

jclc commented Aug 29, 2018

My patches are now in vulkan-go/vulkan, you can test Vulkan support yourself with go get github.com/vulkan-go/demos/vulkancube/vulkancube_sdl2

@jclc jclc closed this as completed Aug 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This issue asks for enhancement in a specific part
Projects
None yet
Development

No branches or pull requests

4 participants