Skip to content

Get newer CCCL 2.8 with fix for atomics.#840

Merged
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.06from
bdice:cccl-fix-atomics
May 5, 2025
Merged

Get newer CCCL 2.8 with fix for atomics.#840
rapids-bot[bot] merged 1 commit intorapidsai:branch-25.06from
bdice:cccl-fix-atomics

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented May 5, 2025

Description

This updates CCCL to get NVIDIA/cccl#4616, which was backported to branch/2.8.x.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The cmake-format.json is up to date with these changes.
  • I have added new files under rapids-cmake/
    • I have added include guards (include_guard(GLOBAL))
    • I have added the associated docs/ rst file and update the api.rst

@bdice bdice requested a review from a team as a code owner May 5, 2025 17:30
@bdice bdice added bug Something isn't working non-breaking Introduces a non-breaking change labels May 5, 2025
Copy link
Copy Markdown

@ttnghia ttnghia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on it.

@bdice bdice self-assigned this May 5, 2025
@@ -16,7 +16,7 @@
"version": "2.8.3",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be 2.8.4 now? Or should will we bump this again after an official CCCL release?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will bump this again once a tag comes out. We know we need this fix, with or without a tag.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK sounds good.

@@ -16,7 +16,7 @@
"version": "2.8.3",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK sounds good.

@vyasr
Copy link
Copy Markdown
Contributor

vyasr commented May 5, 2025

/merge

@rapids-bot rapids-bot bot merged commit fe92d93 into rapidsai:branch-25.06 May 5, 2025
15 checks passed
rapids-bot bot pushed a commit to rapidsai/cudf that referenced this pull request May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants