Skip to content

Context: add new constructor, and check for throw#1171

Merged
bader merged 9 commits intomainfrom
context_new_constructor
Feb 2, 2026
Merged

Context: add new constructor, and check for throw#1171
bader merged 9 commits intomainfrom
context_new_constructor

Conversation

@TApplencourt
Copy link
Copy Markdown
Contributor

For KhronosGroup/SYCL-Docs#628 and fix #947

(I did add the test for throw because I didn't read the issue...)

@TApplencourt TApplencourt requested a review from a team as a code owner January 13, 2026 22:50
@TApplencourt
Copy link
Copy Markdown
Contributor Author

Fail due to Sim-SYCL (expected AFAIK)

@TApplencourt TApplencourt force-pushed the context_new_constructor branch from 338bbad to 82334df Compare January 14, 2026 22:38
@keryell keryell changed the title Context add new constructor, and checl for throw Context: add new constructor, and check for throw Jan 15, 2026
@TApplencourt
Copy link
Copy Markdown
Contributor Author

Thomas: Will disable the new test for SimSycl (or implement it there)

@TApplencourt
Copy link
Copy Markdown
Contributor Author

(Note to myself: simsycl take super long to run, maybe we forgot to run them in // )

@TApplencourt
Copy link
Copy Markdown
Contributor Author

I did it \o/ @bader you can merge at your convenience :)

@bader bader merged commit bfa9472 into main Feb 2, 2026
11 checks passed
@bader bader deleted the context_new_constructor branch February 2, 2026 18:07
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.

[Spec change] Fix contradiction with context constructors

3 participants