diff options
author | Jason Merrill <jason@redhat.com> | 2022-06-24 14:40:12 -0400 |
---|---|---|
committer | Jason Merrill <jason@redhat.com> | 2022-06-24 18:23:48 -0400 |
commit | 113844d68e94f4e9c0e946db351ba7d3d4a1335a (patch) | |
tree | 4700337540406594bea63da9bf9543c45207dd64 /gcc/c-family/c.opt | |
parent | 75fa80bb5654d1f8b21310118f41705b74168039 (diff) | |
download | gcc-113844d68e94f4e9c0e946db351ba7d3d4a1335a.zip gcc-113844d68e94f4e9c0e946db351ba7d3d4a1335a.tar.gz gcc-113844d68e94f4e9c0e946db351ba7d3d4a1335a.tar.bz2 |
c++: Include -Woverloaded-virtual in -Wall [PR87729]
This seems like a good warning to have in -Wall, as requested. But as
pointed out in PR20423, some users want a warning only when a derived
function doesn't override any base function. So let's put that lesser
version in -Wall (and -Woverloaded-virtual=1) while leaving the semantics
for the existing option the same.
PR c++/87729
PR c++/20423
gcc/c-family/ChangeLog:
* c.opt (Woverloaded-virtual): Add levels, include in -Wall.
gcc/ChangeLog:
* doc/invoke.texi: Document changes.
gcc/cp/ChangeLog:
* class.cc (warn_hidden): Handle -Woverloaded-virtual=1.
gcc/testsuite/ChangeLog:
* g++.dg/warn/Woverloaded-virt1.C: New test.
* g++.dg/warn/Woverloaded-virt2.C: New test.
Diffstat (limited to 'gcc/c-family/c.opt')
-rw-r--r-- | gcc/c-family/c.opt | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt index 41a20bc6..44e1a60 100644 --- a/gcc/c-family/c.opt +++ b/gcc/c-family/c.opt @@ -1126,7 +1126,11 @@ C ObjC C++ ObjC++ Var(warn_overlength_strings) Warning LangEnabledBy(C ObjC C++ Warn if a string is longer than the maximum portable length specified by the standard. Woverloaded-virtual -C++ ObjC++ Var(warn_overloaded_virtual) Warning +C++ ObjC++ Warning Alias(Woverloaded-virtual=,2,0) +Warn about overloaded virtual function names. + +Woverloaded-virtual= +C++ ObjC++ Joined UInteger IntegerRange(0,2) Var(warn_overloaded_virtual) Warning LangEnabledBy(C++ ObjC++,Wall,1,0) Warn about overloaded virtual function names. Woverride-init |