Skip to content

Mark zend_version() as @compile-time-eval - #23239

Merged
DanielEScherzer merged 2 commits into
php:masterfrom
DanielEScherzer:comp-time-eval-version
Sep 2, 2026
Merged

Mark zend_version() as @compile-time-eval#23239
DanielEScherzer merged 2 commits into
php:masterfrom
DanielEScherzer:comp-time-eval-version

Conversation

@DanielEScherzer

Copy link
Copy Markdown
Member

No description provided.

@DanielEScherzer

Copy link
Copy Markdown
Member Author

I'm not an expert in opcache - are the compile-time evaluations kept when the PHP version changes?

@kocsismate

Copy link
Copy Markdown
Member

I'm not an expert in opcache - are the compile-time evaluations kept when the PHP version changes?

I'M not an expert either, but yes, as soon as anyting changes in the build config (e.g. PHP version), the cache is invalidated. See zend_startup_system_id.c

@DanielEScherzer

Copy link
Copy Markdown
Member Author

Rebased, will merge locally so commits stay signed once CI passes

@DanielEScherzer
DanielEScherzer merged commit ce7896d into php:master Sep 2, 2026
18 checks passed
@DanielEScherzer
DanielEScherzer deleted the comp-time-eval-version branch September 2, 2026 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants