Skip to content

Store any_resource members in polymorphic_allocator, thrust_allocator, and device_check_resource_adaptor#2340

Merged
bdice merged 1 commit intorapidsai:stagingfrom
bdice:any-resource-members-pr
Mar 31, 2026
Merged

Store any_resource members in polymorphic_allocator, thrust_allocator, and device_check_resource_adaptor#2340
bdice merged 1 commit intorapidsai:stagingfrom
bdice:any-resource-members-pr

Conversation

@bdice
Copy link
Copy Markdown
Collaborator

@bdice bdice commented Mar 31, 2026

Description

Replace device_async_resource_ref members with cuda::mr::any_resource<device_accessible> in polymorphic_allocator, thrust_allocator, and device_check_resource_adaptor. This eliminates the CCCL #8037 recursive constraint cycle for these classes.

Constructor signatures are unchanged; they still accept device_async_resource_ref, which implicitly converts to any_resource.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

…, and device_check_resource_adaptor

Replace device_async_resource_ref members with
cuda::mr::any_resource<device_accessible> in polymorphic_allocator,
thrust_allocator, and device_check_resource_adaptor. This eliminates
the CCCL #8037 recursive constraint cycle for these classes.

Constructor signatures are unchanged; they still accept
device_async_resource_ref, which implicitly converts to any_resource.
@bdice bdice requested a review from a team as a code owner March 31, 2026 20:49
@bdice bdice requested review from rongou and ttnghia and removed request for a team March 31, 2026 20:49
@bdice bdice added breaking Breaking change improvement Improvement / enhancement to an existing function labels Mar 31, 2026
@bdice bdice self-assigned this Mar 31, 2026
@bdice bdice merged commit 933c075 into rapidsai:staging Mar 31, 2026
86 of 87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change improvement Improvement / enhancement to an existing function

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants