Re: New compiler warnings in buildfarm

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Andres Freund <andres(at)anarazel(dot)de>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: New compiler warnings in buildfarm
Date: 2024-08-02 17:22:02
Message-ID: 78f5e12b-e016-4699-b8ea-9a04fa4511be@eisentraut.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 31.07.24 20:39, Andres Freund wrote:
> On 2024-07-31 11:32:56 -0700, Andres Freund wrote:
>> On 2024-07-31 10:11:07 -0400, Tom Lane wrote:
>>> It looks like serinus needs this fix too.
>>
>> Added to both. I've forced runs for both animals, so the bf should show
>> results of that soon.
>
> I Wonder if I should also should add -Wno-clobbered to serinus' config. Afaict
> -Wclobbered is pretty useless once optimizations are used. I've long added
> that to my local dev environment flags because it's so noisy (which is too
> bad, in theory a good warning for this would be quite helpful).

It's unclear to me what to make of this. We have in the past fixed a
number of these, IIRC, and clearly in theory the risk that the warning
points out does exist. But these warnings appear erratic and
inconsistent. I'm using the same compiler versions but I don't see any
of these warnings. So I don't understand exactly what triggers these.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2024-08-02 17:39:50 Re: can we mark upper/lower/textlike functions leakproof?
Previous Message Peter Eisentraut 2024-08-02 17:13:40 Re: [PoC] Federated Authn/z with OAUTHBEARER