[PATCH] D25645: [ADT] Add CachedHashString.

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 19 05:28:36 PDT 2016

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?


More information about the llvm-commits mailing list