[PATCH] D24458: Using murmurhash2 instead of fnv

Ed Maste via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 10:17:41 PDT 2016

Previous message: [PATCH] D24458: Using murmurhash2 instead of fnv
Next message: [PATCH] D24458: Using murmurhash2 instead of fnv
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

emaste accepted this revision.
emaste added a comment.
This revision is now accepted and ready to land.

That said, I believe this is as good or better than fnv1 in all aspects (other than being slightly more complex) so no reason to avoid making this change now and then investigating xxhash or siphash.


https://reviews.llvm.org/D24458

Previous message: [PATCH] D24458: Using murmurhash2 instead of fnv
Next message: [PATCH] D24458: Using murmurhash2 instead of fnv
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list