- Notifications
You must be signed in to change notification settings - Fork190
Documentation changes for wait_parity/try_wait_parity#190
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| __global__ void example_kernel(cuda::barrier<cuda::thread_scope_block>& bar) { | ||
| bar.wait_parity(false); | ||
| } | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
It would be good to have an example that shows how to use these APIs to accomplish something minimally useful.
It might not be clear to the user what these APIs are for, what the parity of the barrier is initially, how to get the parity of a phase, etc.
| __host__ __device__ bool cuda::std::barrier::try_wait_parity(bool phase); | ||
| ``` | ||
| `barrier::wait_parity` stalls execution while the barrier is not at the specified parity. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We should swap it to match atomic wait. Also make sense as we are waiting for phase to change
jrhemstad commentedMay 15, 2023
@wmaxey is this still relevant? Are these APIs still around? |
No description provided.