mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-12-05 05:52:05 +00:00
23044ac02c
clang (10.0.0 is fine BTW) will warn about this anyway on CI. For the debug build gcc10: - before patch: - concat.cpp -> >5m (stuck on cc1plus, not as) - has.cpp -> >10m (stuck on cc1plus, not as) - after this patch: - concat.cpp -> 1m16s - has.cpp -> 4m (and most of the time eats, as from binutils 2.34.50.20200508) Command for build: - ninja src/Functions/GatherUtils/CMakeFiles/clickhouse_functions_gatherutils.dir/concat.cpp.o - ninja src/Functions/GatherUtils/CMakeFiles/clickhouse_functions_gatherutils.dir/has.cpp.o The test case should be reduced and then it can be reported to the gcc bugzilla. P.S. Looks like a signal not to switch to gcc10 for now
172 lines
8.0 KiB
CMake
172 lines
8.0 KiB
CMake
# Our principle is to enable as many warnings as possible and always do it with "warnings as errors" flag.
|
|
#
|
|
# But it comes with some cost:
|
|
# - we have to disable some warnings in 3rd party libraries (they are located in "contrib" directory)
|
|
# - we have to include headers of these libraries as -isystem to avoid warnings from headers
|
|
# (this is the same behaviour as if these libraries were located in /usr/include)
|
|
# - sometimes warnings from 3rd party libraries may come from macro substitutions in our code
|
|
# and we have to wrap them with #pragma GCC/clang diagnostic ignored
|
|
|
|
if (NOT MSVC)
|
|
set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wextra")
|
|
endif ()
|
|
|
|
if (USE_DEBUG_HELPERS)
|
|
set (INCLUDE_DEBUG_HELPERS "-I${ClickHouse_SOURCE_DIR}/base -include ${ClickHouse_SOURCE_DIR}/src/Core/iostream_debug_helpers.h")
|
|
set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${INCLUDE_DEBUG_HELPERS}")
|
|
endif ()
|
|
|
|
# Add some warnings that are not available even with -Wall -Wextra -Wpedantic.
|
|
|
|
option (WEVERYTHING "Enables -Weverything option with some exceptions. This is intended for exploration of new compiler warnings that may be found to be useful. Only makes sense for clang." ON)
|
|
|
|
if (COMPILER_CLANG)
|
|
add_warning(pedantic)
|
|
no_warning(vla-extension)
|
|
no_warning(zero-length-array)
|
|
|
|
add_warning(comma)
|
|
add_warning(conditional-uninitialized)
|
|
add_warning(covered-switch-default)
|
|
add_warning(deprecated)
|
|
add_warning(embedded-directive)
|
|
add_warning(empty-init-stmt) # linux-only
|
|
add_warning(extra-semi-stmt) # linux-only
|
|
add_warning(extra-semi)
|
|
add_warning(gnu-case-range)
|
|
add_warning(inconsistent-missing-destructor-override)
|
|
add_warning(newline-eof)
|
|
add_warning(old-style-cast)
|
|
add_warning(range-loop-analysis)
|
|
add_warning(redundant-parens)
|
|
add_warning(reserved-id-macro)
|
|
add_warning(shadow-field) # clang 8+
|
|
add_warning(shadow-uncaptured-local)
|
|
add_warning(shadow)
|
|
add_warning(string-plus-int) # clang 8+
|
|
add_warning(undef)
|
|
add_warning(unreachable-code-return)
|
|
add_warning(unreachable-code)
|
|
add_warning(unused-exception-parameter)
|
|
add_warning(unused-macros)
|
|
add_warning(unused-member-function)
|
|
add_warning(zero-as-null-pointer-constant)
|
|
|
|
if (WEVERYTHING)
|
|
add_warning(everything)
|
|
no_warning(c++98-compat-pedantic)
|
|
no_warning(c++98-compat)
|
|
no_warning(c99-extensions)
|
|
no_warning(conversion)
|
|
no_warning(ctad-maybe-unsupported) # clang 9+, linux-only
|
|
no_warning(deprecated-dynamic-exception-spec)
|
|
no_warning(disabled-macro-expansion)
|
|
no_warning(documentation-unknown-command)
|
|
no_warning(double-promotion)
|
|
no_warning(exit-time-destructors)
|
|
no_warning(float-equal)
|
|
no_warning(global-constructors)
|
|
no_warning(missing-prototypes)
|
|
no_warning(missing-variable-declarations)
|
|
no_warning(nested-anon-types)
|
|
no_warning(packed)
|
|
no_warning(padded)
|
|
no_warning(return-std-move-in-c++11) # clang 7+
|
|
no_warning(shift-sign-overflow)
|
|
no_warning(sign-conversion)
|
|
no_warning(switch-enum)
|
|
no_warning(undefined-func-template)
|
|
no_warning(unused-template)
|
|
no_warning(vla)
|
|
no_warning(weak-template-vtables)
|
|
no_warning(weak-vtables)
|
|
|
|
# TODO Enable conversion, sign-conversion, double-promotion warnings.
|
|
endif ()
|
|
elseif (COMPILER_GCC)
|
|
# Add compiler options only to c++ compiler
|
|
function(add_cxx_compile_options option)
|
|
add_compile_options("$<$<STREQUAL:$<TARGET_PROPERTY:LINKER_LANGUAGE>,CXX>:${option}>")
|
|
endfunction()
|
|
# Warn about boolean expression compared with an integer value different from true/false
|
|
add_cxx_compile_options(-Wbool-compare)
|
|
# Warn whenever a pointer is cast such that the required alignment of the target is increased.
|
|
add_cxx_compile_options(-Wcast-align)
|
|
# Warn whenever a pointer is cast so as to remove a type qualifier from the target type.
|
|
add_cxx_compile_options(-Wcast-qual)
|
|
# Warn when deleting a pointer to incomplete type, which may cause undefined behavior at runtime
|
|
add_cxx_compile_options(-Wdelete-incomplete)
|
|
# Warn if a requested optimization pass is disabled. Code is too big or too complex
|
|
add_cxx_compile_options(-Wdisabled-optimization)
|
|
# Warn about duplicated conditions in an if-else-if chain
|
|
add_cxx_compile_options(-Wduplicated-cond)
|
|
# Warn about a comparison between values of different enumerated types
|
|
add_cxx_compile_options(-Wenum-compare)
|
|
# Warn about uninitialized variables that are initialized with themselves
|
|
add_cxx_compile_options(-Winit-self)
|
|
# Warn about logical not used on the left hand side operand of a comparison
|
|
add_cxx_compile_options(-Wlogical-not-parentheses)
|
|
# Warn about suspicious uses of logical operators in expressions
|
|
add_cxx_compile_options(-Wlogical-op)
|
|
# Warn if there exists a path from the function entry to a use of the variable that is uninitialized.
|
|
add_cxx_compile_options(-Wmaybe-uninitialized)
|
|
# Warn when the indentation of the code does not reflect the block structure
|
|
add_cxx_compile_options(-Wmisleading-indentation)
|
|
# Warn if a global function is defined without a previous declaration - disabled because of build times
|
|
# add_cxx_compile_options(-Wmissing-declarations)
|
|
# Warn if a user-supplied include directory does not exist
|
|
add_cxx_compile_options(-Wmissing-include-dirs)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wnon-virtual-dtor)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wno-return-local-addr)
|
|
# This warning is disabled due to false positives if compiled with libc++: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90037
|
|
#add_cxx_compile_options(-Wnull-dereference)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wodr)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wold-style-cast)
|
|
# Warn when a function declaration hides virtual functions from a base class
|
|
# add_cxx_compile_options(-Woverloaded-virtual)
|
|
# Warn about placement new expressions with undefined behavior
|
|
add_cxx_compile_options(-Wplacement-new=2)
|
|
# Warn about anything that depends on the “size of” a function type or of void
|
|
add_cxx_compile_options(-Wpointer-arith)
|
|
# Warn if anything is declared more than once in the same scope
|
|
add_cxx_compile_options(-Wredundant-decls)
|
|
# Member initialization reordering
|
|
add_cxx_compile_options(-Wreorder)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wshadow)
|
|
# Warn if left shifting a negative value
|
|
add_cxx_compile_options(-Wshift-negative-value)
|
|
# Warn about a definition of an unsized deallocation function
|
|
add_cxx_compile_options(-Wsized-deallocation)
|
|
# Warn when the sizeof operator is applied to a parameter that is declared as an array in a function definition
|
|
add_cxx_compile_options(-Wsizeof-array-argument)
|
|
# Warn for suspicious length parameters to certain string and memory built-in functions if the argument uses sizeof
|
|
add_cxx_compile_options(-Wsizeof-pointer-memaccess)
|
|
|
|
if (CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 9)
|
|
# Warn about overriding virtual functions that are not marked with the override keyword
|
|
add_cxx_compile_options(-Wsuggest-override)
|
|
endif ()
|
|
|
|
# Warn whenever a switch statement has an index of boolean type and the case values are outside the range of a boolean type
|
|
add_cxx_compile_options(-Wswitch-bool)
|
|
# Warn if a self-comparison always evaluates to true or false
|
|
add_cxx_compile_options(-Wtautological-compare)
|
|
# Warn about trampolines generated for pointers to nested functions
|
|
add_cxx_compile_options(-Wtrampolines)
|
|
# Obvious
|
|
add_cxx_compile_options(-Wunused)
|
|
# Warn if vector operation is not implemented via SIMD capabilities of the architecture
|
|
add_cxx_compile_options(-Wvector-operation-performance)
|
|
|
|
# XXX: gcc10 stuck with this option while compiling GatherUtils code
|
|
# (anyway there are builds with clang, that will warn)
|
|
if (CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 10)
|
|
add_cxx_compile_options(-Wno-sequence-point)
|
|
endif()
|
|
endif ()
|