Using murmurhash2 instead of fnv

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 10:31:11 PDT 2016


This is very interesting. I'm fine with this change, but I'm wondering if
you want to add that function to LLVM rather than LLD.

On Fri, Sep 9, 2016 at 7:03 AM, Rafael EspĂ­ndola <rafael.espindola at gmail.com
> wrote:

> One annoying property of fnv is that it does one multiplication per
> byte. Murmurhash2 does 3 per every 8 bytes.
>
> With the attached hack linking libxul.so goes from 7.496263743s to
> 6.661119354.
>
> If there is no objections I will write a proper patch for it.
>
> Cheers,
> Rafael
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160909/09711fab/attachment.html>


More information about the llvm-commits mailing list