[PATCH] D25645: [ADT] Add CachedHashString.

[Rafael Ávila de Espíndola via llvm-commits]

rafael added inline comments.


================
Comment at: llvm/include/llvm/ADT/CachedHashString.h:166
+
+  friend bool operator==(const CachedHashString &LHS, StringRef RHS) {
+    return LHS.val() == RHS;
----------------
It is common practice to declare == out of class.

Do you still need if you declare a StringRef() operator?


https://reviews.llvm.org/D25645