X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/b34a0fc8695b7a1556c9b933400bbe6efb068020..402657efbc46fcd19d49946f07569e59556ffbaa:/tools/cmake/GCCFlags.cmake diff --git a/tools/cmake/GCCFlags.cmake b/tools/cmake/GCCFlags.cmake index f71ff39fe0..514075919d 100644 --- a/tools/cmake/GCCFlags.cmake +++ b/tools/cmake/GCCFlags.cmake @@ -13,27 +13,46 @@ set(warnCFLAGS "") set(optCFLAGS "") set(warnCXXFLAGS "") -if(enable_compile_warnings AND enable_debug) - 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 -Wno-format-nonliteral -Werror") +if(enable_compile_warnings) + 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_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 -Wno-unused-local-typedefs -Wno-error=attributes") endif() - 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 -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_CXX_COMPILER_ID MATCHES "Clang|GCC") + set(warnCXXFLAGS "${warnCXXFLAGS} -Wno-format-nonliteral") + endif() if(CMAKE_COMPILER_IS_GNUCXX) 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(warnCXXFLAGS "${warnCXXFLAGS} -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() - # the one specific to C but refused by C++ + # the one specific to C but refused by C++ set(warnCFLAGS "${warnCFLAGS} -Wmissing-prototypes") - set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -Wall") + if(CMAKE_Fotran_COMPILER_ID MATCHES "GCC|PGI") + set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -Wall") + endif() + if(CMAKE_Fotran_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) @@ -57,7 +76,7 @@ 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 @@ -115,8 +134,8 @@ if(enable_model-checking AND enable_compile_optimizations) 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.cpp src/xbt/dict_elm.c src/xbt/dict_cursor.c - src/xbt/dynar.cpp 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/dynar.cpp src/xbt/heap.c src/xbt/swag.c + src/xbt/str.c src src/xbt/snprintf.c src/xbt/queue.c src/xbt/xbt_os_time.c src/xbt/xbt_os_thread.c src/xbt/backtrace_linux.c @@ -146,7 +165,7 @@ 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_COMMAND "${GCOV_PATH} -p" CACHE TYPE FILEPATH FORCE) 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") @@ -157,6 +176,25 @@ if(enable_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) +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}") @@ -173,8 +211,8 @@ if(NOT $ENV{LDFLAGS} STREQUAL "") endif() if(MINGW) - # 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 + # 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") @@ -187,7 +225,7 @@ if(MINGW) 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(ARCH_32_BITS) + 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()