[PATCH] D133715: [ADT] Add HashMappedTrie
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 6 17:51:11 PST 2023
dexonsmith added a comment.
In D133715#4032713 <https://reviews.llvm.org/D133715#4032713>, @steven_wu wrote:
> Maybe `RawHashTrieMap`? It reads better when `Raw` is in the front, and it contains hash-trie and trie-map, which are both terms describing data structures similar to this but this is much simpler, thus raw.
I think from a client perspective, this is some sort of variant of a HashMap, so ending with that makes sense to me. It’s an implementation detail that it’s a trie so that seems better in the prefix.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133715/new/
https://reviews.llvm.org/D133715
More information about the llvm-commits
mailing list