[PATCH] D45766: [Sema] Add -Wno-self-assign-overloaded

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added inline comments.


================
Comment at: lib/Sema/SemaExpr.cpp:11527-11528
 
-  S.Diag(OpLoc, diag::warn_self_assignment)
-      << LHSDeclRef->getType()
-      << LHSExpr->getSourceRange() << RHSExpr->getSourceRange();
+  S.Diag(OpLoc, IsBuiltin ? diag::warn_self_assignment_builtin
+                          : diag::warn_self_assignment_overloaded)
+      << LHSDeclRef->getType() << LHSExpr->getSourceRange()
----------------
dblaikie wrote:
> Presumably this also changes how the warning is enabled? But that doesn't seem to be tested in this patch?
What testing do you have in mind?
The `test/SemaCXX/warn-self-assign-overloaded.cpp` change was supposed to show how it is enabled..


Repository:
  rC Clang

https://reviews.llvm.org/D45766