From 0ba24635579f1825c1617a96dd0ffa987d2815d0 Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Wed, 2 Mar 2022 16:05:04 +0100 Subject: [PATCH] Use a single definition for coverage build flags. --- tools/cmake/Flags.cmake | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/tools/cmake/Flags.cmake b/tools/cmake/Flags.cmake index 5e2d24a11b..1d558cc407 100644 --- a/tools/cmake/Flags.cmake +++ b/tools/cmake/Flags.cmake @@ -221,13 +221,12 @@ if(enable_coverage) find_program(GCOV_PATH NAMES ENV{GCOV} gcov) if(GCOV_PATH) set(COVERAGE_COMMAND "${GCOV_PATH}" CACHE FILEPATH "Coverage testing tool (gcov)" FORCE) - set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -DCOVERAGE") - set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fprofile-arcs -ftest-coverage") - set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fprofile-arcs -ftest-coverage") - set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fprofile-arcs -ftest-coverage") - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DCOVERAGE") - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fprofile-arcs -ftest-coverage") - add_definitions(-fprofile-arcs -ftest-coverage) + set(COVERAGE_BUILD_FLAGS "-fprofile-arcs -ftest-coverage") + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -DCOVERAGE ${COVERAGE_BUILD_FLAGS}") + set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} ${COVERAGE_BUILD_FLAGS}") + set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} ${COVERAGE_BUILD_FLAGS}") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -DCOVERAGE ${COVERAGE_BUILD_FLAGS}") + add_definitions(${COVERAGE_BUILD_FLAGS}) endif() endif() -- 2.20.1