diff options
author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-02-03 18:01:15 +0000 |
---|---|---|
committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2011-02-03 18:01:15 +0000 |
commit | 7272d9cf36cdcbc13f35ce8f37f92263ee91fe98 (patch) | |
tree | c16d4e5089323458ec61a7bfff5b4c24837e0de2 /clang/lib/Sema/SemaDecl.cpp | |
parent | 25c98707da7ccb1c976f6eab03d1cfd024258114 (diff) | |
download | bcm5719-llvm-7272d9cf36cdcbc13f35ce8f37f92263ee91fe98.tar.gz bcm5719-llvm-7272d9cf36cdcbc13f35ce8f37f92263ee91fe98.zip |
Implement -Woverloaded-virtual.
The difference with gcc is that it warns if you overload virtual methods only if
the method doesn't also override any method. This is to cut down on the number of warnings
and make it more useful like reported here: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=20423.
If we want to warn that not all overloads are overriden we can have an additional
warning like -Wpartial-override.
-Woverloaded-virtual, unlike gcc, is added to -Wmost. Addresses rdar://8757630.
llvm-svn: 124805
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 6341de996ad..9c4fdc99f27 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -4276,7 +4276,7 @@ void Sema::CheckFunctionDeclaration(Scope *S, FunctionDecl *NewFD, diag::note_overridden_virtual_function); } } - } + } } } |