-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(profiling) support new ZendMM api #2969
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2969 +/- ##
=========================================
Coverage 74.82% 74.82%
Complexity 2741 2741
=========================================
Files 110 110
Lines 10863 10863
=========================================
Hits 8128 8128
Misses 2735 2735
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
|
Benchmarks [ profiler ]Benchmark execution time: 2024-12-03 16:07:49 Comparing candidate commit e68c066 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 27 metrics, 8 unstable metrics. scenario:walk_stack/1
|
1ef435b
to
9f23f8d
Compare
9f23f8d
to
44f1982
Compare
146d1ec
to
ca68121
Compare
ca68121
to
840daa4
Compare
840daa4
to
e0e3862
Compare
Description
This PR changes the way we hook into the ZendMM starting from PHP 8.4 thanks to the new ZendMM API that got introduced in php/php-src#13432
PROF-10116
Reviewer checklist