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