Skip to content

Commit 75a9ad0

Browse files
committed
Enable more warnings
1 parent 2467cc7 commit 75a9ad0

File tree

2 files changed

+19
-8
lines changed

2 files changed

+19
-8
lines changed

.github/workflows/main.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -559,7 +559,9 @@ jobs:
559559
cmake -S . -B build
560560
-D CMAKE_C_COMPILER_LAUNCHER=ccache -D CMAKE_CXX_COMPILER_LAUNCHER=ccache
561561
-G Ninja -D USE_SEQ=ON -D USE_MPI=ON -D USE_OMP=ON -D USE_TBB=ON -D USE_STL=ON
562-
-DCMAKE_C_FLAGS="-I$(brew --prefix)/opt/libomp/include" -DCMAKE_CXX_FLAGS="-I$(brew --prefix)/opt/libomp/include"
562+
-DCMAKE_C_FLAGS="-Xclang -fopenmp -I$(brew --prefix)/opt/libomp/include"
563+
-DCMAKE_CXX_FLAGS="-Xclang -fopenmp -I$(brew --prefix)/opt/libomp/include"
564+
-DCMAKE_EXE_LINKER_FLAGS="-L$(brew --prefix libomp)/lib -lomp"
563565
-D CMAKE_BUILD_TYPE=RELEASE -D USE_FUNC_TESTS=ON -D USE_PERF_TESTS=ON
564566
-DCMAKE_INSTALL_PREFIX=install
565567
- name: Build project

cmake/configure.cmake

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,16 +21,25 @@ set( CMAKE_RUNTIME_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}/bin" )
2121
set( CMAKE_CXX_STANDARD 20 )
2222

2323
if( UNIX )
24-
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} \
25-
-Wall -Wextra \
26-
-Wno-unused-parameter \
27-
-Wno-cast-function-type \
28-
-Wsign-compare -Werror")
29-
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} \
30-
-Wall -Wextra \
24+
set(COMMON_COMPILER_FLAGS
25+
"-Wall -Wextra \
26+
-Wpedantic \
27+
-Wpointer-arith \
28+
-Wcast-align \
29+
-Wwrite-strings \
30+
-Wswitch-enum \
3131
-Wno-unused-parameter \
3232
-Wno-cast-function-type \
33+
-Wnull-dereference \
34+
-Wold-style-definition \
35+
-Wswitch-enum \
36+
-Wformat=2 \
37+
-Wmissing-prototypes \
38+
-Wmissing-declarations \
39+
-Wno-c11-extensions \
3340
-Wsign-compare -Werror")
41+
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${COMMON_COMPILER_FLAGS}")
42+
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${COMMON_COMPILER_FLAGS}")
3443
if (USE_COVERAGE)
3544
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} --coverage")
3645
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} --coverage")

0 commit comments

Comments
 (0)