[PATCH] D142862: [Support] change StringMap hash function from djbHash to xxHash

Erik Desjardins via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 07:14:22 PST 2023


erikdesjardins added a subscriber: nikic.
erikdesjardins added a comment.

In D142862#4089860 <https://reviews.llvm.org/D142862#4089860>, @serge-sans-paille wrote:

> Can  you take a shot against https://llvm-compile-time-tracker.com/ so that we get an hint of the practical speedup?

I don't think I have permission to use that. @nikic?

> Looks like xxhash.h is missing from the patch.

It already exists: https://github.com/llvm/llvm-project/blob/main/llvm/include/llvm/Support/xxhash.h


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142862/new/

https://reviews.llvm.org/D142862



More information about the llvm-commits mailing list