Skip to content

Commit

Permalink
ci: remove msan build
Browse files Browse the repository at this point in the history
It doesn't work correctly. All componentes have to be instrumented for
msan to function properly. Changing linking mode to C++ exposes more
issues, valid as the memory comes from uninstrumented driver for
example.
  • Loading branch information
kasper93 committed Sep 4, 2023
1 parent 7d1bc36 commit 84d3962
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 22 deletions.
21 changes: 0 additions & 21 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -178,24 +178,3 @@ sanitize:
-Dshaderc=enabled
- ninja -C build
- cd build && time meson test -t 5 -v --num-processes=1

msan:
image: $IMAGE_UBUNTU_JAMMY
stage: sanitize
tags:
- gpu
variables:
MSAN_OPTIONS: 'exitcode=1'
script:
- env CC=clang CXX=clang++ CC_LD=lld CXX_LD=lld
meson build --buildtype debugoptimized
--werror
--default-library static
-Dtests=true
-Ddebug-abort=true
-Dc_args='-DMSAN -DCI_MAXGL -Wno-deprecated-declarations'
-Dglslang=disabled
-Db_sanitize=memory
-Db_lundef=false
- ninja -C build
- cd build && time meson test -t 5 -v --num-processes=1
1 change: 0 additions & 1 deletion src/convert.cc
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
*/

#include <charconv>
#include <cmath>
#include <limits>
#include <system_error>

Expand Down

0 comments on commit 84d3962

Please sign in to comment.