Skip to content

[SYCL][Doc] Add draft of sycl_ext_oneapi_prefetch #9282

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

Merged
merged 13 commits into from
May 30, 2023

Conversation

Pennycook
Copy link
Contributor

Adds prefetch and joint_prefetch functions for providing prefetch hints associated with specific levels of cache.

Adds prefetch and joint_prefetch functions for providing prefetch hints
associated with specific levels of cache.

Signed-off-by: John Pennycook <john.pennycook@intel.com>
@Pennycook Pennycook added the spec extension All issues/PRs related to extensions specifications label May 2, 2023
@Pennycook Pennycook requested a review from a team as a code owner May 2, 2023 16:32
Pennycook added 2 commits May 8, 2023 08:42
Prefetching into the lowest level of cache is consistent with the behavior of
other prefetch extensions. On some hardware, prefetching into the lowest level
of cache will populate all levels of cache, and many developers may expect
this to be the default behavior.
Allows for additional information to be specified alongside a level of cache.
The current proposal provides only a "nontemporal" hint that could be
meaningful on multiple devices; additional extensions could define their own
hints (e.g. pvc::uncached).
@Pennycook
Copy link
Contributor Author

One open question for reviewers: Does it make sense to have cache_level as an enum, or should it just be a size_t?

The former is nice from a readability perspective, because "L1" is clearly a level of cache, whereas "1" could be anything. But the latter is nice from a verbosity perspective, because syclex::prefetch_hint<syclex::cache_level::L1, ...> is much more typing than syclex::prefetch_hint<1, ...>.

@Pennycook Pennycook requested a review from gmlueck May 17, 2023 14:05
@Pennycook
Copy link
Contributor Author

@bader - I think this can be merged, now.

@bader bader merged commit 034ac09 into intel:sycl May 30, 2023
@Pennycook Pennycook deleted the sycl_ext_oneapi_prefetch branch May 30, 2023 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec extension All issues/PRs related to extensions specifications
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants