diff options
| author | David Blaikie <dblaikie@gmail.com> | 2015-09-30 23:36:12 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2015-09-30 23:36:12 +0000 |
| commit | 3830d68bc19890df83961799c644de96346b6459 (patch) | |
| tree | 374bfd6b78514dd4276bb6f56a4be4fd2753707e /llvm/cmake/modules/HandleLLVMOptions.cmake | |
| parent | 25c58d124b6205962a4687c814ec77f20d91ce9e (diff) | |
| download | bcm5719-llvm-3830d68bc19890df83961799c644de96346b6459.tar.gz bcm5719-llvm-3830d68bc19890df83961799c644de96346b6459.zip | |
Enable -Wdeprecated in the cmake build now that LLVM (& Clang, Polly, and LLD) are -Wdeprecated clean
This particularly helps enforce the C++ Rule of 5 (for new move ops this
is already an error, but for a type only using C++98 features (copy
ctor/assign, dtor) it is only deprecated, not invalid)
Applying the flag for any GCC compatible compiler - GCC doesn't warn on
the Rule of 5 cases that C++11 deprecates, but it doesn't have other
false positives so far as I could see (compiling with GCC 4.8 didn't
produce any -Wdeprecated warnings I could spot).
Reviewers: aaron.ballman
Differential Revision: http://reviews.llvm.org/D13314
llvm-svn: 248963
Diffstat (limited to 'llvm/cmake/modules/HandleLLVMOptions.cmake')
| -rw-r--r-- | llvm/cmake/modules/HandleLLVMOptions.cmake | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/cmake/modules/HandleLLVMOptions.cmake b/llvm/cmake/modules/HandleLLVMOptions.cmake index e0bea6470fb..046187aa59d 100644 --- a/llvm/cmake/modules/HandleLLVMOptions.cmake +++ b/llvm/cmake/modules/HandleLLVMOptions.cmake @@ -364,7 +364,7 @@ if( MSVC ) elseif( LLVM_COMPILER_IS_GCC_COMPATIBLE ) if (LLVM_ENABLE_WARNINGS) - append("-Wall -W -Wno-unused-parameter -Wwrite-strings" CMAKE_C_FLAGS CMAKE_CXX_FLAGS) + append("-Wall -W -Wno-unused-parameter -Wwrite-strings -Wdeprecated" CMAKE_C_FLAGS CMAKE_CXX_FLAGS) append("-Wcast-qual" CMAKE_CXX_FLAGS) # Turn off missing field initializer warnings for gcc to avoid noise from |
