Re: Clang function pointer type warnings in v14, v15 - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: Clang function pointer type warnings in v14, v15
Date
Msg-id CA+hUKGLsD9Zc4aCvxp+L-MKPLvPtr=_R60cXAJt1bbxr0nJruQ@mail.gmail.com
Whole thread Raw
In response to Re: Clang function pointer type warnings in v14, v15  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Clang function pointer type warnings in v14, v15
List pgsql-hackers
On Fri, Jul 12, 2024 at 2:26 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Thomas Munro <thomas.munro@gmail.com> writes:
> > On Sat, Jul 6, 2024 at 2:35 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> >> I see that there are a boatload of related warnings in older
> >> branches too; do we want to try to do anything about that?  (I doubt
> >> code changes would be in-scope, but maybe adding a -Wno-foo switch
> >> to the build flags would be appropriate.)
>
> > I don't see any on clang 16 in the 12 and 13 branches.  Where are you
> > seeing them?
>
> In the buildfarm.  "adder" and a bunch of other machines are throwing
> -Wcast-function-type in about two dozen places in v13, and "jay" is
> emitting several hundred -Wdeprecated-non-prototype warnings.

Ah, I see.  A few animals running with -Wextra.  Whereas in v14+ we
have -Wcast-function-type in the actual tree, which affects people's
workflows more directly.  Like my regular machine, or CI, when a
couple of the OSes' house compilers eventually reach clang 16.

I gess we have to decide if it's a matter for the tree, or for the
people who add -Wextra, ie to decide if they want to filter that down
a bit with some -Wno-XXX.  Adder already has some of those:

                                     'CFLAGS' => '-O1 -ggdb -g3
-fno-omit-frame-pointer -Wall -Wextra -Wno-unused-parameter
-Wno-sign-compare -Wno-missing-field-initializers -O0',



pgsql-hackers by date:

Previous
From: Tatsuo Ishii
Date:
Subject: Re: Optimize WindowAgg's use of tuplestores
Next
From: Tom Lane
Date:
Subject: Re: Clang function pointer type warnings in v14, v15