Re: Fix compilation warnings when CFLAGS -Og is specified

From: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
To: kuroda(dot)hayato(at)fujitsu(dot)com
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Fix compilation warnings when CFLAGS -Og is specified
Date: 2023-08-01 06:46:55
Message-ID: 20230801.154655.540713355418274655.horikyota.ntt@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

At Tue, 1 Aug 2023 04:51:55 +0000, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com> wrote in
> Dear hackers,
>
> # Background
>
> Based on [1], I did configure and build with options:
> (I used Meson build system, but it could be reproduced by Autoconf/Make)
>
> ```
> $ meson setup -Dcassert=true -Ddebug=true -Dc_args=-Og ../builder
> $ cd ../builder
> $ ninja
> ```
>
> My gcc version is 4.8.5, and ninja is 1.10.2.

gcc 4.8 looks very old?

AFAIS all of those complaints are false positives and if I did this
correclty, gcc 11.3 seems to have been fixed in this regard.

> # Solution
>
> PSA the patch to keep the compiler quiet. IIUC my compiler considered that
> substitutions in PG_TRY() might be skipped. I'm not sure it is real problem,
> but the workarounds are harmless.
>
> Or, did I miss something for ignoring above?
>
> [1]: https://wiki.postgresql.org/wiki/Developer_FAQ#:~:text=or%20MSVC%20tracepoints.-,What%20debugging%20features%20are%20available%3F,-Compile%2Dtime

I think we don't want "fix" those as far as modern compilers don't
emit the false positives.

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Rajendra Kumar Dangwal 2023-08-01 06:47:33 Pgoutput not capturing the generated columns
Previous Message Alvaro Herrera 2023-08-01 06:40:46 Re: Simplify some logical replication worker type checking