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 Generators #575

Merged

Conversation

tomadamatkinson
Copy link
Collaborator

Based on #574

The commit history here is longer than it needs to be as it includes some of the changes from the upcoming Vulkan Builders. This will hopefully be squashed and rebased before that PR is merged.

This PR adds the ability to generate Vulkan code fairly easily. A small framework on top of the Vulkan Headers generator allows for small fixed-purpose generators to be created with a minimal amount of code. In the future, I aim to add a Vulkan test framework using these generators.

For now, a single generator is used to create the get_structure_type_name helper. This helper is later used in the pNextChain builder which allows for an easy way to collect a pNext chain without dealing with tracking the chain itself. It will be used in the Instance and Device builder to allow samples to append extensions to the create structs dynamically.

See codegen/vulkan/readme.md for basic documentation on the generator itself.

@tomadamatkinson
Copy link
Collaborator Author

@asuessenbach resolved your comments and also renamed the generator and header along the same lines

@tomadamatkinson
Copy link
Collaborator Author

tomadamatkinson commented Jan 10, 2023

Fixed phantom line issue flagged by @asuessenbach. Merging on the original approval as no functional changes were made

@tomadamatkinson tomadamatkinson merged commit 3e6490c into KhronosGroup:framework/v2.0 Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
framework This is relevant to the framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants