[Lldb-commits] [PATCH] D43596: Replace HashStringUsingDJB with llvm::djbHash
Jonas Devlieghere via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Feb 22 07:57:54 PST 2018
JDevlieghere added a comment.
In https://reviews.llvm.org/D43596#1015848, @clayborg wrote:
> No need to add additional dSYM tests, because all dSYM tests will fail if the hashing isn't correct.
Maybe I misunderstand, but https://reviews.llvm.org/D42740 suggested that this never worked (or maybe it did accidentally, because we ended up in the right bucket) for unicode characters?
More information about the lldb-commits