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 a few missing enums to the XML file #1118

Merged
merged 6 commits into from
Apr 2, 2024

Conversation

bashbaug
Copy link
Contributor

Fixes #1092

Adds a few missing enums to the XML file.

Also, use the feature dictionaries in a lot more places, especially in the OpenCL C and OpenCL SPIR-V environment specs.

use the feature dictionaries in a lot more places
@bashbaug bashbaug requested a review from alycm March 31, 2024 19:08
@bashbaug
Copy link
Contributor Author

This isn't quite doing what I want it to do, and it's treating the macros for things like CL_VERSION_MAJOR to be types. Converting to draft while I figure this out.

@bashbaug bashbaug marked this pull request as draft March 31, 2024 21:32
@bashbaug
Copy link
Contributor Author

OK, I had to add a section to the dictionary generation script for "macros", but I think this is doing what I want it to do now, so I am marking this ready for review again.

@bashbaug bashbaug marked this pull request as ready for review March 31, 2024 22:06
@oddhack oddhack mentioned this pull request Apr 1, 2024
Copy link
Contributor

@alycm alycm left a comment

Choose a reason for hiding this comment

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

LGTM.

@alycm alycm merged commit 2515b1d into KhronosGroup:main Apr 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

missing XML file entries
2 participants