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

Make scan and histogram shaders compatible #6

Closed
pbbastian opened this issue Feb 13, 2017 · 2 comments
Closed

Make scan and histogram shaders compatible #6

pbbastian opened this issue Feb 13, 2017 · 2 comments

Comments

@pbbastian
Copy link
Owner

pbbastian commented Feb 13, 2017

Options:

  1. Store histogram as AOS and make version of scan that operates on this structure.
  2. Store histogram as SOA and run a scan for each array using an offset per dispatch.

Option 1 has some advantages for smaller input sizes, as it can do all the scans at once. OTOH it increases register usage which is bad for performance. Larger input sizes should satisfy option 2 just fine, which could then be made to operate on multiple inputs from the same sequence, thus not increasing register usage as much.

@pbbastian pbbastian changed the title Make global scan and scan compute shaders work on 8 sequences at a time Make scan and histogram shaders compatible Feb 14, 2017
@pbbastian pbbastian mentioned this issue Feb 14, 2017
4 tasks
@pbbastian
Copy link
Owner Author

Option 2 is chosen.

  • Make scan shader use an offset and length
  • Make global scan shader use an offset and length
  • Make group add shader use an offset and length
  • Make histogram shader output a structure of arrays

@pbbastian
Copy link
Owner Author

Implemented and tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant