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

[Spec change] Fix description of sycl::free #1060

Open
sycl-issue-bot bot opened this issue Feb 27, 2025 · 1 comment
Open

[Spec change] Fix description of sycl::free #1060

sycl-issue-bot bot opened this issue Feb 27, 2025 · 1 comment

Comments

@sycl-issue-bot
Copy link

Please review whether KhronosGroup/SYCL-Docs#758 by @Pennycook requires any changes to the CTS.

If changes are required: Open a new PR addressing the changes and link it to this issue.
If no changes are required: Close this issue and proceed with the spec PR.

@Pennycook
Copy link
Contributor

I don't think this is an easy one to answer, because there may be CTS tests that rely on specific behaviors of sycl::free.

My recommendation would be to close this issue because no CTS test changes are required now, but with the expectation that we may need to adjust some tests later if an implementation fails a test due to the test making an invalid assumption. The alternative would be to review every CTS test to ensure that every usage of sycl::free is safe, which is a non-trivial amount of work.

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

No branches or pull requests

1 participant