Introduce FixedUnorderedMap/FixedUnorderedSet #87
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bobobalink
force-pushed
the
fixed_unordered_map
branch
from
940a115 to
69f7edb
Compare
Bobobalink
force-pushed
the
fixed_unordered_map
branch
3 times, most recently
from
9daa13d to
49cadff
Compare
Bobobalink
force-pushed
the
fixed_unordered_map
branch
from
49cadff to
d0625a7
Compare
alexkaratarakis
force-pushed
the
fixed_unordered_map
branch
from
2af7a89 to
d021a81
Compare
alexkaratarakis
force-pushed
the
fixed_unordered_map
branch
3 times, most recently
from
afe2999 to
dc52292
Compare
alexkaratarakis
changed the title
alexkaratarakis
force-pushed
the
fixed_unordered_map
branch
from
dc52292 to
089d373
Compare
This is an implmentation of a hashtable based on Robin Hood hashing with backwards shift deletion. The code was initially based on Martin Ankerl's `UnorderedDense` and still uses an identical algorithm, but nearly all of it has been rewritten. This is intentionally a bare-bones API, with only the functions required to implement a full-fat Map API.
alexkaratarakis
force-pushed
the
fixed_unordered_map
branch
from
089d373 to
a2f7947
Compare
This uses `FixedRobinhoodHashtable` to implement a standards-compliant `UnorderedMap`. It does this by introducing `FixedMapAdapter`, which is meant to unify different map backends with a simple API. Future work will use this for the existing `FixedMap`.
alexkaratarakis
force-pushed
the
fixed_unordered_map
branch
from
a2f7947 to
095b1bc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
based on ankerl::UnorderedDense
This is based on Martin's dense unordered map using robinhood hashing and backwards shift deletion.
This does intensive surgery to make it a fixed-container, using
FixedDoublyLinkedListfor backing storage. Most of the interface code is removed, leaving only the minimal amount of logic required to implement a general map interface in the future.Additionally, white-box tests are added to inspect the operations on the internal data structures of the map.