Skip to content

Conversation

@eytan-starkware
Copy link
Contributor

@eytan-starkware eytan-starkware commented Feb 10, 2026

Summary

Added benchmarking capabilities to the Cairo compiler using Criterion. This PR introduces a new benchmark for measuring compilation performance, starting with a simple Fibonacci example.


Type of change

Please check one:

  • Bug fix (fixes incorrect behavior)
  • New feature
  • Performance improvement
  • Documentation change with concrete technical impact
  • Style, wording, formatting, or typo-only change

Why is this change needed?

To establish a baseline for measuring compiler performance and track improvements over time. This will help identify performance regressions and validate optimization efforts in the compilation pipeline.


What was the behavior or documentation before?

The project lacked structured benchmarking for the compiler, making it difficult to quantify performance changes between versions.


What is the behavior or documentation after?

The compiler now has a Criterion-based benchmark that measures compilation time for a Fibonacci example. This provides a foundation for adding more complex benchmarks and tracking performance metrics over time.


Additional context

The benchmark is configured with a smaller sample size (10) to balance accuracy with reasonable execution time. More benchmark cases can be added in the future to cover different compilation scenarios.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

eytan-starkware commented Feb 10, 2026

@eytan-starkware eytan-starkware marked this pull request as ready for review February 10, 2026 11:35
@eytan-starkware eytan-starkware force-pushed the eytan_graphite/_feat_using_criterion_benchmarking_framework_with_simple_compile_benchmark branch from c5c34f2 to ac232c9 Compare February 10, 2026 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants