From c1a7aeaa910aaa8b7d81dcf3680ef54f8bf34e81 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ole=20Sch=C3=BCtt?= Date: Sat, 29 Nov 2025 12:25:23 +0100 Subject: [PATCH] CMake: Add -Wall;-Wextra;-Werror to CFLAGS --- cmake/CompilerConfiguration.cmake | 8 +------- src/dbm/dbm_multiply.c | 2 ++ 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/cmake/CompilerConfiguration.cmake b/cmake/CompilerConfiguration.cmake index 2492e820a0..6507f46dc3 100644 --- a/cmake/CompilerConfiguration.cmake +++ b/cmake/CompilerConfiguration.cmake @@ -69,7 +69,7 @@ add_compile_options( add_compile_options( "$<$,$>:-O1;-march=native;-mtune=native>" "$<$,$>:-O1;-march=native;-mtune=native>" - "$<$,$>:-O1;-march=native;-mtune=native>" + "$<$,$>:-O1;-march=native;-mtune=native;-Wall;-Wextra;-Werror>" ) add_compile_options( "$<$,$>:-fsanitize=leak;-Werror=realloc-lhs>" @@ -89,12 +89,6 @@ add_link_options( "$<$,$>:-fsanitize=leak>" "$<$,$>:-fsanitize=leak>") -# TODO Prevent CMake from wrongly adding -fbacktrace and -# -fallow-argument-mismatch to CFLAGS. add_compile_options( -# "$<$,$>:-Wall;-Wextra;-Werror>" -# "$<$,$>:-Wall;-Wextra;-Werror>" -# ) - # Coverage add_compile_options( "$<$:-coverage;-fkeep-static-functions;-O1;-march=native;-mtune=native>" diff --git a/src/dbm/dbm_multiply.c b/src/dbm/dbm_multiply.c index baa2f2331c..30d37449df 100644 --- a/src/dbm/dbm_multiply.c +++ b/src/dbm/dbm_multiply.c @@ -377,6 +377,7 @@ void dbm_multiply(const bool transa, const bool transb, const double alpha, backend_stop(ctx); if (NULL != matrix_d) { + ctx = backend_start(matrix_d); iter = dbm_comm_iterator_start(transa, transb, matrix_a, matrix_b, matrix_d); while (dbm_comm_iterator_next(iter, &pack_a, &pack_b)) { @@ -384,6 +385,7 @@ void dbm_multiply(const bool transa, const bool transb, const double alpha, matrix_d, rows_max_eps, retain_sparsity, true, NULL, ctx); } dbm_comm_iterator_stop(iter); + backend_stop(ctx); const double epsilon = dbm_maxeps(matrix_d, matrix_c); if (maxeps < epsilon) { if (1 == verify) {