[Lldb-commits] [PATCH] D43596: Replace HashStringUsingDJB with llvm::djbHash

[Jonas Devlieghere via Phabricator via lldb-commits]

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?


https://reviews.llvm.org/D43596