-
Notifications
You must be signed in to change notification settings - Fork 48
Implement libfunc profiler. #946
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
Conversation
Benchmarking resultsBenchmark for program
|
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
11.388 ± 0.061 | 11.329 | 11.498 | 2.27 ± 0.02 |
cairo-native (embedded AOT) |
5.014 ± 0.021 | 4.980 | 5.050 | 1.00 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
5.113 ± 0.032 | 5.069 | 5.174 | 1.02 ± 0.01 |
Benchmark for program dict_snapshot
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
810.6 ± 8.0 | 798.6 | 822.4 | 1.00 |
cairo-native (embedded AOT) |
4914.7 ± 27.8 | 4865.0 | 4957.1 | 6.06 ± 0.07 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
5027.0 ± 30.3 | 4987.5 | 5087.9 | 6.20 ± 0.07 |
Benchmark for program factorial_2M
Open benchmarks
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
5.259 ± 0.019 | 5.229 | 5.290 | 1.00 |
cairo-native (embedded AOT) |
5.305 ± 0.024 | 5.268 | 5.348 | 1.01 ± 0.01 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
5.374 ± 0.041 | 5.316 | 5.465 | 1.02 ± 0.01 |
Benchmark for program fib_2M
Open benchmarks
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
5.116 ± 0.011 | 5.103 | 5.132 | 1.06 ± 0.01 |
cairo-native (embedded AOT) |
4.835 ± 0.021 | 4.809 | 4.866 | 1.00 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
4.878 ± 0.024 | 4.835 | 4.910 | 1.01 ± 0.01 |
Benchmark for program linear_search
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
832.8 ± 4.6 | 824.3 | 838.6 | 1.00 |
cairo-native (embedded AOT) |
4984.5 ± 19.6 | 4948.1 | 5011.7 | 5.99 ± 0.04 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
5110.3 ± 33.6 | 5069.3 | 5163.4 | 6.14 ± 0.05 |
Benchmark for program logistic_map
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
539.1 ± 3.4 | 534.3 | 545.7 | 1.00 |
cairo-native (embedded AOT) |
5025.6 ± 31.4 | 4965.9 | 5058.4 | 9.32 ± 0.08 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
5209.6 ± 24.5 | 5178.4 | 5257.8 | 9.66 ± 0.08 |
Benchmark results Main vs HEAD.Base
Head
Base
Head
Base
Head
Base
Head
Base
Head
Base
Head
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #946 +/- ##
=======================================
Coverage ? 80.21%
=======================================
Files ? 112
Lines ? 29880
Branches ? 0
=======================================
Hits ? 23967
Misses ? 5913
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
f711785
to
43dde6e
Compare
16218e2
to
d1f3594
Compare
…native into add-libfunc-profiling
…native into add-libfunc-profiling
…native into add-libfunc-profiling
517633b
to
ae97f9c
Compare
All comments were resolved |
2dbc2ad
to
1cf82a7
Compare
…native into add-libfunc-profiling
1cf82a7
to
76fd95a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
These changes were squashed in #1253. |
Implementation of a libfunc profiler.
For each executed libfunc, generates:
Also implements an outlier detection system to filter out operating system noise.
Output Example:
Where
[2]
refers to the libfunc id.Checklist