thrust icon indicating copy to clipboard operation
thrust copied to clipboard

Remove redundant `reduce` implementation

Open alliepiper opened this issue 4 years ago • 0 comments

thrust/system/cuda/detail/reduce.h contains a complete reduction implementation in the __reduce namespace. In 8702bfe89640c9efcf68cf3ad2b89d4d00e96494, the thrust::cuda_cub::reduce entry points were changed to just call cub::DeviceReduce, but the old implementation remains.

The __reduce implementation is still used to implement the algorithms in extrema.h (minmax_element, etc). These should be updated to use cub::DeviceReduce and the __reduce implementation should be removed.

alliepiper avatar May 07 '21 22:05 alliepiper