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.

Sign up for GitHub

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

Jump to bottom

Check forbidden kernel constructs in constant evaluated context #976

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Check forbidden kernel constructs in constant evaluated context #976

wants to merge 5 commits into from

Conversation

keryell
Copy link
Member

@keryell keryell commented Oct 30, 2024

There are a lot of restrictions for kernel code, except when present in a constant evaluated context.

This is a test corresponding to the specification clarification introduced by PR KhronosGroup/SYCL-Docs#388

@keryell keryell added the enhancement New feature or request label Oct 30, 2024
@keryell keryell requested a review from a team as a code owner October 30, 2024 21:52
@keryell
Check forbidden kernel constructs in constant evaluated context
b3ec4b2 
There are a lot of restrictions for kernel code, except when present in a
constant evaluated context.

This is a test corresponding to the specification clarification introduced by PR
KhronosGroup/SYCL-Docs#388
@keryell keryell mentioned this pull request Oct 31, 2024
Merged
@tomdeakin
Copy link
Contributor

KhronosGroup/SYCL-Docs#388 was approved to merge.
This is just waiting the temporary disable.

@steffenlarsen
Copy link
Contributor

With #987 merged, the DPC++ CI should be passing. @keryell - I took the liberty of updating the branch. Be aware of the need for a local pull. 😄

@bader bader requested a review from gmlueck November 25, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@steffenlarsen steffenlarsen steffenlarsen left review comments

@bader bader bader left review comments

@gmlueck gmlueck gmlueck approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@keryell @tomdeakin @steffenlarsen @bader @gmlueck