Skip to content

Update std::uniform_int_distribution#6799

Merged
RAMitchell merged 4 commits intoNVIDIA:mainfrom
RAMitchell:uniform_int_distribution
Dec 5, 2025
Merged

Update std::uniform_int_distribution#6799
RAMitchell merged 4 commits intoNVIDIA:mainfrom
RAMitchell:uniform_int_distribution

Conversation

@RAMitchell
Copy link
Contributor

Notes:

  • This distribution is now constexpr
  • Adding a stream operator means this distribution no longer supports 128 bit integers - no stream operator is defined for this type. I think its better that we support stream operators consistently than keep 128 bit int. If we implement a cuda::std::uint128_t with defined stream operators then all we need to do is add this type to the templated test function.

@RAMitchell RAMitchell requested a review from a team as a code owner November 27, 2025 10:17
@RAMitchell RAMitchell requested a review from griwes November 27, 2025 10:17
@github-project-automation github-project-automation bot moved this to Todo in CCCL Nov 27, 2025
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Nov 27, 2025
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

🥳 CI Workflow Results

🟩 Finished in 2h 37m: Pass: 100%/91 | Total: 2d 07h | Max: 2h 02m | Hits: 85%/199104

See results here.

@RAMitchell RAMitchell merged commit fdc6d86 into NVIDIA:main Dec 5, 2025
102 of 103 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Dec 5, 2025
srinivasyadav18 pushed a commit to srinivasyadav18/cccl that referenced this pull request Dec 9, 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.

3 participants