From 47c823e5be1eba690cb5b2c94962b8c4ac02cae4 Mon Sep 17 00:00:00 2001 From: John Snow Date: Wed, 11 Sep 2019 21:44:42 -0400 Subject: [PATCH] tests/docker: add sanitizers back to clang build MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Fedora23 is but a distant twinkle. The sanitizer works again, and even if not, we have --enable-sanitizers now. Signed-off-by: John Snow Message-Id: <20190912014442.5757-1-jsnow@redhat.com> Signed-off-by: Alex Bennée --- tests/docker/test-clang | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tests/docker/test-clang b/tests/docker/test-clang index 324e341cea..db9e6970b7 100755 --- a/tests/docker/test-clang +++ b/tests/docker/test-clang @@ -17,11 +17,7 @@ requires clang cd "$BUILD_DIR" -OPTS="--cxx=clang++ --cc=clang --host-cc=clang" -# -fsanitize=undefined is broken on Fedora 23, skip it for now -# See also: https://bugzilla.redhat.com/show_bug.cgi?id=1263834 -#OPTS="$OPTS --extra-cflags=-fsanitize=undefined \ - #--extra-cflags=-fno-sanitize=float-divide-by-zero" +OPTS="--cxx=clang++ --cc=clang --host-cc=clang --enable-sanitizers" build_qemu $OPTS check_qemu install_qemu -- 2.39.5