Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Documentation of warp-wide collectives refers to __syncthreads instead of __syncwarp #677

Closed
fkallen opened this issue Apr 18, 2023 · 1 comment · Fixed by #722
Closed

Documentation of warp-wide collectives refers to __syncthreads instead of __syncwarp #677

fkallen opened this issue Apr 18, 2023 · 1 comment · Fixed by #722
Assignees
@gevtushenko

Comments

@fkallen
Copy link
Contributor

fkallen commented Apr 18, 2023 •
edited
Loading

For example in https://nvlabs.github.io/cub/class_warp_exchange.html#a078092b662bf8cdc67c2322d71f0a776

A subsequent __syncthreads() threadblock barrier should be invoked after calling this method if the collective's temporary storage (e.g., temp_storage) is to be reused or repurposed.
@gevtushenko
Copy link
Collaborator

@fkallen thank you for reporting this! We should fix this after the following PR is merged.

@gevtushenko gevtushenko self-assigned this Jun 21, 2023
@gevtushenko gevtushenko moved this to In Progress in CCCL Jun 21, 2023
@gevtushenko gevtushenko mentioned this issue Jun 21, 2023
Merged
@gevtushenko gevtushenko moved this from In Progress to In Review in CCCL Jun 21, 2023
@gevtushenko gevtushenko linked a pull request Jun 21, 2023 that will close this issue
Improve docs for warp-wide primitives #722
Merged
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Jun 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gevtushenko gevtushenko

Labels
None yet
Projects
CCCL
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve docs for warp-wide primitives gevtushenko/cub
2 participants
@gevtushenko @fkallen