Skip to content

Conversation

@davebayer
Copy link
Contributor

@davebayer davebayer commented Jan 14, 2026

We are having problems with [[no_unique_address]] causing data corruption. This PR tries to provide a solution to this problem by introducing __no_unique_member class wrapper that should be used as a base class with combination with _CCCL_DECLSPEC_EMPTY_BASES.

I tried to demonstrate how it should be used on some simple ranges.

@davebayer davebayer requested a review from a team as a code owner January 14, 2026 10:55
@davebayer davebayer requested a review from wmaxey January 14, 2026 10:55
@github-project-automation github-project-automation bot moved this to Todo in CCCL Jan 14, 2026
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Jan 14, 2026
@davebayer
Copy link
Contributor Author

pre-commit.ci autofix

@copy-pr-bot
Copy link
Contributor

copy-pr-bot bot commented Jan 14, 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.

@davebayer davebayer force-pushed the dont_use_no_unique_address branch from bf4a50a to ca84123 Compare January 14, 2026 11:07
@github-actions

This comment has been minimized.

@davebayer davebayer force-pushed the dont_use_no_unique_address branch from ca84123 to 335885f Compare January 14, 2026 13:29
@davebayer davebayer changed the title Implement __no_unique replacement for [[no_unique_address]] Implement __no_unique_member and use it in ranges Jan 14, 2026
@github-actions
Copy link
Contributor

😬 CI Workflow Results

🟥 Finished in 1h 38m: Pass: 47%/84 | Total: 1d 16h | Max: 1h 20m | Hits: 66%/56089

See results here.

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.

2 participants