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

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 18 05:52:53 PDT 2018


lebedev.ri created this revision.
lebedev.ri added reviewers: dblaikie, aaron.ballman, thakis, rjmccall, rsmith.

It seems there isn't much enthusiasm for `-wtest` https://reviews.llvm.org/D45685.

This is more conservative version, which i had in the very first
revision of https://reviews.llvm.org/D44883, but that 'erroneously' got removed because of the review.

**Based on some [irc] discussions, it must really be documented that
we want all the new diagnostics to have their own flags, to ease
rollouts, transitions, etc.**

Please do note that i'm only adding `-Wno-self-assign-overloaded`,
but not `-Wno-self-assign-field-overloaded`, because i'm honestly
not aware of any false-positives from the `-field` variant,
but i can just as easily add it if wanted.
https://reviews.llvm.org/D44883#1068561


Repository:
  rC Clang

https://reviews.llvm.org/D45766

Files:
  docs/ReleaseNotes.rst
  include/clang/Basic/DiagnosticGroups.td
  include/clang/Basic/DiagnosticSemaKinds.td
  lib/Sema/SemaExpr.cpp
  test/SemaCXX/warn-self-assign-overloaded.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45766.142917.patch
Type: text/x-patch
Size: 5647 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180418/eafeda5e/attachment.bin>


More information about the cfe-commits mailing list