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 framework for unified spec #616

Merged
merged 2 commits into from
Sep 8, 2024

Conversation

gmlueck
Copy link
Contributor

@gmlueck gmlueck commented Sep 8, 2024

  • Add a section to the introduction explaining that this document is a
    specification for SYCL 2020 and SYCL-Next and also for the KHR
    extensions.

  • Add a (mostly empty) appendix that will hold the future KHR
    extensions.

  • Change some references to "SYCL 2020" to more general wording, since
    they apply also to SYCL-Next.

  • Remove references to "SYCL Next" and use the Asciidoc SYCL_VERSION
    attribute instead.

This corresponds to internal MR 755

* Add a section to the introduction explaining that this document is a
  specification for SYCL 2020 and SYCL-Next and also for the KHR
  extensions.

* Add a (mostly empty) appendix that will hold the future KHR
  extensions.

* Change some references to "SYCL 2020" to more general wording, since
  they apply also to SYCL-Next.

* Remove references to "SYCL Next" and use the Asciidoc `SYCL_VERSION`
  attribute instead.
It seems confusing to have an appendix with no KHR extensions.  I think
we will have at least one by the time this spec is published, but just
to be safe add a sentence stating that there are no KHR extensions yet
in the appendix.  We'll remove this sentence when we have our first KHR.
@gmlueck
Copy link
Contributor Author

gmlueck commented Sep 8, 2024

Merging as the WG already approved this change when it was an internal MR.

@gmlueck gmlueck merged commit a222068 into KhronosGroup:main Sep 8, 2024
3 checks passed
@gmlueck gmlueck deleted the gmlueck/unified-spec branch September 8, 2024 22:05
keryell pushed a commit that referenced this pull request Sep 10, 2024
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.

1 participant