aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRomain Forlot <romain.forlot@iot.bzh>2017-07-25 12:31:49 +0200
committerRomain Forlot <romain.forlot@iot.bzh>2017-07-25 12:31:49 +0200
commit5989f68a47567569a957b81ba64cbd8fef0bccc5 (patch)
tree2418a3688b68c9ef2322b7350d172afba72d7205
parentdb6a07f636a3f2a381dfcc0f52b16f59127496f0 (diff)
Change compile flag and clearer to read
Use -O0 with DEBUG instead of -0g Change-Id: I100e2188b34f3506d400379808ce875a4817be8d Signed-off-by: Romain Forlot <romain.forlot@iot.bzh>
-rw-r--r--cmake/cmake.d/04-build_options.cmake17
1 files changed, 9 insertions, 8 deletions
diff --git a/cmake/cmake.d/04-build_options.cmake b/cmake/cmake.d/04-build_options.cmake
index d1b25f4..899505f 100644
--- a/cmake/cmake.d/04-build_options.cmake
+++ b/cmake/cmake.d/04-build_options.cmake
@@ -75,18 +75,19 @@ add_compile_options(-fPIC)
# TODO: make more visible readable compile (macro ? split ?)
# Compilation option depending on CMAKE_BUILD_TYPE
##################################################
-add_compile_options($<$<OR:$<CONFIG:DEBUG>,$<CONFIG:PROFILING>,$<CONFIG:CCOV>>:-g>)
-
-add_compile_options($<$<CONFIG:DEBUG>:-ggdb>)
-add_compile_options($<$<CONFIG:CCOV>:--coverage>)
-
-add_compile_options($<$<CONFIG:DEBUG>:-Og>)
+add_compile_options($<$<CONFIG:PROFILING>:-g>)
add_compile_options($<$<CONFIG:PROFILING>:-O0>)
add_compile_options($<$<CONFIG:PROFILING>:-pg>)
-add_compile_options($<$<OR:$<CONFIG:CCOV>,$<CONFIG:PROFILING>>:-O2>)
+add_compile_options($<$<CONFIG:PROFILING>:-Wp,-U_FORTIFY_SOURCE>)
-add_compile_options($<$<OR:$<CONFIG:DEBUG>,$<CONFIG:PROFILING>>:-Wp,-U_FORTIFY_SOURCE>)
+add_compile_options($<$<CONFIG:DEBUG>:-g>)
+add_compile_options($<$<CONFIG:DEBUG>:-O0>)
+add_compile_options($<$<CONFIG:DEBUG>:-ggdb>)
+add_compile_options($<$<CONFIG:DEBUG>:-Wp,-U_FORTIFY_SOURCE>)
+add_compile_options($<$<CONFIG:CCOV>:-g>)
+add_compile_options($<$<CONFIG:CCOV>:-O2>)
+add_compile_options($<$<CONFIG:CCOV>:--coverage>)
# Env variable overload default
if(DEFINED ENV{INSTALL_PREFIX})