Skip to content

Conversation

@bernhardmgruber
Copy link
Contributor

@bernhardmgruber bernhardmgruber commented Nov 25, 2025

And use it in the scan tunings to test it.

  • No SASS diff for cub.test.device.scan.lid_0.types_0 on sm120

@bernhardmgruber bernhardmgruber requested a review from a team as a code owner November 25, 2025 09:33
@github-project-automation github-project-automation bot moved this to Todo in CCCL Nov 25, 2025
@cccl-authenticator-app cccl-authenticator-app bot moved this from Todo to In Review in CCCL Nov 25, 2025
And use it in the scan tunings to test it.
@github-actions

This comment has been minimized.


template <class ValueT, class AccumT, class OffsetT>
struct sm75_tuning<ValueT, AccumT, OffsetT, op_type::plus, primitive_accum::yes, offset_size::_8, value_size::_4>
struct sm75_tuning<ValueT, AccumT, OffsetT, op_kind_t::plus, primitive_accum::yes, offset_size::_8, value_size::_4>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question. Most of the code is not in the detail namespace. Is this something that we can change without affecting users?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes entities are in the namespace cub::detail::scan. We should be fine.

@github-actions
Copy link
Contributor

🥳 CI Workflow Results

🟩 Finished in 8h 43m: Pass: 100%/93 | Total: 5d 01h | Max: 4h 55m | Hits: 39%/95086

See results here.

@bernhardmgruber bernhardmgruber merged commit cd09270 into NVIDIA:main Nov 26, 2025
206 of 212 checks passed
@github-project-automation github-project-automation bot moved this from In Review to Done in CCCL Nov 26, 2025
@bernhardmgruber bernhardmgruber deleted the type_enum branch November 26, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants