[PATCH] D43615: [Support] Remove HashString and replace it with djbHash. NFC.
Pavel Labath via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 22 09:30:10 PST 2018
labath added a comment.
In https://reviews.llvm.org/D43615#1015902, @aprantl wrote:
> Also please be sure to not land this before we fixed the implementation of `llvm::djbHash()` to use unsigned chars or else this isn't NFC.
The HashString function used a different initial value (0 vs 5381), so this is still not fully NFC, but I hope that does not matter.
Repository:
rL LLVM
https://reviews.llvm.org/D43615
More information about the llvm-commits
mailing list