[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