Skip to content
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

draft: Iai on CI #65

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

draft: Iai on CI #65

wants to merge 2 commits into from

Conversation

mattxwang
Copy link
Member

@mattxwang mattxwang commented Mar 20, 2023

Trivial setup that shows how iai could be used on CI.

The Bench workflow outputs:

iai_benchmark_sdd_canonicity_c1
  Instructions:             2507685
  L1 Accesses:              4301000
  L2 Accesses:                17784
  RAM Accesses:                4114
  Estimated Cycles:         4533910

iai_benchmark_sdd_canonicity_c2
  Instructions:             2529817
  L1 Accesses:              4331311
  L2 Accesses:                17648
  RAM Accesses:                4187
  Estimated Cycles:         4566096

iai_benchmark_sdd_canonicity_c3
  Instructions:             2569170
  L1 Accesses:              4385154
  L2 Accesses:                17692
  RAM Accesses:                4264
  Estimated Cycles:         4622854

This PR also make it clear to me that we really need to cache CI, hence #66; I'm surprised that there's not a first-party cache action, but I'll poke around and see what the bigger Rust projects do.

(caching valgrind would also be nice, but this isn't the big performance hit from caching)

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.

1 participant