Re: gcc build warnings at -O3

From: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Alexander Korotkov <akorotkov(at)postgresql(dot)org>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: gcc build warnings at -O3
Date: 2024-02-08 20:00:56
Message-ID: CAPpHfdvGGNSXae6V_PUFqpoF=sT7oE=RJu6jeeT_HvKXgtJkxQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

On Wed, Feb 7, 2024 at 10:31 PM Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2024-01-11 21:55:19 -0500, Tom Lane wrote:
> > Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> writes:
> > > Hi, I'm seeing a compiler warning with CFLAGS -O3 but not with -O2.
> >
> > > In file included from dbcommands.c:20:
> > > dbcommands.c: In function ‘createdb’:
> > > ../../../src/include/postgres.h:104:16: warning: ‘src_hasloginevt’ may
> > > be used uninitialized in this function [-Wmaybe-uninitialized]
> >
> > Hmm, I also see that at -O3 (not at -O2) when using Fedora 39's
> > gcc 13.2.1, but *not* when using RHEL8's gcc 8.5.0.
>
> It's visible here with gcc >= 10. That's enough versions that I think we
> should care. Interestingly enough, it seems to have recently have gotten
> fixed in gcc master (14 to be).

I managed to reproduce this warning locally. Fixed.

------
Regards,
Alexander Korotkov

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2024-02-08 20:23:26 pgsql: Apply pg_dump test cleanups to test_pg_dump as well
Previous Message Alexander Korotkov 2024-02-08 20:00:22 pgsql: Fix gcc >= 10 warning

Browse pgsql-hackers by date

  From Date Subject
Next Message Nathan Bossart 2024-02-08 20:07:37 Re: glibc qsort() vulnerability
Previous Message Andres Freund 2024-02-08 19:59:54 Re: glibc qsort() vulnerability