Skip to content

Conversation

defkit
Copy link
Contributor

@defkit defkit commented Sep 8, 2025

Description

  • Separated acir and brillig rutnimes
  • added just brillig target

Problem*

Resolves #9743

Summary*

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

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

@defkit defkit changed the title Sn/ssa fuzzer bin chore(ssa_fuzzer): separate fuzzer runtimes + add brillig fuzz target Sep 8, 2025
@defkit defkit requested a review from Rumata888 September 8, 2025 12:04
Copy link

socket-security bot commented Sep 8, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License

View full report

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 'Execution Time'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 6c536c6 Previous: da7ea9a Ratio
semaphore-depth-10 0.025 s 0.019 s 1.32

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

CC: @TomAFrench

@defkit defkit enabled auto-merge September 10, 2025 08:04
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 'Test Suite Duration'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 9781cbd Previous: 1b5e1de Ratio
test_report_zkpassport_noir-ecdsa_ 4 s 3 s 1.33

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

CC: @TomAFrench

@defkit defkit added this pull request to the merge queue Sep 10, 2025
Merged via the queue into master with commit d672757 Sep 10, 2025
124 checks passed
@defkit defkit deleted the sn/ssa_fuzzer_bin branch September 10, 2025 08:43
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.

ssa_fuzzer: separate acir and brillig builders
2 participants