From 0910ac07a2dc13036b0e6e36cd8ad753c2d2b487 Mon Sep 17 00:00:00 2001 From: Rick Altherr Date: Tue, 9 Jan 2018 12:27:14 -0800 Subject: [PATCH] Use CMake build types to set optimization level CMake has reasonable defaults for Debug and Release builds but the build type isn't specified by default. That led to me adding explicit flags. Remove those flags and default to using Release builds. Signed-off-by: Rick Altherr --- CMakeLists.txt | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 4a0bd664..13645ab1 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -3,18 +3,20 @@ cmake_minimum_required(VERSION 3.5.0) project(prjxray) option(PRJXRAY_BUILD_TESTING "" OFF) +if(NOT CMAKE_BUILD_TYPE) + set(CMAKE_BUILD_TYPE Release CACHE STRING + "Choose the type of build, options are: None Debug Release RelWithDebInfo MinSizeRel." + FORCE) +endif() + set(CMAKE_CXX_STANDARD 14) add_compile_options(-Wall -Werror) # Hack for missing option in cctz option(BUILD_TESTING "" OFF) - if(PRJXRAY_BUILD_TESTING) enable_testing() - add_compile_options(-O0 -g) -else() - add_compile_options(-O3) endif() add_subdirectory(third_party/googletest EXCLUDE_FROM_ALL)