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][Graph] Relieve users of dependency managment for multiple command-buffer submissions #345

Closed
wants to merge 1 commit into from

Conversation

mfrancepillois
Copy link
Collaborator

@mfrancepillois mfrancepillois commented Nov 28, 2023

Synchronizations between multiple submission of a graph are managed by the runtime.
The dependency to previous submission is automatically added when a graph is resubmitted.
Updates spec and tests accordingly.

@mfrancepillois mfrancepillois added the Graph Implementation Related to DPC++ implementation and testing label Nov 28, 2023
@mfrancepillois mfrancepillois changed the title [SYCL][Graph] Release host sync for multiple command-buffer submissions [SYCL][Graph] Relieve users of dependency managment for multiple command-buffer submissions Dec 1, 2023
Copy link
Collaborator

@EwanC EwanC left a comment

Choose a reason for hiding this comment

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

LGTM apart from some language nitpicks

sycl/doc/design/CommandGraph.md Outdated Show resolved Hide resolved
sycl/doc/design/CommandGraph.md Outdated Show resolved Hide resolved
sycl/source/detail/graph_impl.cpp Outdated Show resolved Hide resolved
…fer submissions

Synchronizations between multiple submission of a graph are managed by the runtime.
The dependency to previous submission is automatically added when a graph is resubmitted.
Updates spec and tests accordingly.
@mfrancepillois
Copy link
Collaborator Author

Upstream PR intel#12063

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Graph Implementation Related to DPC++ implementation and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants