Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:118200
To: Marco Pivetta <ocramius@gmail.com>
Cc: Larry Garfield <larry@garfieldtech.com>, PHP Internals List <internals@lists.php.net>
Date: Tue, 05 Jul 2022 18:33:12 +0200
Message-ID: <4489397.ejJDZkT8p0@arnaud-t490>
In-Reply-To: <CADyq6sKuuuMMSHZvKDNe0soht5rmN98WNS6Kb=DMVne8Khd5Cw@mail.gmail.com>
References: <e506a378-2060-4cc1-98a6-3c9c7c0bd903@www.fastmail.com> <6128711.lOV4Wx5bFT@arnaud-t490> <CADyq6sKuuuMMSHZvKDNe0soht5rmN98WNS6Kb=DMVne8Khd5Cw@mail.gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] [RFC][Vote] Auto-capture closures
From: arnaud.lb@gmail.com (Arnaud Le Blanc)

On mardi 5 juillet 2022 09:27:34 CEST Marco Pivetta wrote:
> This is fine, but it creates a bidirectional dependency between components
> where the dependency graph was previously (in theory) acyclic.
>=20
> This will become "someone else's problem" =F0=9F=98=AC

I don't think that the Optimizer conceptually depends on the Compiler=20
currently. It does include `Zend/zend_compile.h`, but that's only for the=20
types defined in this header. It needs these types because its responsibili=
ty=20
is to transform data structures produced by the compiler (op_arrays), or in=
=20
this case, to produce meta data about these structures. These types can be=
=20
trivially moved to an other header. It doesn't call any compiler code. Also=
,=20
this doesn't introduces circular includes involving Zend/zend_compile.h. Do=
es=20
it seems ok to you ?