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] Optimize OOO profiling tag when profiling is enabled #16116

Draft
wants to merge 1 commit into
base: sycl
Choose a base branch
from

Conversation

steffenlarsen
Copy link
Contributor

The case where profiling tags are enqueued onto a queue that already has profiling enabled, the timestamp enqueue is redundant, as the event resulting from the barrier would be sufficient. This commit short-circuits this case to simply return the event of a barrier submission.

The case where profiling tags are enqueued onto a queue that already has
profiling enabled, the timestamp enqueue is redundant, as the event
resulting from the barrier would be sufficient. This commit
short-circuits this case to simply return the event of a barrier
submission.

Signed-off-by: Larsen, Steffen <[email protected]>
@steffenlarsen
Copy link
Contributor Author

This originates from #16112, but appears to fail on Gen12.

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.

1 participant