[Lldb-commits] [PATCH] D122974: prevent ConstString from calling djbHash() more than once

Luboš Luňák via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 29 15:57:20 PDT 2022

llunak added a comment.

In D122974#3483203 <https://reviews.llvm.org/D122974#3483203>, @clayborg wrote:

> If the string pool caches the hash value, we could actually write out the hash in the cache file to speed up loading.

The patch doing that is D124704 <https://reviews.llvm.org/D124704> (except for the string pool caching).

> I didn't want to re-hash every string when I saved the strings out to the cache file due to the cost, but it would be interesting to look into to see if it improves performance.

I can measure 10% startup time saved when everything is already cached. Funnily enough, profiler consistently claims that saving caches got faster too (I already use D122975 <https://reviews.llvm.org/D122975>).

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list