Skip to content

Commit b516c92

Browse files
committed
fix: sync compiler flags with instrument-hooks
1 parent 86049c6 commit b516c92

File tree

2 files changed

+12
-10
lines changed

2 files changed

+12
-10
lines changed

core/BUILD

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,14 @@ cc_library(
2424
"/wd4146", # unary minus operator applied to unsigned type
2525
],
2626
"//conditions:default": [
27-
"-Wno-maybe-uninitialized",
28-
"-Wno-unused-variable",
29-
"-Wno-unused-parameter",
27+
# IMPORTANT: Keep in sync with instrument-hooks/ci.yml (COMMON_CFLAGS)
28+
"-Wall",
29+
"-Werror",
30+
"-Wno-format-security",
3031
"-Wno-unused-but-set-variable",
32+
"-Wno-unused-const-variable",
3133
"-Wno-type-limits",
32-
"-Wno-format",
33-
"-Wno-format-security",
34+
"-Wno-uninitialized",
3435
],
3536
}),
3637
visibility = ["//visibility:public"],

core/CMakeLists.txt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,16 +40,17 @@ target_include_directories(
4040

4141
# Suppress warnings for the instrument_hooks library
4242
if(CMAKE_CXX_COMPILER_ID MATCHES "GNU|Clang")
43+
# IMPORTANT: Keep in sync with instrument-hooks/ci.yml (COMMON_CFLAGS)
4344
target_compile_options(
4445
instrument_hooks
4546
PRIVATE
46-
-Wno-maybe-uninitialized
47-
-Wno-unused-variable
48-
-Wno-unused-parameter
47+
-Wall
48+
-Werror
49+
-Wno-format-security
4950
-Wno-unused-but-set-variable
51+
-Wno-unused-const-variable
5052
-Wno-type-limits
51-
-Wno-format
52-
-Wno-format-security
53+
-Wno-uninitialized
5354
)
5455
elseif(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC")
5556
target_compile_options(

0 commit comments

Comments
 (0)