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

Add vk_function_loader.hpp #152

Conversation

charles-lunarg
Copy link
Collaborator

@charles-lunarg charles-lunarg commented Oct 16, 2023

Adds a new header only utility library which contains a loader for the Vulkan-Loader,
tables to hold vulkan function pointers, and init functions to fill them out.

This is for #73 - will close the issue when VVL & VEL are using this library.

Copy link
Contributor

@juan-lunarg juan-lunarg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to see a branch with the functionality implemented in VEL or VVL.

tests/test_setting_env.cpp Outdated Show resolved Hide resolved
@charles-lunarg
Copy link
Collaborator Author

Note that this PR relies on #155 so I will rebase when that is merged.

@charles-lunarg charles-lunarg force-pushed the add_function_loader_helper branch 2 times, most recently from 8d15953 to 01e678b Compare October 17, 2023 22:57
Adds a new header only utility library which contains a loader for the Vulkan-Loader,
tables to hold vulkan function pointers, and init functions to fill them out.
@charles-lunarg charles-lunarg force-pushed the add_function_loader_helper branch from 01e678b to c846501 Compare October 19, 2023 00:45
@juan-lunarg juan-lunarg dismissed their stale review October 19, 2023 17:24

Looks good now. But I want Spencer's opinion since this affect VVL/VEL testing.

@charles-lunarg
Copy link
Collaborator Author

Opened this up a month ago, but the reality is that the solution is to use volk, not write another new library. If there is future interest in library for function loading in this repo, it can be revisited.

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

Successfully merging this pull request may close these issues.

2 participants