Skip to content

Fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB#7882

Open
andrewcorrigan wants to merge 1 commit intoNVIDIA:mainfrom
andrewcorrigan:main
Open

Fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB#7882
andrewcorrigan wants to merge 1 commit intoNVIDIA:mainfrom
andrewcorrigan:main

Conversation

@andrewcorrigan
Copy link
Contributor

@andrewcorrigan andrewcorrigan commented Mar 4, 2026

Fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB

Fixes #7855

  • implements is_compatible to verify that the tuple structure matches to fix conversions
  • TestZipIteratorDereferenceToValue verifies that casting from the result of dereferencing to the corresponding value type works
  • TestZipIteratorNestedCopy verifies that copying using nested zip_iterators works

Checklist

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

@andrewcorrigan andrewcorrigan requested a review from a team as a code owner March 4, 2026 16:11
@github-project-automation github-project-automation bot moved this to Todo in CCCL Mar 4, 2026
@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Mar 4, 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.

@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Mar 4, 2026
@andrewcorrigan andrewcorrigan marked this pull request as draft March 4, 2026 16:40
@andrewcorrigan
Copy link
Contributor Author

I found one more case to fix, and will add a test case for that.

@cccl-authenticator-app cccl-authenticator-app bot moved this from In Review to In Progress in CCCL Mar 4, 2026
@andrewcorrigan andrewcorrigan changed the title fixes dereferencing zip_iterator over tuples of length-1 fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB Mar 4, 2026
@andrewcorrigan andrewcorrigan changed the title fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB Fixes dereferencing zip_iterator over tuples of length-1 as well as copies from nested zip_iterators using OpenMP/TBB Mar 4, 2026
@andrewcorrigan
Copy link
Contributor Author

I found one more case to fix, and will add a test case for that.

Done

@andrewcorrigan andrewcorrigan marked this pull request as ready for review March 4, 2026 17:11
@cccl-authenticator-app cccl-authenticator-app bot moved this from In Progress to In Review in CCCL Mar 4, 2026
@andrewcorrigan andrewcorrigan force-pushed the main branch 2 times, most recently from 0c648ef to bdb335d Compare March 4, 2026 17:16
…opies from nested zip_iterators using OpenMP/TBB

- implements is_compatible to verify that the tuple structure matches to fix conversions
- TestZipIteratorDereferenceToValue verifies that casting from the result of dereferencing to the corresponding value type works
- TestZipIteratorNestedCopy verifies that copying using nested zip_iterators works

Fixes NVIDIA#7855
@andrewcorrigan
Copy link
Contributor Author

Ran clang-format and updated

@miscco
Copy link
Contributor

miscco commented Mar 4, 2026

Will look at this on friday

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

[BUG]: Dereferencing a zip_iterator and converting to its value type leads to a compilation error

2 participants