[LLVMdev] [PATCH] FoldingSetNodeID: use MurmurHash2 instead of SuperFastHash

Chandler Carruth chandlerc at google.com
Sat Feb 6 16:51:15 PST 2010

While I've not reviewed the patch in too much detail, it looks
promising. Can you run some end-to-end benchmarks to make sure that
cache pressure in the full program or other variables not accounted
for in a micro-benchmark don't dominate performance? Specifically the
nightly tester includes a number of real programs and machinery to
measure total compile time.

On Sat, Feb 6, 2010 at 7:09 AM, Gregory Petrosyan
<gregory.petrosyan at gmail.com> wrote:
> Some additional info can be found at:
>  http://murmurhash.googlepages.com/
>  http://en.wikipedia.org/wiki/MurmurHash
>  http://www.codeproject.com/KB/recipes/hash_functions.aspx
> as well as in the patch description itself. Patch and benchmark attached.
>        Gregory
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

More information about the llvm-dev mailing list