-
Notifications
You must be signed in to change notification settings - Fork 89
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
Add dynamic_bitset
#352
Add dynamic_bitset
#352
Conversation
Added simple size test
Possible by shifting from static to dynamic allocation
This was only used in one method and is replaced by size_type
Pull requests from external contributors require approval from a |
Can one of the admins verify this patch? Admins can comment |
rank, select -> rank_true, select_true rank0, select0 -> rank_false, select_false
abs -> base rels -> offsets
This will allow us to rename ranks() to rank()
ranks -> rank selects -> select
Query methods will check and perform build()
…to trie-bitvector
/ok to test |
/ok to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PR adds
dynamic_bitset
code that will be used in Trie data structure. Trie will be integrated in a separate PR #350.Since
dynamic_bitset
is not intended to be part of public-facing API, all files (.cuh and .inl) are located in include/cuco/detail/trie/dynamic_bitsetTests are added in tests/dynamic_bitset