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

Better support multidevice placement with stream.async.barrier #19651

Merged
merged 3 commits into from
Jan 13, 2025

Conversation

rsuderman
Copy link
Contributor

Barriers / transfers should have semantics that attempt to parallelize partitioning. If a value has a barrier placed it should divide partitions to avoid spaning behavior with cross device dependencies.

Intermediate and ending transfers we want to place on the producing partition so that any produced operator ends by producing the value at the needed desetination

For incoming transfers we place in the destination partition as these will not add a dependency on the incoming data.

Barriers / transfers should have semantics that attempt to parallelize
partitioning. If a value has a barrier placed it should divide
partitions to avoid spaning behavior with cross device dependencies.

Intermediate and ending transfers we want to place on the producing
partition so that any produced operator ends by producing the value at
the needed desetination

For incoming transfers we place in the destination partition as these
will not add a dependency on the incoming data.
@rsuderman rsuderman merged commit 40c19e3 into iree-org:main Jan 13, 2025
33 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.

2 participants