2015-10-29 08:13:02 +01:00
|
|
|
From: Alexander Dahl <alex@netz39.de>
|
|
|
|
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 <post@lespocky.de>
|
|
|
|
---
|
2016-04-02 12:38:28 +02:00
|
|
|
cmake/checks.cmake | 14 --------------
|
|
|
|
src/CMakeLists.txt | 9 +++++++++
|
|
|
|
2 files changed, 9 insertions(+), 14 deletions(-)
|
2015-10-29 08:13:02 +01:00
|
|
|
|
|
|
|
diff --git a/cmake/checks.cmake b/cmake/checks.cmake
|
2016-04-02 12:38:28 +02:00
|
|
|
index 27c073f..8a671a1 100644
|
2015-10-29 08:13:02 +01:00
|
|
|
--- 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 <sys/types.h>
|
|
|
|
#include <sys/socket.h>
|
|
|
|
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
|
2016-04-02 12:38:28 +02:00
|
|
|
index 829b3ab..7612b40 100644
|
2015-10-29 08:13:02 +01:00
|
|
|
--- a/src/CMakeLists.txt
|
|
|
|
+++ b/src/CMakeLists.txt
|
2016-04-02 12:38:28 +02:00
|
|
|
@@ -48,6 +48,15 @@ add_executable(fastd
|
2015-10-29 08:13:02 +01:00
|
|
|
set_property(TARGET fastd PROPERTY COMPILE_FLAGS "${FASTD_CFLAGS}")
|
2016-04-02 12:38:28 +02:00
|
|
|
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})
|
2015-10-29 08:13:02 +01:00
|
|
|
+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)
|
2016-04-02 12:38:28 +02:00
|
|
|
target_link_libraries(fastd protocols methods ciphers macs ${RT_LIBRARY} ${LIBCAP_LIBRARY} ${LIBUECC_LIBRARIES} ${NACL_LIBRARIES} ${OPENSSL_CRYPTO_LIBRARY} ${JSON_C_LIBRARIES})
|
2015-10-29 08:13:02 +01:00
|
|
|
|
|
|
|
add_dependencies(fastd version)
|
|
|
|
--
|
|
|
|
2.1.4
|
|
|
|
|