d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Enable -Warray-parameter for clang.

Browse Source 
I fixed many of these previously for GCC 12 and make tinderbox passes
with this enabled.

Differential Revision:	https://reviews.freebsd.org/D39378
This commit is contained in:
John Baldwin 2023-04-11 13:47:59 -07:00
parent 61330e494f
commit cd800d3c96
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
share/mk/bsd.sys.mk
View File

@ -95,7 +95,6 @@ CWARNFLAGS.clang+= -Wno-unused-const-variable
CWARNFLAGS.clang+= -Wno-error=unused-but-set-variable
.endif
.if ${COMPILER_TYPE} == "clang" && ${COMPILER_VERSION} >= 150000
CWARNFLAGS.clang+= -Wno-error=array-parameter
CWARNFLAGS.clang+= -Wno-error=deprecated-non-prototype
CWARNFLAGS.clang+= -Wno-error=unused-but-set-parameter
.endif
@ -127,6 +126,7 @@ CWARNFLAGS+= -Wno-misleading-indentation
NO_WBITWISE_INSTEAD_OF_LOGICAL= -Wno-bitwise-instead-of-logical
.endif
.if ${COMPILER_TYPE} == "clang" && ${COMPILER_VERSION} >= 150000
NO_WARRAY_PARAMETER= -Wno-array-parameter
NO_WSTRICT_PROTOTYPES= -Wno-strict-prototypes
NO_WDEPRECATED_NON_PROTOTYPE=-Wno-deprecated-non-prototype
.endif

7
sys/conf/kern.mk
View File

@ -47,10 +47,9 @@ CWARNEXTRA?= -Wno-error=tautological-compare -Wno-error=empty-body \
CWARNEXTRA+= -Wno-error=shift-negative-value
CWARNEXTRA+= -Wno-address-of-packed-member
.if ${COMPILER_VERSION} >= 150000
# Clang 15 has much more aggressive diagnostics about inconsistently declared
# array parameters, K&R prototypes, mismatched prototypes, and unused-but-set
# variables. Make these non-fatal for the time being.
CWARNEXTRA+= -Wno-error=array-parameter
# Clang 15 has much more aggressive diagnostics about K&R prototypes,
# mismatched prototypes, and unused-but-set variables. Make these
# non-fatal for the time being.
CWARNEXTRA+= -Wno-error=deprecated-non-prototype
CWARNEXTRA+= -Wno-error=strict-prototypes
CWARNEXTRA+= -Wno-error=unused-but-set-variable