Skip to content

Commit

Permalink
Re-enable GCC warning.
Browse files Browse the repository at this point in the history
  • Loading branch information
mjp41 committed Apr 14, 2020
1 parent d4fccfa commit 47547c4
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,6 @@ macro(warnings_high)
add_compile_options(-Wsign-conversion)
endif ()
add_compile_options(-Wall -Wextra -Werror -Wundef)
# There are a few places with subtle reasons for array access being correct
# GCC's warnings are too aggressive and incorrectly assume the code is wrong.
# Disabling only in Release is so the ASSUME can be mapped to assert and check
# at runtime in debug. This ensures we are covering the cases of concern with
# debug checks, but not incurring runtime penalties in release.
if ((CMAKE_CXX_COMPILER_ID STREQUAL "GNU") AND (CMAKE_BUILD_TYPE STREQUAL "Release"))
add_compile_options(-Wno-array-bounds)
endif ()
endif()
endmacro()

Expand Down

0 comments on commit 47547c4

Please sign in to comment.