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

chore: Add benchmarking dashboard #7068

Merged
merged 15 commits into from
Jan 17, 2025
Prev Previous commit
Next Next commit
.
TomAFrench committed Jan 17, 2025

Verified

This commit was signed with the committer’s verified signature.
yohang Yohan Giarelli
commit 69cc9463133e546a7f0f30c0132ea173adb8f821
6 changes: 5 additions & 1 deletion .github/workflows/reports.yml
Original file line number Diff line number Diff line change
@@ -735,6 +735,10 @@ jobs:
runs-on: ubuntu-latest
permissions:
pull-requests: write
# deployments permission to deploy GitHub pages website
deployments: write
# contents permission to update benchmark contents in gh-pages branch
contents: write

steps:
- uses: actions/checkout@v4
@@ -773,7 +777,7 @@ jobs:

- name: Convert to `benchmark-action` format
run: |
jq ".execution_report | map({name: .artifact_name, value: (.time[:-1] | tonumber), unit: \"s\"}) " ./execution_report.json > time_bench.json
jq ".execution_reports | map({name: .artifact_name, value: (.time[:-1] | tonumber), unit: \"s\"}) " ./execution_report.json > time_bench.json

- name: Store benchmark result
# We always push new benchmarks so we want this to only run on master to avoid garbage data.``

Unchanged files with check annotations Beta

MakeArray { .. } | Noop => false,
// Some binary math can overflow or underflow
Binary(binary) => match binary.operator {

Check warning on line 414 in compiler/noirc_evaluator/src/ssa/ir/instruction.rs

GitHub Actions / Code

Unknown word (effectful)
BinaryOp::Add { unchecked: false }
| BinaryOp::Sub { unchecked: false }
| BinaryOp::Mul { unchecked: false }
/// br loop_entry(v0)
/// loop_entry(i: Field):
/// v2 = lt i v1
/// brif v2, then: loop_body, else: loop_end

Check warning on line 525 in compiler/noirc_evaluator/src/ssa/ssa_gen/mod.rs

GitHub Actions / Code

Unknown word (brif)
/// loop_body():
/// v3 = ... codegen body ...
/// v4 = add 1, i
///
/// ```text
/// v0 = ... codegen cond ...
/// brif v0, then: then_block, else: else_block

Check warning on line 596 in compiler/noirc_evaluator/src/ssa/ssa_gen/mod.rs

GitHub Actions / Code

Unknown word (brif)
/// then_block():
/// v1 = ... codegen a ...
/// br end_if(v1)
///
/// ```text
/// v0 = ... codegen cond ...
/// brif v0, then: then_block, else: end_if

Check warning on line 611 in compiler/noirc_evaluator/src/ssa/ssa_gen/mod.rs

GitHub Actions / Code

Unknown word (brif)
/// then_block:
/// v1 = ... codegen a ...
/// br end_if()