Skip to content

Conversation

@giladchase
Copy link
Contributor

@giladchase giladchase commented Jan 4, 2026

Summary


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

⚠️ Note:
To keep maintainer workload sustainable, we generally do not accept PRs that
are only minor wording, grammar, formatting, or style changes.
Such PRs may be closed without detailed review.


Why is this change needed?


What was the behavior or documentation before?


What is the behavior or documentation after?


Related issue or discussion (if any)


Additional context

Add casm/sierra/diagnostic compilation phases (currently still only walltime tests)
@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link

github-actions bot commented Jan 4, 2026

Benchmark Comparison

Benchmark                                 Baseline       Current      Diff

corelib-clean-casm-walltime               522.8 ms      524.9 ms     +0.4% [SLOWER]
corelib-clean-diagnostics-walltime         22.80 s       22.61 s     -0.8% [faster]
corelib-clean-full-walltime                26.90 s       26.44 s     -1.7% [faster]
corelib-clean-sierra-walltime              26.45 s       26.35 s     -0.4% [faster]
perpetual-clean-casm-walltime                 0 ns          0 ns     +0.0% 
perpetual-clean-diagnostics-walltime          0 ns          0 ns     +0.0% 
perpetual-clean-full-walltime                 0 ns          0 ns     +0.0% 
perpetual-clean-sierra-walltime               0 ns          0 ns     +0.0% 

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.

3 participants