[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