[CUB] Refactor DeviceAdjacentDifference::SubtractLeft to always take an environment#9418
[CUB] Refactor DeviceAdjacentDifference::SubtractLeft to always take an environment#9418miscco wants to merge 1 commit into
DeviceAdjacentDifference::SubtractLeft to always take an environment#9418Conversation
|
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. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughDeviceAdjacentDifference SubtractLeft and SubtractLeftCopy overloads switch stream parameters to const EnvT& env, route execution through dispatch_with_env_and_tuning with a policy selector, update convenience-enable_if to require indirectly-binary-invocable, and add iterator concepts plus Catch2 tests covering multiple environment types and temp-storage usage. ChangesEnvironment-based dispatch for SubtractLeft APIs*
Possibly related PRs
Suggested reviewers
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Enterprise
Run ID: 0203dc71-7802-4f71-a286-b6a226094643
📒 Files selected for processing (2)
cub/cub/device/device_adjacent_difference.cuhcub/test/catch2_test_device_adjacent_difference_substract_left.cu
3c91b42 to
f351e63
Compare
3c91b42 to
9004022
Compare
This comment has been minimized.
This comment has been minimized.
9004022 to
30ac019
Compare
This comment has been minimized.
This comment has been minimized.
30ac019 to
ebfba0b
Compare
This comment has been minimized.
This comment has been minimized.
32c5fb1 to
7e4f535
Compare
This comment has been minimized.
This comment has been minimized.
7e4f535 to
9fdb70a
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
9fdb70a to
287fe50
Compare
287fe50 to
ec65ce0
Compare
This comment has been minimized.
This comment has been minimized.
ec65ce0 to
c77736c
Compare
This comment has been minimized.
This comment has been minimized.
c77736c to
ecb8b70
Compare
…e an environment We want to be able to pass tunings to the APIs that take user provided memory.
ecb8b70 to
478203c
Compare
🥳 CI Workflow Results🟩 Finished in 2h 55m: Pass: 100%/287 | Total: 2d 20h | Max: 1h 15m | Hits: 99%/198283See results here. |
Description
closes
Checklist