-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL] Remove old extension specifications #5381
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
Conversation
All of these extensions were adopted into SYCL 2020, so they are no longer a language extension. The SYCL 2020 specification is a more accurate description of these features, so remove these old versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. I spotted a couple of things that might need follow-up actions, but I'm not sure.
sycl/doc/extensions/KernelRHSAttributes/SYCL_INTEL_attribute_style.asciidoc
Show resolved
Hide resolved
sycl/doc/extensions/UnnamedKernelLambda/SYCL_INTEL_unnamed_kernel_lambda.asciidoc
Show resolved
Hide resolved
This was also adopted into SYCL 2020.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Add a table of the extensions that were incorporated into SYCL 2020.
Ping to reviewers: Could I get re-approvals now that I have added the table of removed extensions in "sycl/doc/extensions/removed/README.md" as @steffenlarsen proposed? |
All of these extensions were adopted into SYCL 2020, so they are no
longer a language extension. The SYCL 2020 specification is a more
accurate description of these features, so remove these old versions.