X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/1e5bc975432fb8d5f2501e3bafa85ad4a6f2e859..323010d1c247d0097b2cdc79d1da143846461999:/tools/cmake/GCCFlags.cmake diff --git a/tools/cmake/GCCFlags.cmake b/tools/cmake/GCCFlags.cmake index 6b41adf478..2b51c858f6 100644 --- a/tools/cmake/GCCFlags.cmake +++ b/tools/cmake/GCCFlags.cmake @@ -5,33 +5,74 @@ ## ## These flags do break some classical CMake tests, so you don't ## want to do so before the very end of the configuration. -## +## ## Other compiler flags (C/C++ standard version) are tested and set ## by the beginning of the configuration, directly in ~/CMakeList.txt set(warnCFLAGS "") set(optCFLAGS "") - - +set(warnCXXFLAGS "") if(enable_compile_warnings) - set(warnCFLAGS "-fno-common -Wall -Wunused -Wmissing-prototypes -Wmissing-declarations -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing -Wno-format-nonliteral -Werror ") + set(warnCFLAGS "-fno-common -Wall -Wunused -Wmissing-declarations -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing") + if(CMAKE_COMPILER_IS_GNUCC AND (NOT (CMAKE_C_COMPILER_VERSION VERSION_LESS "5.0"))) + set(warnCFLAGS "${warnCFLAGS} -Wformat-signedness") + endif() + if(CMAKE_C_COMPILER_ID MATCHES "Clang|GCC") + set(warnCFLAGS "${warnCFLAGS} -Wno-format-nonliteral") + endif() if(CMAKE_COMPILER_IS_GNUCC) - set(warnCFLAGS "${warnCFLAGS}-Wclobbered -Wno-error=clobbered ") + set(warnCFLAGS "${warnCFLAGS} -Wclobbered -Wno-error=clobbered -Wno-unused-local-typedefs -Wno-error=attributes") + endif() + if (CMAKE_CXX_COMPILER_ID MATCHES "Intel") + # ignore remark #1418: external function definition with no prior declaration + # 3179: deprecated conversion of string literal to char* (should be const char*) + # 191: type qualifier is meaningless on cast type + # 597: entity-kind "entity" will not be called for implicit or explicit conversions + set(warnCFLAGS "${warnCFLAGS} -wd1418 -wd191 -wd3179 -ww597") endif() - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Wunused -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing -Wno-format-nonliteral -Werror") + set(warnCXXFLAGS "${warnCFLAGS} -Wall -Wextra -Wunused -Wmissing-declarations -Wpointer-arith -Wchar-subscripts -Wcomment -Wformat -Wwrite-strings -Wno-unused-function -Wno-unused-parameter -Wno-strict-aliasing") + if(CMAKE_COMPILER_IS_GNUCXX AND (NOT (CMAKE_CXX_COMPILER_VERSION VERSION_LESS "5.0"))) + set(warnCFLAGS "${warnCFLAGS} -Wformat-signedness") + endif() + if(CMAKE_CXX_COMPILER_ID MATCHES "Clang|GCC") + set(warnCXXFLAGS "${warnCXXFLAGS} -Wno-format-nonliteral") + endif() if(CMAKE_COMPILER_IS_GNUCXX) - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wclobbered -Wno-error=clobbered") + set(warnCXXFLAGS "${warnCXXFLAGS} -Wclobbered -Wno-error=clobbered -Wno-unused-local-typedefs -Wno-error=attributes") endif() - if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") # don't care about class that become struct - set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-mismatched-tags") + if (CMAKE_CXX_COMPILER_ID MATCHES "Clang") + # don't care about class that become struct, avoid issue of empty C structs + # size (coming from libunwind.h) + set(warnCXXFLAGS "${warnCXXFLAGS} -Wno-mismatched-tags -Wno-extern-c-compat") endif() - set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -Wall") + # the one specific to C but refused by C++ + set(warnCFLAGS "${warnCFLAGS} -Wmissing-prototypes") + + if(CMAKE_Fortran_COMPILER_ID MATCHES "GCC|PGI") + set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -Wall") + endif() + if(CMAKE_Fortran_COMPILER_ID MATCHES "Intel") + set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -warn all") + endif() set(CMAKE_JAVA_COMPILE_FLAGS "-Xlint") endif() +# NDEBUG gives a lot of "initialized but unused variables" errors. Don't die anyway. +if(enable_compile_warnings AND enable_debug) + set(warnCFLAGS "${warnCFLAGS} -Werror") + set(warnCXXFLAGS "${warnCXXFLAGS} -Werror") +endif() + +# Activate the warnings on #if FOOBAR when FOOBAR has no value +# It breaks on FreeBSD within Boost headers, so activate this only in Pure Hardcore debug mode. +if(enable_maintainer_mode) + set(warnCFLAGS "${warnCFLAGS} -Wundef") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wundef") +endif() + # Se the optimisation flags # NOTE, we should CMAKE_BUILD_TYPE for this if(enable_compile_optimizations) @@ -45,41 +86,56 @@ if(enable_compile_optimizations AND CMAKE_COMPILER_IS_GNUCC set(optCFLAGS "${optCFLAGS} -finline-functions ") endif() +# Do not leak the current directory into the binaries +if(CMAKE_COMPILER_IS_GNUCC) + execute_process(COMMAND realpath --relative-to=${CMAKE_BINARY_DIR} ${CMAKE_SOURCE_DIR} + RESULT_VARIABLE RESULT OUTPUT_VARIABLE RELATIVE_SOURCE_DIR ERROR_QUIET OUTPUT_STRIP_TRAILING_WHITESPACE) + if(RESULT EQUAL 0) + message(STATUS "Relative source directory is \"${RELATIVE_SOURCE_DIR}\".") + else() + message(WARNING "Failed to find relative source directory. Using \".\".") + set(RELATIVE_SOURCE_DIR ".") + endif() + set(optCFLAGS "${optCFLAGS} -fdebug-prefix-map=${CMAKE_SOURCE_DIR}=${RELATIVE_SOURCE_DIR}") +endif() + # Configure LTO # NOTE, cmake 3.0 has a INTERPROCEDURAL_OPTIMIZATION target # property for this (http://www.cmake.org/cmake/help/v3.0/prop_tgt/INTERPROCEDURAL_OPTIMIZATION.html) -if(enable_lto) # User wants LTO. Try if we can do that +if(enable_lto) # User wants LTO. Try if we can do that set(enable_lto OFF) if(enable_compile_optimizations AND CMAKE_COMPILER_IS_GNUCC AND (NOT enable_model-checking)) - if(WIN32) - if (COMPILER_C_VERSION_MAJOR_MINOR STRGREATER "4.8") - # On windows, we need 4.8 or higher to enable lto because of http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50293 - # - # We are experiencing assertion failures even with 4.8 on MinGW. - # Push the support forward: will see if 4.9 works when we test it. - set(enable_lto ON) - endif() - elseif(LINKER_VERSION STRGREATER "2.22") + # On windows, we need 4.8 or higher to enable lto because of http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50293 + # We are experiencing assertion failures even with 4.8 on MinGW. + # Push the support forward: will see if 4.9 works when we test it. + # + # On Linux, we got the following with GCC 4.8.4 on Centos and Ubuntu + # lto1: internal compiler error: in output_die, at dwarf2out.c:8478 + # Please submit a full bug report, with preprocessed source if appropriate. + # So instead, we push the support forward + + if ( (CMAKE_C_COMPILER_VERSION VERSION_GREATER "4.8.5") + AND (LINKER_VERSION VERSION_GREATER "2.22")) set(enable_lto ON) endif() endif() if(enable_lto) - message("-- LTO seems usable.") + message(STATUS "LTO seems usable.") else() if(NOT enable_compile_optimizations) - message("-- LTO disabled: Compile-time optimizations turned off.") - else() + message(STATUS "LTO disabled: Compile-time optimizations turned off.") + else() if(enable_model-checking) - message("-- LTO disabled when compiling with model-checking.") + message(STATUS "LTO disabled when compiling with model-checking.") else() - message("-- LTO does not seem usable -- try updating your build chain.") - endif() + message(STATUS "LTO does not seem usable -- try updating your build chain.") + endif() endif() endif() else() - message("-- LTO disabled on the command line.") + message(STATUS "LTO disabled on the command line.") endif() if(enable_lto) # User wants LTO, and it seems usable. Go for it set(optCFLAGS "${optCFLAGS} -flto ") @@ -89,7 +145,7 @@ if(enable_lto) # User wants LTO, and it seems usable. Go for it # these objects you have to use the gcc wrappers: # gcc-ar, gcc-nm and gcc-ranlib." if(${CMAKE_C_COMPILER_ID} STREQUAL "GNU" - AND COMPILER_C_VERSION_MAJOR_MINOR STRGREATER "4.8") + AND CMAKE_C_COMPILER_VERSION VERSION_GREATER "4.8") set (CMAKE_AR gcc-ar) set (CMAKE_RANLIB gcc-ranlib) endif() @@ -100,18 +156,15 @@ if(enable_model-checking AND enable_compile_optimizations) set(optCFLAGS "-O0 ") # But you can still optimize this: foreach(s + src/kernel/lmm/fair_bottleneck.cpp src/kernel/lmm/lagrange.cpp src/kernel/lmm/maxmin.cpp src/xbt/mmalloc/mm.c src/xbt/log.c src/xbt/xbt_log_appender_file.c src/xbt/xbt_log_layout_format.c src/xbt/xbt_log_layout_simple.c - src/xbt/dict.c src/xbt/dict_elm.c src/xbt/dict_multi.c src/xbt/dict_cursor.c - src/xbt/set.c src/xbt/setset.c - src/xbt/dynar.c src/xbt/fifo.c src/xbt/heap.c src/xbt/swag.c - src/xbt/str.c src/xbt/strbuff.c src/xbt/snprintf.c - src/xbt/queue.c + src/xbt/dict.cpp src/xbt/dict_elm.c src/xbt/dict_cursor.c + src/xbt/dynar.cpp + src/xbt/xbt_str.cpp src/xbt/snprintf.c src/xbt/xbt_os_time.c src/xbt/xbt_os_thread.c - src/xbt/sha.c - src/xbt/matrix.c - src/xbt/backtrace_linux.c + src/xbt/backtrace_linux.cpp ${MC_SRC_BASE} ${MC_SRC}) set (mcCFLAGS "-O3 -funroll-loops -fno-strict-aliasing") if(CMAKE_COMPILER_IS_GNUCC) @@ -121,38 +174,69 @@ if(enable_model-checking AND enable_compile_optimizations) endforeach() endif() +if (CMAKE_C_COMPILER_ID MATCHES "Intel") + # honor parentheses when determining the order of expression evaluation. + set(optCFLAGS "${optCFLAGS} -fprotect-parens ") +endif() + if(NOT enable_debug) set(CMAKE_C_FLAGS "-DNDEBUG ${CMAKE_C_FLAGS}") set(CMAKE_CXX_FLAGS "-DNDEBUG ${CMAKE_CXX_FLAGS}") endif() -if(enable_msg_deprecated) - set(CMAKE_C_FLAGS "-DMSG_USE_DEPRECATED ${CMAKE_C_FLAGS}") -endif() - -set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${optCFLAGS} ${warnCFLAGS}") -set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${optCFLAGS}") +set(CMAKE_C_FLAGS "${warnCFLAGS} ${CMAKE_C_FLAGS} ${optCFLAGS}") +set(CMAKE_CXX_FLAGS "${warnCXXFLAGS} ${CMAKE_CXX_FLAGS} ${optCFLAGS}") # Try to make Mac a bit more complient to open source standards if(CMAKE_SYSTEM_NAME MATCHES "Darwin") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -D_XOPEN_SOURCE=700 -D_DARWIN_C_SOURCE") endif() +# Avoid a failure seen with gcc 7.2.0 and ns3 3.27 +if(enable_ns3) + set_source_files_properties(src/surf/network_ns3.cpp PROPERTIES COMPILE_FLAGS " -Wno-unused-local-typedef") +endif() + set(TESH_OPTION "") if(enable_coverage) find_program(GCOV_PATH gcov) if(GCOV_PATH) set(COVERAGE_COMMAND "${GCOV_PATH}" CACHE TYPE FILEPATH FORCE) + set(COVERAGE_EXTRA_FLAGS "-l -p") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -DCOVERAGE") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fprofile-arcs -ftest-coverage") set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fprofile-arcs -ftest-coverage") set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fprofile-arcs -ftest-coverage") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fprofile-arcs -ftest-coverage") - set(TESH_OPTION --enable-coverage) add_definitions(-fprofile-arcs -ftest-coverage) endif() endif() +if(enable_address_sanitizer) + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=address -fno-omit-frame-pointer") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=address -fno-omit-frame-pointer") + set(CMAKE_C_LINK_FLAGS "${CMAKE_C_LINK_FLAGS} -fsanitize=address") + set(TESH_OPTION --enable-sanitizers) + try_compile(HAVE_SANITIZER_ADDRESS ${CMAKE_BINARY_DIR} ${CMAKE_HOME_DIRECTORY}/tools/cmake/test_prog/prog_asan.cpp) + try_compile(HAVE_SANITIZER_ADDRESS_FIBER_SUPPORT ${CMAKE_BINARY_DIR} ${CMAKE_HOME_DIRECTORY}/tools/cmake/test_prog/prog_asan.cpp + COMPILE_DEFINITIONS -DCHECK_FIBER_SUPPORT) +else() + set(HAVE_SANITIZER_ADDRESS FALSE CACHE INTERNAL "") + set(HAVE_SANITIZER_ADDRESS_FIBER_SUPPORT FALSE CACHE INTERNAL "") +endif() + +if(enable_thread_sanitizer) + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=thread -fno-omit-frame-pointer -no-pie") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=thread -fno-omit-frame-pointer -no-pie") + set(CMAKE_C_LINK_FLAGS "${CMAKE_C_LINK_FLAGS} -fsanitize=thread -no-pie") +endif() + +if(enable_undefined_sanitizer) + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -fsanitize=undefined -fno-omit-frame-pointer") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fsanitize=undefined -fno-omit-frame-pointer") + set(CMAKE_C_LINK_FLAGS "${CMAKE_C_LINK_FLAGS} -fsanitize=undefined") +endif() + if(NOT $ENV{CFLAGS} STREQUAL "") message(STATUS "Add CFLAGS: \"$ENV{CFLAGS}\" to CMAKE_C_FLAGS") set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} $ENV{CFLAGS}") @@ -168,12 +252,26 @@ if(NOT $ENV{LDFLAGS} STREQUAL "") set(CMAKE_C_LINK_FLAGS "${CMAKE_C_LINK_FLAGS} $ENV{LDFLAGS}") endif() -# We don't want to ship libgcc_s_seh-1.dll nor libstdc++-6.dll -#  This will probably be troublesome if someone wants to use the lib in -# its own project, but we only want to have java working on Windows. if(MINGW) - set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -static-libgcc") + # http://stackoverflow.com/questions/10452262/create-64-bit-jni-under-windows + # We don't want to ship libgcc_s_seh-1.dll nor libstdc++-6.dll + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -static-libgcc") set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -static-libgcc -static-libstdc++") - set(CMAKE_SHARED_LIBRARY_LINK_C_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_C_FLAGS} -static-libgcc -s") - set(CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS} -static-libgcc -static-libstdc++ -s") + set(CMAKE_SHARED_LIBRARY_LINK_C_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_C_FLAGS} -static-libgcc") + set(CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS} -static-libgcc -static-libstdc++") + + # JNI searches for stdcalls + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wl,--add-stdcall-alias") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wl,--add-stdcall-alias") + set(CMAKE_SHARED_LIBRARY_LINK_C_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_C_FLAGS} -Wl,--add-stdcall-alias") + set(CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS "${CMAKE_SHARED_LIBRARY_LINK_CXX_FLAGS} -Wl,--add-stdcall-alias") + + # Specify the data model that we are using (yeah it may help Java) + if(CMAKE_SIZEOF_VOID_P EQUAL 4) # 32 bits + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -m32") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -m32") + else() + set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -m64") + set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -m64") + endif() endif()