Skip to content

do not introduce a pack in a structured binding with nvcc#6994

Merged
ericniebler merged 1 commit intoNVIDIA:mainfrom
ericniebler:nvcc-does-not-support-packs-in-structured-bindings
Dec 17, 2025
Merged

do not introduce a pack in a structured binding with nvcc#6994
ericniebler merged 1 commit intoNVIDIA:mainfrom
ericniebler:nvcc-does-not-support-packs-in-structured-bindings

Conversation

@ericniebler
Copy link
Contributor

Description

cudax/__execution/visit.cuh is using P1061, "Structured Bindings can introduce a Pack", whenever the feature is available.

however, it fails to account for the fact that P1061 is not available when your CUDA compiler is nvcc.

this pr disables the use of packs in structured bindings when the compiler is nvcc.

Checklist

  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Dec 16, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Progress in CCCL Dec 16, 2025
@ericniebler
Copy link
Contributor Author

/ok to test 312d006

@github-project-automation github-project-automation bot moved this from In Progress to In Review in CCCL Dec 16, 2025
@github-actions
Copy link
Contributor

🥳 CI Workflow Results

🟩 Finished in 51m 04s: Pass: 100%/43 | Total: 4h 10m | Max: 23m 22s | Hits: 96%/19689

See results here.

@ericniebler ericniebler marked this pull request as ready for review December 17, 2025 00:06
@ericniebler ericniebler requested a review from a team as a code owner December 17, 2025 00:06
@ericniebler ericniebler requested a review from miscco December 17, 2025 00:06
@ericniebler ericniebler merged commit 22fa667 into NVIDIA:main Dec 17, 2025
59 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Dec 17, 2025
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.

2 participants