Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124890
Precedence: bulk
MIME-Version: 1.0
References: <CAPyj-LB11wiZyHV8=7fEs=J==tDO+m+z4qRsXRroPnqXzNZvvg@mail.gmail.com>
 <9e1068ae-593e-40bc-91d7-dd63545d9e60@gmx.de>
In-Reply-To: <9e1068ae-593e-40bc-91d7-dd63545d9e60@gmx.de>
Date: Mon, 12 Aug 2024 09:56:15 -0600
Message-ID: <CAAKU0ukMEAdCYQmti=aqM7QyjtnuzOx6fCd0aWGGf6EN3CRZBw@mail.gmail.com>
Subject: Re: [PHP-DEV] Require C11 in PHP 8.4
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: Ilija Tovilo <tovilo.ilija@gmail.com>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: levi.morrison@datadoghq.com (Levi Morrison)

On Sat, Aug 10, 2024 at 6:22=E2=80=AFAM Christoph M. Becker <cmbecker69@gmx=
.de> wrote:
>
> On 01.08.2024 at 23:57, Ilija Tovilo wrote:
>
> > Hence, it seems like it would be ok to bump our C compiler requirement
> > to C11. We'd like to make this change before beta 1 if there are no
> > objections. There are no immediate plans to make non-optional use of
> > other C11 features, although that is conceivable at some point.
>
> So skimming the whole discussion[1] it seems that most are generally
> fine with bumping the requirements to C11, except for Giovanni Giacobbi
> (whose draft PR[2] had no further discussion so far), and maybe for some
> uncertainties regarding some less used compilers.
>
> Given that feature freeze is in 3 days[3], it appears to be prudent to
> have a final decision now, and to close PR #15079[4] if we're going with
> C11, or merge that or Giovanni's PR[2], otherwise.
>
> And if we're going with C11, figuring out the details (which
> configuration check to use, or only documenting the requirement) can
> still be decided somehwat later, in my opinion.
>
> [1] <https://externals.io/message/124706>
> [2] <https://github.com/php/php-src/pull/15202>
> [3] <https://wiki.php.net/todo/php84#timetable>
> [4] <https://github.com/php/php-src/pull/15079>
>
> Cheers,
> Christoph

Given the timetable, I wouldn't change the C std requirements for 8.4.
I would stop relying on the typedef and forward declare only the
struct, and that. Note that although Windows supports C11, it does not
support all features including atomics. Someone chimed in to say that
they are available, but this doesn't match the information I got from
a coworker who did a similar test. Given conflicting information and
the short timetable, I think we should lean towards being cautious. I
hope for 8.5/9.0 we can move to C11/C17 which can improve the typedef
situation, simplify our atomics handling, and more.