[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