Re: C11 / VS 2019

From: Peter Eisentraut <peter(at)eisentraut(dot)org>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andrew Dunstan <andrew(at)dunslane(dot)net>, Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: C11 / VS 2019
Date: 2025-07-07 10:51:40
Message-ID: 46154df9-f514-4d79-89bb-df4af46b1b9f@eisentraut.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 06.07.25 00:27, Tom Lane wrote:
> Peter Eisentraut <peter(at)eisentraut(dot)org> writes:
>> On 04.06.25 08:15, Peter Eisentraut wrote:
>>> For an explanation, the background is that MSVC has a "traditional"
>>> preprocessor and a new "conforming" one.  The latter is available
>>> starting in VS 2019, but it's not the default.  We have some code,
>>> especially around __VA_ARGS__ that specifically caters to this
>>> traditional preprocessor.
>
>> I have committed this.
>
> Buildfarm member drongo has been failing in initdb since 1 July:
>
> selecting default time zone ... UTC
> creating configuration files ... ok
> running bootstrap script ...
> ----------------------------------- stderr -----------------------------------
> TRAP: failed Assert("SysCache[cacheId]->cc_nkeys == 2"), File: "../pgsql/src/backend/utils/cache/syscache.c", Line: 237, PID: 2684
> child process was terminated by exception 0xC0000409
>
> While there are 19 new commits in the first run that shows this
> failure [1], the only one that looks plausibly related is
>
> 8fd9bb1d965 Tue Jul 1 07:41:40 2025 UTC Enable MSVC conforming preprocessor
>
> because that changed our implementation of VA_ARGS_NARGS(), which is
> what's used to compute the cc_nkeys fields for syscaches.
>
> My conclusion is that Microsoft's "standards conforming" preprocessor
> is not so standards conforming as all that.

Hmm. We have the (allegedly) same VS version in Cirrus CI, and it's
never shown a problem like this.

I wonder if there could be a local problem on this host. Perhaps the
ccache or pch cache could use a cleaning up?

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Arseniy Mukhin 2025-07-07 11:06:24 Re: amcheck support for BRIN indexes
Previous Message Chiranmoy.Bhattacharya@fujitsu.com 2025-07-07 10:41:52 Re: [PATCH] Hex-coding optimizations using SVE on ARM.