[PATCH] D77038: Fix bug in BitVector and SmallBitVector DenseMap hashing
Alexandre Ganea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 17 19:08:32 PDT 2020
aganea accepted this revision.
aganea added a comment.
This revision is now accepted and ready to land.
In my sense a hash is a function of the represented data, not of the implementation. I feel that they should both resolve to the same hash, but at the same time I don't have practical example for this. Let's leave this open, until there's a practical application.
Thanks again! LTGM.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D77038/new/
https://reviews.llvm.org/D77038
More information about the llvm-commits
mailing list