Re: git: 526f57bf85a2 - main - Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15.
- Reply: Warner Losh : "Re: git: 526f57bf85a2 - main - Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15."
- Reply: Dimitry Andric : "Re: git: 526f57bf85a2 - main - Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15."
- In reply to: Dimitry Andric : "git: 526f57bf85a2 - main - Update CWARNFLAGS in bsd.sys.mk and CWARNEXTRA in kern.mk for clang 15."
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 27 Dec 2022 17:40:41 UTC
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