Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:112638
MIME-Version: 1.0
Date: Tue, 29 Dec 2020 17:57:58 +0100
Message-ID: <CAH5C8xVr9zVyyzKAVM=D56-oMVGf9Nsxqa4rq9k1xM+gsH2SPg@mail.gmail.com>
To: PHP Internals List <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000996d2f05b79d4b2a"
Subject: [RFC] Bundling ext/simdjson into core
From: kocsismate90@gmail.com (=?UTF-8?B?TcOhdMOpIEtvY3Npcw==?=)

--000000000000996d2f05b79d4b2a
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Internals,

I think this will be my last proposal for quite some while :)
But this time, I'd like to propose bundling the
https://github.com/crazyxman/simdjson_php extension
with some major modifications.

The proposed OO API is included in the description of the
PR that I've just created: https://github.com/php/php-src/pull/6551

The main motivation behind this RFC is two-fold:
- the underlying simdjson library (https://github.com/simdjson/simdjson)
which is used by ext/simdjson provides huge performance gains
compared to ext/json (see some benchmark results in the PR)
- we can support new use-cases, most notably the so called "on-demand"
parsing: https://github.com/simdjson/simdjson/blob/master/doc/ondemand.md
(This is not implemented currently)

Originally, I planned to include the new API in ext/json, but unfortunately=
,
simdjson is written is C++, so it would make C++ as a hard dependency,
which was not the case so far. That's why I opted for creating ext/simdjson=
.

Please let me know if you have any feedback.

Regards:
M=C3=A1t=C3=A9

--000000000000996d2f05b79d4b2a--