Skip to content

Conversation

KornevNikita
Copy link
Contributor

The extension was enabled, moving to the experimental dir

The extension was enabled, moving to the experimental dir
@KornevNikita KornevNikita requested a review from a team as a code owner September 22, 2025 10:57
@gmlueck
Copy link
Contributor

gmlueck commented Sep 25, 2025

Please update the Status text using the boilerplate here:

https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/template.asciidoc#status

@KornevNikita
Copy link
Contributor Author

@gmlueck oh, right, updated. BTW could you please clarify - how do we define a status as experimental? OpenCL feature cl_khr_kernel_clock is finalized since v3.0.18. Since this feature doesn't use an experimental feature under the hood, should it also be marked as supported instead?

@gmlueck
Copy link
Contributor

gmlueck commented Sep 26, 2025

BTW could you please clarify - how do we define a status as experimental?

I could go either way on this one. When we first started the specification, it seemed like there were a lot of unresolved issues, and that was a good reason to keep it experimental. At this point, I think those issues have been resolved, though.

Do we have a specific customer/user who is asking for this one? Do we expect to get any feedback from them?

Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@KornevNikita
Copy link
Contributor Author

@gmlueck it's a request (CMPLRLLVM-50746) from syclomatic folks to create an alternative for cuda's clock(). Since we just introduced the feature, I believe they'll find some bugs/feedback to share. From this POV maybe we better keep it experimental until they verify it works as expected.

Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@KornevNikita
Copy link
Contributor Author

@intel/llvm-gatekeepers please merge

@steffenlarsen steffenlarsen merged commit c355a3d into intel:sycl Sep 30, 2025
3 checks passed
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.

3 participants