organicmaps/organicmaps-tmp
Archived
13
Fork 1
Code Issues 2.7k Pull requests 182 Projects Releases 65 Wiki Activity Actions

Don't add unsupported flags on MSVC

Browse source 
Drive-by: use add_compile_options instead of add_definitions to set -fno-omit-frame-pointer
Signed-off-by: Osyotr <Osyotr@users.noreply.github.com>
This commit is contained in:
Osyotr 2024-03-18 01:49:06 +03:00
parent ae04ac3d41
commit d825eb4695
1 changed files with 14 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
CMakeLists.txt
View file

@ -103,7 +103,13 @@ if (NOT CMAKE_BUILD_TYPE)
endif()
# Global compile options for all configurations.
add_compile_options(-ffast-math)
if (MSVC)
add_compile_options(/utf-8)
add_link_options(/INCREMENTAL:NO)
add_link_options(/DEBUG:FULL)
else()
add_compile_options(-ffast-math)
endif()
if (PLATFORM_WIN)
add_definitions(
@ -114,10 +120,15 @@ endif()
# Built-in CMake configurations: Debug, Release, RelWithDebInfo, MinSizeRel
if (${CMAKE_BUILD_TYPE} STREQUAL "Debug")
add_definitions(-DDEBUG -fno-omit-frame-pointer)
add_definitions(-DDEBUG)
if (NOT MSVC)
add_compile_options(-fno-omit-frame-pointer)
endif()
elseif (${CMAKE_BUILD_TYPE} MATCHES "Rel")
add_definitions(-DRELEASE)
add_compile_options(-Ofast) # Also enables -ffast-math
if (NOT MSVC)
add_compile_options(-Ofast) # Also enables -ffast-math
endif()
else()
message(FATAL_ERROR "Unknown build type: " ${CMAKE_BUILD_TYPE})
endif()