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: > 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? Added & committed. Thank you for the review! Repository: rL LLVM https://reviews.llvm.org/D45766 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits