Skip to content

Commit f4f2a88

Browse files
committed
cmake : enable warnings in llama
ggml-ci
1 parent 9336db4 commit f4f2a88

File tree

3 files changed

+65
-0
lines changed

3 files changed

+65
-0
lines changed

common/CMakeLists.txt

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,27 @@
22

33
find_package(Threads REQUIRED)
44

5+
if (LLAMA_ALL_WARNINGS)
6+
if (NOT MSVC)
7+
list(APPEND WARNING_FLAGS -Wall -Wextra -Wpedantic -Wcast-qual -Wno-unused-function)
8+
list(APPEND C_FLAGS -Wshadow -Wstrict-prototypes -Wpointer-arith -Wmissing-prototypes
9+
-Werror=implicit-int -Werror=implicit-function-declaration)
10+
list(APPEND CXX_FLAGS -Wmissing-declarations -Wmissing-noreturn)
11+
12+
list(APPEND C_FLAGS ${WARNING_FLAGS})
13+
list(APPEND CXX_FLAGS ${WARNING_FLAGS})
14+
15+
get_flags(${CMAKE_CXX_COMPILER_ID} ${CMAKE_CXX_COMPILER_VERSION})
16+
17+
add_compile_options("$<$<COMPILE_LANGUAGE:C>:${C_FLAGS};${GF_C_FLAGS}>"
18+
"$<$<COMPILE_LANGUAGE:CXX>:${CXX_FLAGS};${GF_CXX_FLAGS}>")
19+
else()
20+
# todo : msvc
21+
set(C_FLAGS "")
22+
set(CXX_FLAGS "")
23+
endif()
24+
endif()
25+
526
# Build info header
627
#
728

examples/CMakeLists.txt

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,29 @@ find_package(Threads REQUIRED)
66

77
# ...
88

9+
# flags
10+
11+
if (LLAMA_ALL_WARNINGS)
12+
if (NOT MSVC)
13+
list(APPEND WARNING_FLAGS -Wall -Wextra -Wpedantic -Wcast-qual -Wno-unused-function)
14+
list(APPEND C_FLAGS -Wshadow -Wstrict-prototypes -Wpointer-arith -Wmissing-prototypes
15+
-Werror=implicit-int -Werror=implicit-function-declaration)
16+
list(APPEND CXX_FLAGS -Wmissing-declarations -Wmissing-noreturn)
17+
18+
list(APPEND C_FLAGS ${WARNING_FLAGS})
19+
list(APPEND CXX_FLAGS ${WARNING_FLAGS})
20+
21+
get_flags(${CMAKE_CXX_COMPILER_ID} ${CMAKE_CXX_COMPILER_VERSION})
22+
23+
add_compile_options("$<$<COMPILE_LANGUAGE:C>:${C_FLAGS};${GF_C_FLAGS}>"
24+
"$<$<COMPILE_LANGUAGE:CXX>:${CXX_FLAGS};${GF_CXX_FLAGS}>")
25+
else()
26+
# todo : msvc
27+
set(C_FLAGS "")
28+
set(CXX_FLAGS "")
29+
endif()
30+
endif()
31+
932
# examples
1033

1134
include_directories(${CMAKE_CURRENT_SOURCE_DIR})

src/CMakeLists.txt

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,27 @@ if (WIN32)
55
endif()
66
endif()
77

8+
if (LLAMA_ALL_WARNINGS)
9+
if (NOT MSVC)
10+
list(APPEND WARNING_FLAGS -Wall -Wextra -Wpedantic -Wcast-qual -Wno-unused-function)
11+
list(APPEND C_FLAGS -Wshadow -Wstrict-prototypes -Wpointer-arith -Wmissing-prototypes
12+
-Werror=implicit-int -Werror=implicit-function-declaration)
13+
list(APPEND CXX_FLAGS -Wmissing-declarations -Wmissing-noreturn)
14+
15+
list(APPEND C_FLAGS ${WARNING_FLAGS})
16+
list(APPEND CXX_FLAGS ${WARNING_FLAGS})
17+
18+
get_flags(${CMAKE_CXX_COMPILER_ID} ${CMAKE_CXX_COMPILER_VERSION})
19+
20+
add_compile_options("$<$<COMPILE_LANGUAGE:C>:${C_FLAGS};${GF_C_FLAGS}>"
21+
"$<$<COMPILE_LANGUAGE:CXX>:${CXX_FLAGS};${GF_CXX_FLAGS}>")
22+
else()
23+
# todo : msvc
24+
set(C_FLAGS "")
25+
set(CXX_FLAGS "")
26+
endif()
27+
endif()
28+
829
#
930
# libraries
1031
#

0 commit comments

Comments
 (0)