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

Generate OpenCL C feature dictionary #1212

Merged
merged 3 commits into from
Aug 13, 2024

Conversation

kpet
Copy link
Contributor

@kpet kpet commented Jul 17, 2024

Features are stored in a text file for now. Ultimately, we probably want to use the XML registry for this.

Generation script taken from #1174 with a few modifications.

Contributes to #1166.

Change-Id: Ie2c14148d75457030aa1a97cf601daba2c007397

Features are stored in a text file for now. Ultimately, we probably
want to use the XML registry for this.

Generation script taken from KhronosGroup#1174 with a few modifications.

Contributes to KhronosGroup#1166.

Signed-off-by: Ben Ashbaugh <[email protected]>
Signed-off-by: Kevin Petit <[email protected]>
Change-Id: Ie2c14148d75457030aa1a97cf601daba2c007397
@kpet kpet requested a review from bashbaug July 17, 2024 11:34
Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

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

This is a nice improvement!

Once this goes in I might look at generalizing it for other uses in the OpenCL C spec. For example, it might be useful to have dictionaries for long built-in function names, to fix #1214.

c/features.txt Outdated Show resolved Hide resolved
scripts/gen_c_feature_dictionary.py Outdated Show resolved Hide resolved
kpet and others added 2 commits August 9, 2024 11:06
Signed-off-by: Kevin Petit <[email protected]>
Change-Id: I8e0947c30775338dd70803d09c7059d340e86f5a
Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

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

Thanks!

@bashbaug bashbaug merged commit 47dd749 into KhronosGroup:main Aug 13, 2024
2 checks passed
@kpet kpet deleted the generate-c-feature-dict branch August 14, 2024 14:10
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