[llvm] [ADT] Update hash function of uint64_t for DenseMap (PR #95734)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 17 14:45:52 PDT 2024
alexfh wrote:
> However, it would be good to have data on how this actually affects performance on real-world programs.
The patch was prompted by a recent ~5x regression in compilation speed, the corresponding profiles are quoted here: https://github.com/llvm/llvm-project/pull/92083#issuecomment-2165381579
@ilya-biryukov found that changing the hash function effectively resolves the regression (which I would expect to more or less remove the DenseMap code from the hot code. But if necessary, I can profile clang on the same compilation with and without this PR.
https://github.com/llvm/llvm-project/pull/95734
More information about the llvm-commits
mailing list