Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 27 Dec 2022 09:40:41 -0800
From:      John Baldwin <jhb@FreeBSD.org>
To:        Dimitry Andric <dim@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: 526f57bf85a2 - main - Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15.
Message-ID:  <5a772eb7-55c8-5e73-e346-5514ef144dc3@FreeBSD.org>
In-Reply-To: <202212251159.2BPBxmAG029954@gitrepo.freebsd.org>
References:  <202212251159.2BPBxmAG029954@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 12/25/22 3:59 AM, Dimitry Andric wrote:
> The branch main has been updated by dim:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=526f57bf85a2b9b113c55ded47b932abc8567bc5
> 
> commit 526f57bf85a2b9b113c55ded47b932abc8567bc5
> Author:     Dimitry Andric <dim@FreeBSD.org>
> AuthorDate: 2022-07-25 10:29:58 +0000
> Commit:     Dimitry Andric <dim@FreeBSD.org>
> CommitDate: 2022-12-25 11:58:22 +0000
> 
>      Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15.
>      
>      MFC after:      3 days
> ---
>   share/mk/bsd.sys.mk | 17 +++++++++++++++--
>   sys/conf/kern.mk    |  4 ++++
>   2 files changed, 19 insertions(+), 2 deletions(-)
> 
> diff --git a/share/mk/bsd.sys.mk b/share/mk/bsd.sys.mk
> index a220c9d173ed..43359fed29c7 100644
> --- a/share/mk/bsd.sys.mk
> +++ b/share/mk/bsd.sys.mk
> @@ -86,6 +94,11 @@ CWARNFLAGS.clang+=	-Wno-unused-const-variable
>   .if ${COMPILER_TYPE} == "clang" && ${COMPILER_VERSION} >= 130000
>   CWARNFLAGS.clang+=	-Wno-error=unused-but-set-variable
>   .endif
> +.if ${COMPILER_TYPE} == "clang" && ${COMPILER_VERSION} >= 150000
> +CWARNFLAGS.clang+=	-Wno-error=array-parameter

I've fixed all these that GCC 12 finds.  Do we really need to disable it for clang
now after I've pushed those recent fixes?

> +CWARNFLAGS.clang+=	-Wno-error=deprecated-non-prototype

I'm curious how many places are broken by this and how much it would be to fix?
This is just adding '(void)' in place of '()', yes?

-- 
John Baldwin




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5a772eb7-55c8-5e73-e346-5514ef144dc3>