Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121374
Message-ID: <526f40d2-4403-4842-bf48-d4b53a4ae5d7@daniil.it>
Date: Wed, 18 Oct 2023 10:42:38 +0200
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Content-Language: en-US
To: Christian Schneider <cschneid@cschneid.com>, internals@lists.php.net
References: <d8e5463d-c827-47e5-aa18-c6116cbd3533@daniil.it>
 <DA7921FA-A5C4-45E2-B3C6-22268B11B35A@cschneid.com>
In-Reply-To: <DA7921FA-A5C4-45E2-B3C6-22268B11B35A@cschneid.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Change default method of disabling JIT for PHP 8.4
From: daniil@daniil.it (Daniil Gentili)

Hi,

This is precisely why I'm planning to submit the RFC to make the change 
in PHP 8.4, which is when the new JIT will be introduced.

Regards,

Daniil.

On 10/18/23 08:52, Christian Schneider wrote:
> Am 17.10.2023 um 19:46 schrieb Daniil Gentili <daniil@daniil.it>:
>> I would like to submit an RFC and PR to change a few INI defaults for opcache, changing how JIT is disabled by default.
>>
>> Currently, JIT is disabled by default using the |opcache.jit_buffer_size=0| default, instead of |opcache.jit=0|.
>>
>> I would like to change these defaults to:
>>
>> - opcache.jit=off
>> - opcache.jit_buffer_size=64m (taken from https://github.com/php/php-src/pull/12425/files, any smaller multiple of 2 causes the buffer to fill up when running moderately sized projects like Psalm or PHPStan).
> I understand your point but it might unexpectedly switch behavior for people just setting jit_buffer_size and expecting this to turn JIT on.
> While (in the light of a new upcoming JIT)  this might not be a big deal it is also not clear if it is worth it.
> Maybe the settings can be overhauled once the new JIT is introduced (which might have different settings anyway).
>
> Regards,
> - Chris
>