Skip to content

[CTK 12.5]: Avoid another compiler issue with host device detection #4971

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

Merged
merged 3 commits into from
Mar 14, 2025

Conversation

miscco
Copy link
Contributor

@miscco miscco commented Mar 13, 2025

NVCC seems to also incorrectly deduce this in 12.5

NVCC seems to also incorrectly deduce this in 12.5
@miscco miscco requested a review from a team as a code owner March 13, 2025 07:36
@miscco
Copy link
Contributor Author

miscco commented Mar 13, 2025

This is the last issue in getting rapids to build with CCCL main see https://github.com/NVIDIA/cccl/actions/runs/13822026368/job/38669398923?pr=4123#step:6:12069

@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Mar 13, 2025
@miscco miscco changed the title Avoid another compiler issue with host device detection [CTK 12.5]: Avoid another compiler issue with host device detection Mar 13, 2025
@ChuckHastings
Copy link
Collaborator

/merge

@rapids-bot rapids-bot bot merged commit 6759645 into rapidsai:branch-25.04 Mar 14, 2025
79 checks passed
miscco added a commit to miscco/cugraph that referenced this pull request Mar 19, 2025
rapids-bot bot pushed a commit that referenced this pull request Mar 19, 2025
We found the issue to be in some unclear compile issues within libcu++ `not_fn`, so we can revert all those hacks


Revert "Silence compiler warnings about host device destructor (#4960)"
Revert "[CTK 12.5]: Avoid another compiler issue with host device detection (#4971)"

Authors:
  - Michael Schellenberger Costa (https://github.com/miscco)

Approvers:
  - Seunghwa Kang (https://github.com/seunghwak)

URL: #4985
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cuGraph improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants