foster/bitcoin-bitcoin-core
0
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-02-22 12:23:34 -05:00
Code Issues Activity

build: set build type and per-build-type flags as early as possible

Browse source 
With the exception of the first c++ checks, this ensures that compiler tests
are never run with the wrong build type's flags.

Co-Authored-By: Hennadii Stepanov <32963518+hebasto@users.noreply.github.com>
This commit is contained in:
Cory Fields 2025-01-21 22:11:26 +00:00
parent f605f7a9c2
commit 56a9b847bb
2 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
CMakeLists.txt
View file

@ -70,6 +70,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF) set(CMAKE_CXX_EXTENSIONS OFF)
list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake/module) list(APPEND CMAKE_MODULE_PATH ${PROJECT_SOURCE_DIR}/cmake/module)
include(ProcessConfigurations)
#============================= #=============================
# Configurable options # Configurable options
@ -230,8 +231,6 @@ if(BUILD_FOR_FUZZING)
) )
endif() endif()
include(ProcessConfigurations)
include(TryAppendCXXFlags) include(TryAppendCXXFlags)
include(TryAppendLinkerFlag) include(TryAppendLinkerFlag)

4
cmake/module/ProcessConfigurations.cmake
View file

@ -4,8 +4,6 @@
include_guard(GLOBAL) include_guard(GLOBAL)
include(TryAppendCXXFlags)
macro(normalize_string string) macro(normalize_string string)
string(REGEX REPLACE " +" " " ${string} "${${string}}") string(REGEX REPLACE " +" " " ${string} "${${string}}")
string(STRIP "${${string}}" ${string}) string(STRIP "${${string}}" ${string})
@ -119,6 +117,8 @@ endfunction()
set_default_config(RelWithDebInfo) set_default_config(RelWithDebInfo)
include(TryAppendCXXFlags)
# We leave assertions on. # We leave assertions on.
if(MSVC) if(MSVC)
remove_cxx_flag_from_all_configs(/DNDEBUG) remove_cxx_flag_from_all_configs(/DNDEBUG)