From 4cee6f3d7951f0f07a204c72e3830807082873a8 Mon Sep 17 00:00:00 2001 From: Jonathan G Rennison Date: Sat, 27 Jun 2020 19:27:21 +0100 Subject: [PATCH] CMake: Remove disabling asserts for release builds --- cmake/CompileFlags.cmake | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/cmake/CompileFlags.cmake b/cmake/CompileFlags.cmake index f856fa907e..50a6757135 100644 --- a/cmake/CompileFlags.cmake +++ b/cmake/CompileFlags.cmake @@ -36,7 +36,7 @@ macro(compile_flags) # Prepare a generator that checks if we are not a debug, and don't have asserts # on. We need this later on to set some compile options for stable releases. - set(IS_STABLE_RELEASE "$>,$>>") + #set(IS_STABLE_RELEASE "$>,$>>") if (MSVC) add_compile_options(/W3) @@ -68,14 +68,14 @@ macro(compile_flags) -fno-strict-aliasing ) - add_compile_options( + #add_compile_options( # When we are a stable release (Release build + USE_ASSERTS not set), # assertations are off, which trigger a lot of warnings. We disable # these warnings for these releases. - "$<${IS_STABLE_RELEASE}:-Wno-unused-variable>" - "$<${IS_STABLE_RELEASE}:-Wno-unused-but-set-parameter>" - "$<${IS_STABLE_RELEASE}:-Wno-unused-but-set-variable>" - ) + #"$<${IS_STABLE_RELEASE}:-Wno-unused-variable>" + #"$<${IS_STABLE_RELEASE}:-Wno-unused-but-set-parameter>" + #"$<${IS_STABLE_RELEASE}:-Wno-unused-but-set-variable>" + #) if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU") include(CheckCXXCompilerFlag)