[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