[PATCH] D44883: [Sema] Extend -Wself-assign with -Wself-assign-overloaded to warn on overloaded self-assignment (classes)

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 26 15:08:03 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D44883#1048689, @rjmccall wrote:

> I tracked Chandler down, and he doesn't remember why user-defined operators are excluded.


Ok then. Unless reviewers think it would be best to have separate diag groups for "builtin" and "user-provided" binary operators,
i'll just extend the `-Wself-assign` / `-Wself-assign-field` / `-Wself-move` (i wonder, does it work both the fields and usual variables, or not).

(That means, stage2 self-hosting testing will be needed...)


Repository:
  rC Clang

https://reviews.llvm.org/D44883





More information about the cfe-commits mailing list