Skip to content

Conversation

@TomAFrench
Copy link
Member

Description

Problem*

Resolves

Summary*

Additional Context

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'ACIR Opcodes'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.01.

Benchmark suite Current: 8588a5c Previous: c13415f Ratio
evaluate_linear_expression_BLS12_377_Bench.json/main 80273 acir_opcodes 74597 acir_opcodes 1.08
evaluate_linear_expression_BLS12_381_Bench.json/main 80273 acir_opcodes 74597 acir_opcodes 1.08
evaluate_linear_expression_BN254_Bench.json/main 54903 acir_opcodes 50646 acir_opcodes 1.08
evaluate_linear_expression_MNT4_753_Bench.json/main 173448 acir_opcodes 163515 acir_opcodes 1.06
evaluate_linear_expression_MNT6_753_Bench.json/main 173448 acir_opcodes 163515 acir_opcodes 1.06
evaluate_linear_expression_Pallas_Bench.json/main 54903 acir_opcodes 50646 acir_opcodes 1.08
evaluate_linear_expression_Secp256k1_Bench.json/main 54903 acir_opcodes 50646 acir_opcodes 1.08
evaluate_linear_expression_Secp256r1_Bench.json/main 54904 acir_opcodes 50647 acir_opcodes 1.08
evaluate_linear_expression_Secp384r1_Bench.json/main 80274 acir_opcodes 74598 acir_opcodes 1.08
evaluate_linear_expression_Vesta_Bench.json/main 54903 acir_opcodes 50646 acir_opcodes 1.08
mul_BLS12_377_Bench.json/main 53062 acir_opcodes 49162 acir_opcodes 1.08
mul_BLS12_381_Bench.json/main 53062 acir_opcodes 49162 acir_opcodes 1.08
mul_BN254_Bench.json/main 36011 acir_opcodes 33086 acir_opcodes 1.09
mul_MNT4_753_Bench.json/main 115952 acir_opcodes 109127 acir_opcodes 1.06
mul_MNT6_753_Bench.json/main 115952 acir_opcodes 109127 acir_opcodes 1.06
mul_Pallas_Bench.json/main 36011 acir_opcodes 33086 acir_opcodes 1.09
mul_Secp256k1_Bench.json/main 36011 acir_opcodes 33086 acir_opcodes 1.09
mul_Secp256r1_Bench.json/main 36012 acir_opcodes 33087 acir_opcodes 1.09
mul_Secp384r1_Bench.json/main 53063 acir_opcodes 49163 acir_opcodes 1.08
mul_Vesta_Bench.json/main 36011 acir_opcodes 33086 acir_opcodes 1.09

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Circuit Size'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.01.

Benchmark suite Current: 8588a5c Previous: c13415f Ratio
evaluate_linear_expression_BLS12_377_Bench.json/main 236889 circuit_size 212412 circuit_size 1.12
evaluate_linear_expression_BLS12_381_Bench.json/main 238279 circuit_size 213802 circuit_size 1.11
evaluate_linear_expression_BN254_Bench.json/main 160427 circuit_size 144108 circuit_size 1.11
evaluate_linear_expression_MNT4_753_Bench.json/main 498934 circuit_size 451752 circuit_size 1.10
evaluate_linear_expression_MNT6_753_Bench.json/main 498934 circuit_size 451752 circuit_size 1.10
evaluate_linear_expression_Pallas_Bench.json/main 161514 circuit_size 144485 circuit_size 1.12
evaluate_linear_expression_Secp256k1_Bench.json/main 161483 circuit_size 144455 circuit_size 1.12
evaluate_linear_expression_Secp256r1_Bench.json/main 161487 circuit_size 144459 circuit_size 1.12
evaluate_linear_expression_Secp384r1_Bench.json/main 237420 circuit_size 212942 circuit_size 1.11
evaluate_linear_expression_Vesta_Bench.json/main 161514 circuit_size 144485 circuit_size 1.12
mul_BLS12_377_Bench.json/main 159107 circuit_size 142289 circuit_size 1.12
mul_BLS12_381_Bench.json/main 160497 circuit_size 143679 circuit_size 1.12
mul_BN254_Bench.json/main 107796 circuit_size 96583 circuit_size 1.12
mul_MNT4_753_Bench.json/main 335522 circuit_size 303104 circuit_size 1.11
mul_MNT6_753_Bench.json/main 335522 circuit_size 303104 circuit_size 1.11
mul_Pallas_Bench.json/main 108546 circuit_size 96845 circuit_size 1.12
mul_Secp256k1_Bench.json/main 108515 circuit_size 96816 circuit_size 1.12
mul_Secp256r1_Bench.json/main 108519 circuit_size 96820 circuit_size 1.12
mul_Secp384r1_Bench.json/main 159527 circuit_size 142708 circuit_size 1.12
mul_Vesta_Bench.json/main 108546 circuit_size 96845 circuit_size 1.12

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Brillig Bytecode Size'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.01.

Benchmark suite Current: 8588a5c Previous: c13415f Ratio
validate_on_curve_BLS12_377_Bench 4390 opcodes 4313 opcodes 1.02
validate_on_curve_BLS12_381_Bench 4391 opcodes 4314 opcodes 1.02
validate_on_curve_BN254_Bench 4228 opcodes 4153 opcodes 1.02
validate_on_curve_MNT4_753_Bench 4648 opcodes 4565 opcodes 1.02
validate_on_curve_MNT6_753_Bench 4648 opcodes 4565 opcodes 1.02
validate_on_curve_Pallas_Bench 4228 opcodes 4153 opcodes 1.02
validate_on_curve_Secp256k1_Bench 4232 opcodes 4157 opcodes 1.02
validate_on_curve_Secp256r1_Bench 4236 opcodes 4161 opcodes 1.02
validate_on_curve_Secp384r1_Bench 4394 opcodes 4317 opcodes 1.02
validate_on_curve_Vesta_Bench 4228 opcodes 4153 opcodes 1.02

This comment was automatically generated by workflow using github-action-benchmark.

@TomAFrench TomAFrench merged commit a1fe0ee into main Nov 14, 2025
10 checks passed
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