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
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 |