From cc32487a94d0e65482fb2ac5377666d0652b850c Mon Sep 17 00:00:00 2001 From: elsid Date: Fri, 25 Apr 2025 20:14:28 +0200 Subject: [PATCH] Disable -Werror for ASAN builds To avoid warnings like: In file included from /usr/include/c++/13/regex:50, from ../../components/lua/yamlloader.cpp:6: In constructor 'std::function<_Res(_ArgTypes ...)>::function(std::function<_Res(_ArgTypes ...)>&&) [with _Res = bool; _ArgTypes = {char}]', inlined from 'std::__detail::_State<_Char_type>::_State(std::__detail::_State<_Char_type>&&) [with _Char_type = char]' at /usr/include/c++/13/bits/regex_automaton.h:149:4, inlined from 'std::__detail::_StateIdT std::__detail::_NFA<_TraitsT>::_M_insert_subexpr_begin() [with _TraitsT = std::__cxx11::regex_traits]' at /usr/include/c++/13/bits/regex_automaton.h:281:24: /usr/include/c++/13/bits/std_function.h:405:42: error: '*(std::function*)((char*)&__tmp + offsetof(std::__detail::_StateT, std::__detail::_State::.std::__detail::_State_base::)).std::function::_M_invoker' may be used uninitialized [-Werror=maybe-uninitialized] 405 | : _Function_base(), _M_invoker(__x._M_invoker) | ~~~~^~~~~~~~~~ In file included from /usr/include/c++/13/regex:65: /usr/include/c++/13/bits/regex_automaton.h: In member function 'std::__detail::_StateIdT std::__detail::_NFA<_TraitsT>::_M_insert_subexpr_begin() [with _TraitsT = std::__cxx11::regex_traits]': /usr/include/c++/13/bits/regex_automaton.h:279:17: note: '__tmp' declared here 279 | _StateT __tmp(_S_opcode_subexpr_begin); | ^~~~~ See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105562. --- .gitlab-ci.yml | 5 +++++ CI/before_script.linux.sh | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index a79fd0c571..ee3f70695e 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -56,6 +56,7 @@ Ubuntu_GCC_preprocess: stage: build variables: CMAKE_EXE_LINKER_FLAGS: -fuse-ld=mold + OPENMW_CXX_FLAGS: "-Werror -Werror=implicit-fallthrough" script: - df -h - export CCACHE_BASEDIR="`pwd`" @@ -156,6 +157,8 @@ Ubuntu_GCC_asan: CMAKE_CXX_FLAGS_DEBUG: -g -O1 -fno-omit-frame-pointer -fsanitize=address -fsanitize=pointer-subtract -fsanitize=leak CMAKE_EXE_LINKER_FLAGS: -fsanitize=address -fsanitize=pointer-subtract -fsanitize=leak -fuse-ld=mold BUILD_OPENMW_ONLY: 1 + # Disable -Werror due to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105562 + OPENMW_CXX_FLAGS: "" Clang_Format: extends: .Ubuntu_Image @@ -257,6 +260,8 @@ Ubuntu_GCC_tests_asan: CMAKE_CXX_FLAGS_DEBUG: -g -O1 -fno-omit-frame-pointer -fsanitize=address -fsanitize=pointer-subtract -fsanitize=leak CMAKE_EXE_LINKER_FLAGS: -fsanitize=address -fsanitize=pointer-subtract -fsanitize=leak -fuse-ld=mold ASAN_OPTIONS: halt_on_error=1:strict_string_checks=1:detect_stack_use_after_return=1:check_initialization_order=1:strict_init_order=1 + # Disable -Werror due to https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105562 + OPENMW_CXX_FLAGS: "" artifacts: paths: [] name: ${CI_JOB_NAME}-${CI_COMMIT_REF_NAME}-${CI_COMMIT_SHA} diff --git a/CI/before_script.linux.sh b/CI/before_script.linux.sh index c6fd306e25..c5704b900c 100755 --- a/CI/before_script.linux.sh +++ b/CI/before_script.linux.sh @@ -17,7 +17,7 @@ declare -a CMAKE_CONF_OPTS=( -DBUILD_SHARED_LIBS="${BUILD_SHARED_LIBS:-OFF}" -DUSE_SYSTEM_TINYXML=ON -DOPENMW_USE_SYSTEM_RECASTNAVIGATION=ON - -DOPENMW_CXX_FLAGS="-Werror -Werror=implicit-fallthrough" # flags specific to OpenMW project + -DOPENMW_CXX_FLAGS="${OPENMW_CXX_FLAGS}" # flags specific to OpenMW project ) if [[ "${CMAKE_EXE_LINKER_FLAGS}" ]]; then