fsfw/fsfw
fsfw
/
fsfw
10
3
Fork 6
Code Issues 31 Pull Requests 4 Releases 10 Wiki Activity

Add LTO support #616

Merged
gaisser merged 15 commits from mueller/add-lto-support into development 2022-05-12 16:56:55 +02:00
Conversation 0 Commits 15 Files Changed 2 +5 -6
1 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit e05c72b062 - Show all commits

11
CMakeLists.txt
View File

@ -36,13 +36,12 @@ set(FSFW_CATCH2_LIB_VERSION v${FSFW_CATCH2_LIB_MAJOR_VERSION}.0.0-preview5 CACHE
# Keep this off by default for now. See PR: https://egit.irs.uni-stuttgart.de/fsfw/fsfw/pulls/616
# for information which keeping this on by default is problematic
option(FSFW_ENABLE_IPO "Enable interprocedural optimization or link-time optimization if available" OFF)
if(FSFW_ENABLE_IPO)
gaisser marked this conversation as resolved Outdated
gaisser commented 2022-05-10 11:53:39 +02:00
Outdated
Review
Copy Link

Do we need that check if FSFW_ENABLE_IPO is off?

Do we need that check if FSFW_ENABLE_IPO is off?
muellerr commented 2022-05-10 13:07:42 +02:00
Outdated
Review
Copy Link

fixed it

fixed it
include(CheckIPOSupported)
check_ipo_supported(RESULT IPO_SUPPORTED OUTPUT IPO_ERROR)
if(NOT IPO_SUPPORTED)
message(STATUS "FSFW | IPO/LTO not supported: ${IPO_ERROR}")
endif()
include(CheckIPOSupported)
check_ipo_supported(RESULT IPO_SUPPORTED OUTPUT IPO_ERROR)
if(NOT IPO_SUPPORTED)
message(STATUS "FSFW | IPO/LTO not supported: ${IPO_ERROR}")
endif()
endif()
option(FSFW_GENERATE_SECTIONS