Skip to content

[SYCL] variadic_iterator noexcept #19706

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

Open
wants to merge 2 commits into
base: sycl
Choose a base branch
from

Conversation

cperkinsintel
Copy link
Contributor

@cperkinsintel cperkinsintel commented Aug 4, 2025

variadic_iterator was recently introduced and is used by several routines that are marked as noexcept, some being API. Changing its operator++ to be noexcept to prevent issues and put Coverity's mind at ease

…cept, some being API. Changing its operator++ to be noexcept to prevent issues and put Coverity's mind at ease
Signed-off-by: Chris Perkins <[email protected]>
@cperkinsintel
Copy link
Contributor Author

cperkinsintel commented Aug 5, 2025

Windows unit test SchedulerTests.exe failure is known ( CMPLRLLVM-69554) , flaky, under investigation, and in main syclos branch right now.

I have this PR for the CMPLRLLVM-69554 : #19762 . Once it is merged, then the unit tests on this PR could be re-run and it should be passing ( assuming the CI refreshes the unit test builds and no other conflicts are introduced in the interim)

@cperkinsintel cperkinsintel marked this pull request as ready for review August 5, 2025 20:58
@cperkinsintel cperkinsintel requested a review from a team as a code owner August 5, 2025 20:58
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