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

clarify that SVM is optional for all OpenCL 3.0 devices #913

Merged
merged 1 commit into from Apr 25, 2023

Conversation

bashbaug
Copy link
Contributor

fixes #912

Clarifies that all forms of SVM including coarse-grained buffer SVM are optional for OpenCL 3.0 devices.

This was stated clearly in most places in the spec but one place was never updated from the OpenCL 2.x specification and still stated that coarse-grained buffer SVM was required.

@bashbaug bashbaug added the OpenCL API Spec Issues related to the OpenCL API specification. label Apr 21, 2023
@bashbaug bashbaug added this to In progress in OpenCL specification maintenance via automation Apr 21, 2023
Copy link
Contributor

@pjaaskel pjaaskel left a comment

Choose a reason for hiding this comment

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

LGTM

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.

OpenCL specification maintenance automation moved this from In progress to Reviewer approved Apr 24, 2023
@bashbaug bashbaug merged commit bbdda64 into KhronosGroup:main Apr 25, 2023
2 checks passed
OpenCL specification maintenance automation moved this from Reviewer approved to Done Apr 25, 2023
@bashbaug bashbaug deleted the svm-is-optional branch April 25, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OpenCL API Spec Issues related to the OpenCL API specification.
Development

Successfully merging this pull request may close these issues.

Is coarse-grained buffer SVM a required OpenCL 3.0 feature?
3 participants