jlebar added a comment. > It's used in lld/ELF/SymbolTable.h . Dang, you are right. I was thrown off because the lines above use CachedHashString, which is different. This actually looks pretty reasonable, too. I guess that does it for this patch. https://reviews.llvm.org/D25630