Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEA]: Add more benchmarks for thrust::transform #2814

Closed
bernhardmgruber opened this issue Nov 14, 2024 · 2 comments
Closed

[FEA]: Add more benchmarks for thrust::transform #2814

bernhardmgruber opened this issue Nov 14, 2024 · 2 comments
Assignees
Labels
thrust For all items related to Thrust.

Comments

@bernhardmgruber
Copy link
Contributor

bernhardmgruber commented Nov 14, 2024

We currently use the BabelStream kernels as memory bound workloads and a fibonacci kernel (per thread: read random index [0;42], compute fibonacci number, store result). The latter kernel is compute and shows high thread divergence.

In order to better assess regressions, we should add more benchmarks covering:

  • compute bound, but convergent threads
  • overwriting the input sequence 1:1
  • reading from overlapping regions (e.g. something like adjacent difference, or a stencil operation)
@bernhardmgruber bernhardmgruber added the feature request New feature or request. label Nov 14, 2024
@github-project-automation github-project-automation bot moved this to Todo in CCCL Nov 14, 2024
@bernhardmgruber bernhardmgruber self-assigned this Nov 14, 2024
@bernhardmgruber bernhardmgruber added thrust For all items related to Thrust. 2.8.0 target for 2.8.0 release labels Nov 14, 2024
@bernhardmgruber
Copy link
Contributor Author

I think we should actually make them CUB benchmarks, since they should be included in continuous CUB benchmarking and tuning.

bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this issue Nov 20, 2024
bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this issue Nov 20, 2024
bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this issue Nov 20, 2024
@bernhardmgruber
Copy link
Contributor Author

We can probably postpone this, because thrust::transform seems to be performing well.

@bernhardmgruber bernhardmgruber removed 2.8.0 target for 2.8.0 release feature request New feature or request. labels Nov 20, 2024
bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this issue Nov 23, 2024
bernhardmgruber added a commit to bernhardmgruber/cccl that referenced this issue Nov 25, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in CCCL Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
thrust For all items related to Thrust.
Projects
Status: Done
Development

No branches or pull requests

1 participant