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

[SYCL][Doc][RTC] Document [un]supported features and build_options #17459

Merged
merged 4 commits into from
Mar 19, 2025

Conversation

jopperm
Copy link
Contributor

@jopperm jopperm commented Mar 14, 2025

Update the non-normative section in the kernel_compiler extension to reflect what's currently supported in DPCPP.

@jopperm jopperm self-assigned this Mar 14, 2025
@jopperm jopperm requested a review from a team as a code owner March 14, 2025 03:25
Comment on lines +1068 to +1072
The `kernel_compiler` implementation in {dpcpp} supports persistent caching. To
enable it, set the the environment variable `SYCL_CACHE_PERSISTENT=1`. The
location of the cache can be changed by setting `SYCL_CACHE_DIR`. Refer to
https://intel.github.io/llvm/design/KernelProgramCache.html#persistent-cache for
more details on how to control the cache.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has this been reviewed somewhere else, perhaps as part of the design of the run-time compiler?

I'm a little surprised that JIT compilation of kernels embedded in the binary would use the same cache as explicit run-time compilation of kernels. I would have expected separate controls.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The implementation PR is #16823 and has been approved by @cperkinsintel.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, thanks for confirming that.

jopperm added 3 commits March 17, 2025 06:00
Signed-off-by: Julian Oppermann <[email protected]>
Signed-off-by: Julian Oppermann <[email protected]>
@sommerlukas sommerlukas merged commit 9041d78 into intel:sycl Mar 19, 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