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][Docs] Make last event optional for queues with no previous work #16645

Merged

Conversation

steffenlarsen
Copy link
Contributor

This commit changes the extension documentation and implementation of ext_oneapi_get_last_event to return an std::optional and return a std::nullopt for the case where the queue had no work previously submitted to it.

This commit changes the extension documentation and implementation of
ext_oneapi_get_last_event to return an std::optional and return a
std::nullopt for the case where the queue had no work previously
submitted to it.

Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Signed-off-by: Larsen, Steffen <steffen.larsen@intel.com>
Copy link
Contributor

@againull againull left a comment

Choose a reason for hiding this comment

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

Changes look good to me.

I assume that ABI-breakage is ok since extension is experimental.

Copy link
Contributor

@gmlueck gmlueck left a comment

Choose a reason for hiding this comment

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

Spec changes LGTM

@steffenlarsen steffenlarsen merged commit 41ec74c into intel:sycl Jan 16, 2025
18 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