================
@@ -51,6 +53,14 @@ class StringTable {
     constexpr Offset() = default;
     constexpr Offset(unsigned Value) : Value(Value) {}
 
+    constexpr bool operator==(const Offset &RHS) const {
----------------
dwblaikie wrote:

Minor convention, but prefer non-member (can still be defined inline as a 
friend, so not a huge syntactic overhead) op overloads for equal conversion 
(especially since this type is implicitly convertible) on LHS and RHS, perhaps?

https://github.com/llvm/llvm-project/pull/123302
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to