diff options
author | Alexander Potapenko <glider@google.com> | 2014-10-03 09:02:53 +0000 |
---|---|---|
committer | Alexander Potapenko <glider@google.com> | 2014-10-03 09:02:53 +0000 |
commit | e2e8b0e009a9905b523e1d8bc30c1488fae2f6d7 (patch) | |
tree | 2dc6d01171f9646ef044065c9ba0a03a98318616 /clang/test/SemaCXX/attr-gnu.cpp | |
parent | 0b9109c8036c1bfdd91321ae8a8dba3ee41ea7d6 (diff) | |
download | bcm5719-llvm-e2e8b0e009a9905b523e1d8bc30c1488fae2f6d7.tar.gz bcm5719-llvm-e2e8b0e009a9905b523e1d8bc30c1488fae2f6d7.zip |
Revert r218925 - "Patch to warn if 'override' is missing"
This CL has caused bootstrap failures on Linux and OSX buildbots running with -Werror.
Example report from http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/13183/steps/bootstrap%20clang/logs/stdio:
================================================================
[ 91%] Building CXX object tools/clang/tools/diagtool/CMakeFiles/diagtool.dir/ShowEnabledWarnings.cpp.o
In file included from /home/dtoolsbot/build/sanitizer-x86_64-linux/build/llvm/lib/Target/R600/AMDGPUISelDAGToDAG.cpp:20:
In file included from /home/dtoolsbot/build/sanitizer-x86_64-linux/build/llvm/lib/Target/R600/SIISelLowering.h:19:
/home/dtoolsbot/build/sanitizer-x86_64-linux/build/llvm/lib/Target/R600/SIInstrInfo.h:71:8: error: 'getLdStBaseRegImmOfs' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
bool getLdStBaseRegImmOfs(MachineInstr *LdSt,
^
/home/dtoolsbot/build/sanitizer-x86_64-linux/build/llvm/include/llvm/Target/TargetInstrInfo.h:815:16: note: overridden virtual function is here
virtual bool getLdStBaseRegImmOfs(MachineInstr *LdSt,
^
================================================================
llvm-svn: 218969
Diffstat (limited to 'clang/test/SemaCXX/attr-gnu.cpp')
-rw-r--r-- | clang/test/SemaCXX/attr-gnu.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/clang/test/SemaCXX/attr-gnu.cpp b/clang/test/SemaCXX/attr-gnu.cpp index fcc3e46db82..b4e9f4609f6 100644 --- a/clang/test/SemaCXX/attr-gnu.cpp +++ b/clang/test/SemaCXX/attr-gnu.cpp @@ -15,15 +15,14 @@ void g(int a[static [[]] 5]); // expected-error {{static array size is a C99 fea namespace { class B { public: - virtual void test() {} // expected-note {{overridden virtual function is here}} + virtual void test() {} virtual void test2() {} virtual void test3() {} }; class D : public B { public: - void test() __attribute__((deprecated)) final {} // expected-warning {{GCC does not allow an attribute in this position on a function declaration}} \ - // expected-warning {{'test' overrides a member function but is not marked 'override'}} + void test() __attribute__((deprecated)) final {} // expected-warning {{GCC does not allow an attribute in this position on a function declaration}} void test2() [[]] override {} // Ok void test3() __attribute__((cf_unknown_transfer)) override {} // Ok, not known to GCC. }; |