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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 16 09:46:19 PDT 2018


lebedev.ri added a comment.

In https://reviews.llvm.org/D45766#1101533, @ksu.shadura wrote:

> Thanks for your opinion, your guess was right! It is a unit-test, so seems we need to try to suppress warnings on our side.


Ok, great that this time it got resolved this trivially!

> In https://reviews.llvm.org/D45766#1098823, @Quuxplusone wrote:
> 
>> (That said, I continue to think that this diagnostic produces more noise than signal, and wish it weren't in `-Wall`...)

Too bad there is no way to track how many true-positives it produces.
Anecdotal evidence: it just prevented me from wasting time trying to understand what is going on during https://reviews.llvm.org/D46602 development, so sure, it can have false-positive noise, but..


Repository:
  rL LLVM

https://reviews.llvm.org/D45766





More information about the llvm-commits mailing list