[PATCH] D43615: [Support] Remove HashString and replace it with djbHash. NFC.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 22 08:22:13 PST 2018


JDevlieghere 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.


Pavel already landed that change, so we should be good!


Repository:
  rL LLVM

https://reviews.llvm.org/D43615





More information about the llvm-commits mailing list