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

Delete cuda_async_memory_resource copy/move ctors/operators #860

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

jrhemstad
Copy link
Contributor

Fixes #859

Deletes the copy/move ctors to avoid erroneous sharing of the underlying cudaMemPool_t handle.

Fixes #859

Deletes the copy/move ctors to avoid erroneous sharing of the underlying cudaMemPool_t handle.
@jrhemstad jrhemstad requested a review from a team as a code owner September 1, 2021 00:50
@github-actions github-actions bot added the cpp Pertains to C++ code label Sep 1, 2021
@jrhemstad jrhemstad added breaking Breaking change 3 - Ready for review Ready for review by team bug Something isn't working and removed cpp Pertains to C++ code bug Something isn't working labels Sep 1, 2021
@harrism harrism added the bug Something isn't working label Sep 1, 2021
@harrism
Copy link
Member

harrism commented Sep 1, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 12384bc into branch-21.10 Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for review Ready for review by team breaking Breaking change bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] cuda_async_memory_resource copy ctor needs to be deleted
3 participants