diff options
| author | Dave Lee <davelee.com@gmail.com> | 2020-09-21 10:22:51 -0700 |
|---|---|---|
| committer | Dave Lee <davelee.com@gmail.com> | 2020-09-21 10:23:17 -0700 |
| commit | b36bdfe5ca0c2b863248f327b03d41516b38dc11 (patch) | |
| tree | 678b317b8fd8f1499cc1e8b82f8af3b8587a7e2e /llvm/runtimes | |
| parent | 3b3b9ba1c7d89afe4909a42e2a795354bb79e062 (diff) | |
| download | llvm-b36bdfe5ca0c2b863248f327b03d41516b38dc11.zip llvm-b36bdfe5ca0c2b863248f327b03d41516b38dc11.tar.gz llvm-b36bdfe5ca0c2b863248f327b03d41516b38dc11.tar.bz2 | |
[cmake] Centralize LLVM_ENABLE_WARNINGS option
Configure default value of `LLVM_ENABLE_WARNINGS` in `HandleLLVMOptions.cmake`.
`LLVM_ENABLE_WARNINGS` is documented as ON by default, but `HandleLLVMOptions` assumes the default has been set somewhere else. If it has not been explicitly set, then `HandleLLVMOptions` implicitly uses OFF as a default.
This removes the various `option()` declarations in favor of a single declaration in `HandleLLVMOptions`. This will prevent the unwanted use of `-w` that is mentioned in a couple of the comments.
Reviewed By: DavidTruby, #libunwind, JDevlieghere, compnerd
Differential Revision: https://reviews.llvm.org/D87243
Diffstat (limited to 'llvm/runtimes')
| -rw-r--r-- | llvm/runtimes/CMakeLists.txt | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/llvm/runtimes/CMakeLists.txt b/llvm/runtimes/CMakeLists.txt index 04a13e4..3a9046d 100644 --- a/llvm/runtimes/CMakeLists.txt +++ b/llvm/runtimes/CMakeLists.txt @@ -105,10 +105,6 @@ if(${CMAKE_SOURCE_DIR} STREQUAL ${CMAKE_CURRENT_SOURCE_DIR}) # Avoid checking whether the compiler is working. set(LLVM_COMPILER_CHECKED ON) - # Enable warnings, otherwise -w gets added to the cflags by HandleLLVMOptions - # resulting in unjustified successes by check_cxx_compiler_flag. - set(LLVM_ENABLE_WARNINGS ON) - # Handle common options used by all runtimes. include(AddLLVM) include(HandleLLVMOptions) |