5chmidti added a comment.

Question: Should this check be extended to also include C.86 
<https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#c86-make--symmetric-with-respect-to-operand-types-and-noexcept>:
 "Flag an operator==() for which the argument types differ; same for other 
comparison operators: !=, <, <=, >, and >=."? With a warning along the lines 
of: "comparison operators should be symmetric with respect to the parameters 
types" or "parameters of comparison operators should be of the same type" (or 
some other warning). Or should that be a new check? I think it fits.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128861/new/

https://reviews.llvm.org/D128861

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to