Skip to content

[Backport branch/3.4.x] [libcu++] Fix default make_shared_resource construction#9047

Merged
wmaxey merged 1 commit into
branch/3.4.xfrom
backport-9044-to-branch/3.4.x
May 21, 2026
Merged

[Backport branch/3.4.x] [libcu++] Fix default make_shared_resource construction#9047
wmaxey merged 1 commit into
branch/3.4.xfrom
backport-9044-to-branch/3.4.x

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Description

Backport of #9044 to branch/3.4.x.

* [libcu++] Test default make_shared_resource construction

* [libcu++] Fix default make_shared_resource construction

* [libcu++] Fix shared_block_ptr in_place construction ambiguity

* Fix clang-format

(cherry picked from commit 2a660dc)
@github-actions github-actions Bot requested a review from a team as a code owner May 17, 2026 15:14
@github-actions github-actions Bot requested a review from wmaxey May 17, 2026 15:14
@github-project-automation github-project-automation Bot moved this to Todo in CCCL May 17, 2026
@copy-pr-bot
Copy link
Copy Markdown
Contributor

copy-pr-bot Bot commented May 17, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bdice
Copy link
Copy Markdown
Contributor

bdice commented May 17, 2026

RAPIDS will need this bugfix backported to the 3.4 release. RMM tests are blocked without it (cannot construct a shared resource for a type that has no constructor parameters).

@github-project-automation github-project-automation Bot moved this from Todo to In Review in CCCL May 18, 2026
@bdice
Copy link
Copy Markdown
Contributor

bdice commented May 18, 2026

/ok to test e11e8a1

@github-actions
Copy link
Copy Markdown
Contributor Author

🥳 CI Workflow Results

🟩 Finished in 2h 13m: Pass: 100%/113 | Total: 3d 00h | Max: 2h 12m | Hits: 81%/414385

See results here.

rapids-bot Bot pushed a commit to rapidsai/rmm that referenced this pull request May 18, 2026
There is a bug in CCCL's `make_shared_resource<T>()` that doesn't work with types that have no constructor arguments. NVIDIA/cccl#9043

NVIDIA/cccl#9044 fixes it, but we need a workaround to unblock CI until that is backported (NVIDIA/cccl#9047) and rapids-cmake is updated.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - David Wendt (https://github.com/davidwendt)

URL: #2406
@leofang
Copy link
Copy Markdown
Member

leofang commented May 19, 2026

@jrhemstad for vis

@wmaxey wmaxey merged commit e0c0e12 into branch/3.4.x May 21, 2026
134 checks passed
@wmaxey wmaxey deleted the backport-9044-to-branch/3.4.x branch May 21, 2026 21:02
@github-project-automation github-project-automation Bot moved this from In Review to Done in CCCL May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants