[PATCH] D45766: [Sema] Add -Wno-self-assign-overloaded
John McCall via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 23 15:22:09 PDT 2018
rjmccall added a comment.
In https://reviews.llvm.org/D45766#1076090, @dblaikie wrote:
> FWIW I don't fundamentalyl object to also having something like -wtest.
> Probably needs a better name though (unfortunately the double-negative gets
> confusing... - like you want to describe the set of diagnostics that should
> not be used in test code, so that as a group might be "-Wnon-test" but then
> "-Wno-non-test" is pretty awkward) - probably worth chatting to Richard
> Smith about that, I reckon.
That's why I was suggesting putting it in the `-w` namespace. We really wouldn't expect or want users to ever use a *positive* version of this warning option — specifically asking for just the warnings that are known to be problematic for test code, across all warnings. It's just not really a warning group.
It could also be something like `-fsuppress-problematic-test-warnings`, of course, but I was basically thinking of `-w` as meaning `-fsuppress-problematic-*-warnings`.
Repository:
rL LLVM
https://reviews.llvm.org/D45766
More information about the llvm-commits
mailing list