CMakeLists: Move compilation flags into the src directory
We generally shouldn't be hijacking CMAKE_CXX_FLAGS, etc as a means to append flags to the targets, since this adds the compilation flags to everything, including our externals, which can result in weird issues and makes the build hierarchy fragile. Instead, we want to just apply these compilation flags to our targets, and let those managing external libraries to properly specify their compilation flags. This also results in us not getting as many warnings, as we don't raise the warning level on every external target.
This commit is contained in:
Lioncash
parent
41566c615b
commit
13bc74e957
2 changed files with 78 additions and 79 deletions
|
|
@ -104,78 +104,12 @@ endif()
|
||||||
message(STATUS "Target architecture: ${ARCHITECTURE}")
|
message(STATUS "Target architecture: ${ARCHITECTURE}")
|
||||||
|
|
||||||
|
|
||||||
# Configure compilation flags
|
# Configure C++ standard
|
||||||
# ===========================
|
# ===========================
|
||||||
|
|
||||||
set(CMAKE_CXX_STANDARD 17)
|
set(CMAKE_CXX_STANDARD 17)
|
||||||
set(CMAKE_CXX_STANDARD_REQUIRED ON)
|
set(CMAKE_CXX_STANDARD_REQUIRED ON)
|
||||||
|
|
||||||
if (NOT MSVC)
|
|
||||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-attributes")
|
|
||||||
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS}")
|
|
||||||
|
|
||||||
if (MINGW)
|
|
||||||
add_definitions(-DMINGW_HAS_SECURE_API)
|
|
||||||
|
|
||||||
if (MINGW_STATIC_BUILD)
|
|
||||||
add_definitions(-DQT_STATICPLUGIN)
|
|
||||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -static")
|
|
||||||
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -static")
|
|
||||||
endif()
|
|
||||||
endif()
|
|
||||||
else()
|
|
||||||
# Silence "deprecation" warnings
|
|
||||||
add_definitions(/D_CRT_SECURE_NO_WARNINGS /D_CRT_NONSTDC_NO_DEPRECATE /D_SCL_SECURE_NO_WARNINGS)
|
|
||||||
# Avoid windows.h junk
|
|
||||||
add_definitions(/DNOMINMAX)
|
|
||||||
# Avoid windows.h from including some usually unused libs like winsocks.h, since this might cause some redefinition errors.
|
|
||||||
add_definitions(/DWIN32_LEAN_AND_MEAN)
|
|
||||||
|
|
||||||
set(CMAKE_CONFIGURATION_TYPES Debug Release CACHE STRING "" FORCE)
|
|
||||||
|
|
||||||
# Tweak optimization settings
|
|
||||||
# As far as I can tell, there's no way to override the CMake defaults while leaving user
|
|
||||||
# changes intact, so we'll just clobber everything and say sorry.
|
|
||||||
message(STATUS "Cache compiler flags ignored, please edit CMakeLists.txt to change the flags.")
|
|
||||||
|
|
||||||
# /W3 - Level 3 warnings
|
|
||||||
# /MP - Multi-threaded compilation
|
|
||||||
# /Zi - Output debugging information
|
|
||||||
# /Zo - enhanced debug info for optimized builds
|
|
||||||
# /permissive- - enables stricter C++ standards conformance checks
|
|
||||||
set(CMAKE_C_FLAGS "/W3 /MP /Zi /Zo /permissive-" CACHE STRING "" FORCE)
|
|
||||||
# /EHsc - C++-only exception handling semantics
|
|
||||||
# /Zc:throwingNew - let codegen assume `operator new` will never return null
|
|
||||||
# /Zc:inline - let codegen omit inline functions in object files
|
|
||||||
set(CMAKE_CXX_FLAGS "${CMAKE_C_FLAGS} /EHsc /std:c++latest /Zc:throwingNew,inline" CACHE STRING "" FORCE)
|
|
||||||
|
|
||||||
# /MDd - Multi-threaded Debug Runtime DLL
|
|
||||||
set(CMAKE_C_FLAGS_DEBUG "/Od /MDd" CACHE STRING "" FORCE)
|
|
||||||
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG}" CACHE STRING "" FORCE)
|
|
||||||
|
|
||||||
# /O2 - Optimization level 2
|
|
||||||
# /GS- - No stack buffer overflow checks
|
|
||||||
# /MD - Multi-threaded runtime DLL
|
|
||||||
set(CMAKE_C_FLAGS_RELEASE "/O2 /GS- /MD" CACHE STRING "" FORCE)
|
|
||||||
set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_C_FLAGS_RELEASE}" CACHE STRING "" FORCE)
|
|
||||||
|
|
||||||
set(CMAKE_EXE_LINKER_FLAGS_DEBUG "/DEBUG /MANIFEST:NO" CACHE STRING "" FORCE)
|
|
||||||
set(CMAKE_EXE_LINKER_FLAGS_RELEASE "/DEBUG /MANIFEST:NO /INCREMENTAL:NO /OPT:REF,ICF" CACHE STRING "" FORCE)
|
|
||||||
endif()
|
|
||||||
|
|
||||||
# Set file offset size to 64 bits.
|
|
||||||
#
|
|
||||||
# On modern Unixes, this is typically already the case. The lone exception is
|
|
||||||
# glibc, which may default to 32 bits. glibc allows this to be configured
|
|
||||||
# by setting _FILE_OFFSET_BITS.
|
|
||||||
if(CMAKE_SYSTEM_NAME STREQUAL "Linux" OR MINGW)
|
|
||||||
add_definitions(-D_FILE_OFFSET_BITS=64)
|
|
||||||
endif()
|
|
||||||
|
|
||||||
# CMake seems to only define _DEBUG on Windows
|
|
||||||
set_property(DIRECTORY APPEND PROPERTY
|
|
||||||
COMPILE_DEFINITIONS $<$<CONFIG:Debug>:_DEBUG> $<$<NOT:$<CONFIG:Debug>>:NDEBUG>)
|
|
||||||
|
|
||||||
# System imported libraries
|
# System imported libraries
|
||||||
# ======================
|
# ======================
|
||||||
|
|
||||||
|
|
@ -326,25 +260,21 @@ endif()
|
||||||
# Platform-specific library requirements
|
# Platform-specific library requirements
|
||||||
# ======================================
|
# ======================================
|
||||||
|
|
||||||
IF (APPLE)
|
if (APPLE)
|
||||||
find_library(COCOA_LIBRARY Cocoa) # Umbrella framework for everything GUI-related
|
# Umbrella framework for everything GUI-related
|
||||||
|
find_library(COCOA_LIBRARY Cocoa)
|
||||||
set(PLATFORM_LIBRARIES ${COCOA_LIBRARY} ${IOKIT_LIBRARY} ${COREVIDEO_LIBRARY})
|
set(PLATFORM_LIBRARIES ${COCOA_LIBRARY} ${IOKIT_LIBRARY} ${COREVIDEO_LIBRARY})
|
||||||
|
elseif (WIN32)
|
||||||
if (CMAKE_CXX_COMPILER_ID STREQUAL Clang)
|
|
||||||
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -stdlib=libc++")
|
|
||||||
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -stdlib=libc++")
|
|
||||||
endif()
|
|
||||||
ELSEIF (WIN32)
|
|
||||||
# WSAPoll and SHGetKnownFolderPath (AppData/Roaming) didn't exist before WinNT 6.x (Vista)
|
# WSAPoll and SHGetKnownFolderPath (AppData/Roaming) didn't exist before WinNT 6.x (Vista)
|
||||||
add_definitions(-D_WIN32_WINNT=0x0600 -DWINVER=0x0600)
|
add_definitions(-D_WIN32_WINNT=0x0600 -DWINVER=0x0600)
|
||||||
set(PLATFORM_LIBRARIES winmm ws2_32)
|
set(PLATFORM_LIBRARIES winmm ws2_32)
|
||||||
IF (MINGW)
|
if (MINGW)
|
||||||
# PSAPI is the Process Status API
|
# PSAPI is the Process Status API
|
||||||
set(PLATFORM_LIBRARIES ${PLATFORM_LIBRARIES} psapi imm32 version)
|
set(PLATFORM_LIBRARIES ${PLATFORM_LIBRARIES} psapi imm32 version)
|
||||||
ENDIF (MINGW)
|
endif()
|
||||||
ELSEIF (CMAKE_SYSTEM_NAME MATCHES "^(Linux|kFreeBSD|GNU|SunOS)$")
|
elseif (CMAKE_SYSTEM_NAME MATCHES "^(Linux|kFreeBSD|GNU|SunOS)$")
|
||||||
set(PLATFORM_LIBRARIES rt)
|
set(PLATFORM_LIBRARIES rt)
|
||||||
ENDIF (APPLE)
|
endif()
|
||||||
|
|
||||||
# Setup a custom clang-format target (if clang-format can be found) that will run
|
# Setup a custom clang-format target (if clang-format can be found) that will run
|
||||||
# against all the src files. This should be used before making a pull request.
|
# against all the src files. This should be used before making a pull request.
|
||||||
|
|
|
||||||
|
|
@ -1,18 +1,87 @@
|
||||||
# Enable modules to include each other's files
|
# Enable modules to include each other's files
|
||||||
include_directories(.)
|
include_directories(.)
|
||||||
|
|
||||||
|
# CMake seems to only define _DEBUG on Windows
|
||||||
|
set_property(DIRECTORY APPEND PROPERTY
|
||||||
|
COMPILE_DEFINITIONS $<$<CONFIG:Debug>:_DEBUG> $<$<NOT:$<CONFIG:Debug>>:NDEBUG>)
|
||||||
|
|
||||||
|
# Set compilation flags
|
||||||
|
if (MSVC)
|
||||||
|
# Silence "deprecation" warnings
|
||||||
|
add_definitions(/D_CRT_SECURE_NO_WARNINGS /D_CRT_NONSTDC_NO_DEPRECATE /D_SCL_SECURE_NO_WARNINGS)
|
||||||
|
# Avoid windows.h junk
|
||||||
|
add_definitions(/DNOMINMAX)
|
||||||
|
# Avoid windows.h from including some usually unused libs like winsocks.h, since this might cause some redefinition errors.
|
||||||
|
add_definitions(/DWIN32_LEAN_AND_MEAN)
|
||||||
|
|
||||||
|
set(CMAKE_CONFIGURATION_TYPES Debug Release CACHE STRING "" FORCE)
|
||||||
|
|
||||||
|
# Tweak optimization settings
|
||||||
|
# As far as I can tell, there's no way to override the CMake defaults while leaving user
|
||||||
|
# changes intact, so we'll just clobber everything and say sorry.
|
||||||
|
message(STATUS "Cache compiler flags ignored, please edit CMakeLists.txt to change the flags.")
|
||||||
|
|
||||||
|
# /W3 - Level 3 warnings
|
||||||
|
# /MP - Multi-threaded compilation
|
||||||
|
# /Zi - Output debugging information
|
||||||
|
# /Zo - enhanced debug info for optimized builds
|
||||||
|
# /permissive- - enables stricter C++ standards conformance checks
|
||||||
|
# /EHsc - C++-only exception handling semantics
|
||||||
|
# /Zc:throwingNew - let codegen assume `operator new` will never return null
|
||||||
|
# /Zc:inline - let codegen omit inline functions in object files
|
||||||
|
set(CMAKE_CXX_FLAGS "/W3 /MP /Zi /Zo /permissive- /EHsc /std:c++latest /Zc:throwingNew,inline" CACHE STRING "" FORCE)
|
||||||
|
|
||||||
|
# /O2 - Optimization level 2
|
||||||
|
# /GS- - No stack buffer overflow checks
|
||||||
|
# /MD - Multi-threaded runtime DLL
|
||||||
|
set(CMAKE_C_FLAGS_RELEASE "/O2 /GS- /MD" CACHE STRING "" FORCE)
|
||||||
|
set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_C_FLAGS_RELEASE}" CACHE STRING "" FORCE)
|
||||||
|
|
||||||
|
set(CMAKE_EXE_LINKER_FLAGS_DEBUG "/DEBUG /MANIFEST:NO" CACHE STRING "" FORCE)
|
||||||
|
set(CMAKE_EXE_LINKER_FLAGS_RELEASE "/DEBUG /MANIFEST:NO /INCREMENTAL:NO /OPT:REF,ICF" CACHE STRING "" FORCE)
|
||||||
|
else()
|
||||||
|
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-attributes")
|
||||||
|
|
||||||
|
if (APPLE AND CMAKE_CXX_COMPILER_ID STREQUAL Clang)
|
||||||
|
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -stdlib=libc++")
|
||||||
|
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -stdlib=libc++")
|
||||||
|
endif()
|
||||||
|
|
||||||
|
# Set file offset size to 64 bits.
|
||||||
|
#
|
||||||
|
# On modern Unixes, this is typically already the case. The lone exception is
|
||||||
|
# glibc, which may default to 32 bits. glibc allows this to be configured
|
||||||
|
# by setting _FILE_OFFSET_BITS.
|
||||||
|
if(CMAKE_SYSTEM_NAME STREQUAL "Linux" OR MINGW)
|
||||||
|
add_definitions(-D_FILE_OFFSET_BITS=64)
|
||||||
|
endif()
|
||||||
|
|
||||||
|
if (MINGW)
|
||||||
|
add_definitions(-DMINGW_HAS_SECURE_API)
|
||||||
|
|
||||||
|
if (MINGW_STATIC_BUILD)
|
||||||
|
add_definitions(-DQT_STATICPLUGIN)
|
||||||
|
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -static")
|
||||||
|
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -static")
|
||||||
|
endif()
|
||||||
|
endif()
|
||||||
|
endif()
|
||||||
|
|
||||||
add_subdirectory(common)
|
add_subdirectory(common)
|
||||||
add_subdirectory(core)
|
add_subdirectory(core)
|
||||||
add_subdirectory(audio_core)
|
add_subdirectory(audio_core)
|
||||||
add_subdirectory(video_core)
|
add_subdirectory(video_core)
|
||||||
add_subdirectory(input_common)
|
add_subdirectory(input_common)
|
||||||
add_subdirectory(tests)
|
add_subdirectory(tests)
|
||||||
|
|
||||||
if (ENABLE_SDL2)
|
if (ENABLE_SDL2)
|
||||||
add_subdirectory(yuzu_cmd)
|
add_subdirectory(yuzu_cmd)
|
||||||
endif()
|
endif()
|
||||||
|
|
||||||
if (ENABLE_QT)
|
if (ENABLE_QT)
|
||||||
add_subdirectory(yuzu)
|
add_subdirectory(yuzu)
|
||||||
endif()
|
endif()
|
||||||
|
|
||||||
if (ENABLE_WEB_SERVICE)
|
if (ENABLE_WEB_SERVICE)
|
||||||
add_subdirectory(web_service)
|
add_subdirectory(web_service)
|
||||||
endif()
|
endif()
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue