Re: Warning in geqo_main.c from clang 13

2022-01-23 Thread Tom Lane
Thomas Munro writes: > On Sun, Jan 23, 2022 at 11:34 AM Tom Lane wrote: >> We're starting to see more buildfarm animals producing this warning, >> so I took another look, and thought of a slightly less invasive way to >> silence it. I confirmed this works with clang 13.0.0 on Fedora 35. > LGTM.

Re: Warning in geqo_main.c from clang 13

2022-01-22 Thread Thomas Munro
On Sun, Jan 23, 2022 at 11:34 AM Tom Lane wrote: > We're starting to see more buildfarm animals producing this warning, > so I took another look, and thought of a slightly less invasive way to > silence it. I confirmed this works with clang 13.0.0 on Fedora 35. LGTM. Tested on bleeding edge cla

Re: Warning in geqo_main.c from clang 13

2022-01-22 Thread Tom Lane
I wrote: > Thomas Munro writes: >> Clang 13 on my machine and peripatus (but not Apple clang 13 on eg >> sifika, I'm still confused about Apple's versioning but I think that's >> really llvm 12-based) warns: >> geqo_main.c:86:8: warning: variable 'edge_failures' set but not used >> [-Wunused-but-s

Re: Warning in geqo_main.c from clang 13

2021-11-23 Thread Tom Lane
Thomas Munro writes: > Clang 13 on my machine and peripatus (but not Apple clang 13 on eg > sifika, I'm still confused about Apple's versioning but I think that's > really llvm 12-based) warns: > geqo_main.c:86:8: warning: variable 'edge_failures' set but not used > [-Wunused-but-set-variable] >

Warning in geqo_main.c from clang 13

2021-11-23 Thread Thomas Munro
Hi, Clang 13 on my machine and peripatus (but not Apple clang 13 on eg sifika, I'm still confused about Apple's versioning but I think that's really llvm 12-based) warns: geqo_main.c:86:8: warning: variable 'edge_failures' set but not used [-Wunused-but-set-variable] int