test(benches): add basic benchmark infrastructure#602
Merged
ptondereau merged 6 commits intomasterfrom Mar 26, 2026
Merged
Conversation
1 task
Pull Request Test Coverage Report for Build 20556134071Details
💛 - Coveralls |
Xenira
force-pushed
the
599-add-performance-test-infrastructure
branch
9 times, most recently
from
d5bc0a9 to
60971aa
Compare
Member
Author
|
Example upload https://bencher.php.rs/perf/ext-php-rs CI upload only works once this is merged. |
Member
Author
|
@ptondereau I think this works as an initial implementation. We should extend this further especially before tackling any performance issues. |
ptondereau
previously approved these changes
Dec 18, 2025
Member
ptondereau
left a comment
ptondereau
left a comment
There was a problem hiding this comment.
Look great! I tested it locally and am showing some good results indeed.
Some nitpicks
ptondereau
previously approved these changes
Dec 28, 2025
Member
|
@Xenira do you need help finishing your work here? This is a banger to me, and I can't wait to merge it |
Co-authored-by: Pierre Tondereau <ptondereau@users.noreply.github.com>
ptondereau
force-pushed
the
599-add-performance-test-infrastructure
branch
4 times, most recently
from
3fbbbbb to
a1b99e4
Compare
ptondereau
force-pushed
the
599-add-performance-test-infrastructure
branch
8 times, most recently
from
bc22352 to
53bbe29
Compare
Split benches into two independent crates to avoid the links="clang" conflict between ext-php-rs-clang-sys and clang-sys (from gungraun's bindgen): - benches/ext/: PHP extension cdylib (depends on ext-php-rs only) - benches/: benchmark harness (depends on gungraun only) Valgrind client requests reimplemented via inline assembly in the ext crate, removing the need for crabgrind/gungraun client_requests. CI fixes: trigger on master not main, add missing bencher install step, bump github-script to v7, drop flake.lock drift, remove edited PR trigger type.
ptondereau
force-pushed
the
599-add-performance-test-infrastructure
branch
from
53bbe29 to
e926074
Compare
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs: #599
Description
Adds basic benchmarking infrastructure
TODO:
Checklist
Check the boxes that apply (put an
xin the brackets, like[x]). You can also check boxes after the PR is created.❤️ Thank you for your contribution!