From d625b4d1ba9dbcbb2f7a8f6555e1658569aaa093 Mon Sep 17 00:00:00 2001 From: Ilya Fedin Date: Wed, 3 May 2023 18:00:09 +0400 Subject: [PATCH] Get rid of -Ofast on Linux This seem to make some miscompilation that lead to hangs --- external/dispatch/CMakeLists.txt | 12 +++--------- external/jemalloc/CMakeLists.txt | 4 ++-- options_linux.cmake | 1 - 3 files changed, 5 insertions(+), 12 deletions(-) diff --git a/external/dispatch/CMakeLists.txt b/external/dispatch/CMakeLists.txt index 3a463cf..0431d88 100644 --- a/external/dispatch/CMakeLists.txt +++ b/external/dispatch/CMakeLists.txt @@ -14,17 +14,11 @@ if (DESKTOP_APP_USE_PACKAGED) endif() if (NOT DESKTOP_APP_DISPATCH_LIBRARIES OR NOT DESKTOP_APP_DISPATCH_INCLUDE_DIRS) - if (DESKTOP_APP_SPECIAL_TARGET) - set(dispatch_extra_args - -DCMAKE_BUILD_TYPE=None - "-DCMAKE_C_FLAGS=-g -Ofast -pipe -fPIC -fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS" - "-DCMAKE_CXX_FLAGS=-g -Ofast -pipe -fPIC -fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS" - ) - elseif (NOT DESKTOP_APP_USE_PACKAGED) + if (NOT DESKTOP_APP_USE_PACKAGED OR DESKTOP_APP_SPECIAL_TARGET) set(dispatch_extra_args -DCMAKE_BUILD_TYPE=Release - "-DCMAKE_C_FLAGS=-pipe -fPIC -fstack-protector-all -D_GLIBCXX_ASSERTIONS" - "-DCMAKE_CXX_FLAGS=-pipe -fPIC -fstack-protector-all -D_GLIBCXX_ASSERTIONS" + "-DCMAKE_C_FLAGS=-g -pipe -fPIC -fstack-protector-all -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS" + "-DCMAKE_CXX_FLAGS=-g -pipe -fPIC -fstack-protector-all -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS" ) elseif (DEFINED CMAKE_BUILD_TYPE) set(dispatch_extra_args diff --git a/external/jemalloc/CMakeLists.txt b/external/jemalloc/CMakeLists.txt index 0ce500a..c66ea66 100644 --- a/external/jemalloc/CMakeLists.txt +++ b/external/jemalloc/CMakeLists.txt @@ -39,8 +39,8 @@ ExternalProject_Add(jemalloc CONFIGURE_COMMAND env CC=clang CXX=clang++ - "$>,$>,EXTRA_CFLAGS=$<$:-Ofast> -fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS,=>" - "$>,$>,EXTRA_CXXFLAGS=$<$:-Ofast> -fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS,=>" + "$>,$>,EXTRA_CFLAGS=-fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS,=>" + "$>,$>,EXTRA_CXXFLAGS=-fstack-protector-all -DNDEBUG -D_FORTIFY_SOURCE=2 -D_GLIBCXX_ASSERTIONS,=>" ./autogen.sh --disable-shared BUILD_COMMAND make $<$>:-j${N}> BUILD_IN_SOURCE 1 diff --git a/options_linux.cmake b/options_linux.cmake index ef93679..7e9aec0 100644 --- a/options_linux.cmake +++ b/options_linux.cmake @@ -36,7 +36,6 @@ INTERFACE if (DESKTOP_APP_SPECIAL_TARGET) target_compile_options(common_options INTERFACE - $<$>:-Ofast> -Werror $<$>:-g> $<$>:-flto=auto>