NVIDIA / NVIDIA/cccl

[FEA]: Use P2322R6 to determine accumulator type in Thrust

Open
#153 1 comment 1 reaction 1 assignee Claimed by @fbusato View on GitHub
thrust
Dominant language
C++
Stars
2.5k
Forks
487
Avg merge
2d 7h
Merged PRs (30d)
296

Description

### Is this a duplicate?

- [X] I confirmed there appear to be no [duplicate issues](https://github.com/NVIDIA/cccl/issues) for this request and that I agree to the [Code of Conduct](CODE_OF_CONDUCT.md)

### Area

Thrust

### Is your feature request related to a problem? Please describe.

We adopted P2322R6 scheme in CUB (https://github.com/NVIDIA/cub/issues/428), but Thrust still uses accumulator types that are surprising to users and do not follow general C++ rules like integer promotion.

### Describe the solution you'd like

We should decide if we'd like to proceed with P2322R6 adoption in Thrust and follow CUB scheme of selecting accumulator types.

### Tasks
- [x] Decide if the silent change of accumulator type is breaking
- [ ] Prepare a list of standard algorithms that specify accumulator types
- [ ] Consider preparing a paper to adopt P2322R6 to standard algorithms

CC: @jrhemstad @dkolsen-pgi

### Describe alternatives you've considered

_No response_

### Additional context

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.