dblaikie accepted this revision.
dblaikie added inline comments.
This revision is now accepted and ready to land.


================
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()
----------------
lebedev.ri wrote:
> 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..
ah, misread those - figured they were testing the negative case (given the name 
of this patch) but I see they're testing the positive case.

Maybe testing the negative case would be useful too?


Repository:
  rC Clang

https://reviews.llvm.org/D45766



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

Reply via email to