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

Allow applications to access the loaded function pointers #952

Open
wants to merge 1 commit into
base: master
from

Conversation

@branan
Copy link
Contributor

branan commented Apr 26, 2018

Without this, an application has to create their own loader for the Vulkan function pointers if they need to drop down to raw Vulkan calls. It's not a lot of effort to load them, but it's nicer to just have them available on the Instance or Device rather than having to pass an extra parameter around for function pointer access.

Without this, an application has to create their own loader for the
Vulkan function pointers if they need to drop down to raw Vulkan
calls. It's not a lot of effort to load them, but it's nicer to just
have them available on the Instance or Device rather than having to
pass an extra parameter around for function pointer access.
@branan

This comment has been minimized.

Copy link
Contributor Author

branan commented Apr 26, 2018

This will probably interact with #89 in weird ways sometime in the future, but for now I'm just trying to toss up PRs for any patches I'm carrying in my private fork.

@knappador

This comment has been minimized.

Copy link
Contributor

knappador commented Apr 15, 2019

I think we can merge this, and I support it since reducing the Vulkan API is pretty difficult without losing some power. #89 shouldn't cause major issues because if the pointers no longer exist, they will get picked up by the compiler and replaced with the functions to get them. @branan care to update and then ping Rukai for a merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.