From: Alexander Dahl Date: Wed, 28 Oct 2015 16:04:13 +0100 Subject: [PATCH] cmake: use INTERPROCEDURAL_OPTIMIZATION target property Instead of hacking on compiler flags use the cmake way for handling interprocedural optimization. Tests showed cmake currently ignores this for gcc and clang, at least on Debian Wheezy and Jessie. This actually results in no interprocedural optimization for the moment. However it allows to compile fastd against a toolchain without LTO support, which was broken because of a hack included here for finding binutils. According to the upstream author of fastd the main reason for enabling LTO was binary size on an OpenWRT target for Freifunk Gluon, where they have very few space left on devices with only 4 MB flash memory. Signed-off-by: Alexander Dahl --- cmake/checks.cmake | 14 -------------- src/CMakeLists.txt | 9 +++++++++ 2 files changed, 9 insertions(+), 14 deletions(-) diff --git a/cmake/checks.cmake b/cmake/checks.cmake index 27c073f..8a671a1 100644 --- a/cmake/checks.cmake +++ b/cmake/checks.cmake @@ -10,20 +10,6 @@ if(ARCH_X86 OR ARCH_X86_64) endif(ARCH_X86 OR ARCH_X86_64) - -if(ENABLE_LTO) - set(CFLAGS_LTO "-flto") - set(CFLAGS_NO_LTO "-fno-lto") - - check_c_compiler_flag("-fwhole-program" HAVE_FLAG_WHOLE_PROGRAM) - if(HAVE_FLAG_WHOLE_PROGRAM) - set(LDFLAGS_LTO "-flto -fwhole-program") - else(HAVE_FLAG_WHOLE_PROGRAM) - set(LDFLAGS_LTO "-flto") - endif(HAVE_FLAG_WHOLE_PROGRAM) -endif(ENABLE_LTO) - - check_c_source_compiles(" #include #include diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 829b3ab..7612b40 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -48,6 +48,15 @@ add_executable(fastd set_property(TARGET fastd PROPERTY COMPILE_FLAGS "${FASTD_CFLAGS}") set_property(TARGET fastd PROPERTY LINK_FLAGS "${PTHREAD_LDFLAGS} ${LIBUECC_LDFLAGS_OTHER} ${NACL_LDFLAGS_OTHER} ${JSON_C_LDFLAGS_OTHER} ${LDFLAGS_LTO}") set_property(TARGET fastd APPEND PROPERTY INCLUDE_DIRECTORIES ${LIBCAP_INCLUDE_DIR} ${NACL_INCLUDE_DIRS} ${JSON_C_INCLUDE_DIR}) +if(ENABLE_LTO) + set_target_properties(fastd PROPERTIES + INTERPROCEDURAL_OPTIMIZATION ON + ) +else(ENABLE_LTO) + set_target_properties(fastd PROPERTIES + INTERPROCEDURAL_OPTIMIZATION OFF + ) +endif(ENABLE_LTO) target_link_libraries(fastd protocols methods ciphers macs ${RT_LIBRARY} ${LIBCAP_LIBRARY} ${LIBUECC_LIBRARIES} ${NACL_LIBRARIES} ${OPENSSL_CRYPTO_LIBRARY} ${JSON_C_LIBRARIES}) add_dependencies(fastd version) -- 2.1.4