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: pull acvm-backend-barretenberg into main Noir repo #2495

Merged
merged 3 commits into from
Aug 31, 2023

Conversation

TomAFrench
Copy link
Member

@TomAFrench TomAFrench commented Aug 30, 2023

Description

Problem*

Resolves #2523.

Summary*

This is a temporary step which allows us to progress on proper dynamic backends more easily and removes a layer of indirection between ACVM and Noir.

Draft PR as I want to clean things up more.

Documentation

  • This PR requires documentation updates when merged.

    • I will submit a noir-lang/docs PR.
    • I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

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

Base automatically changed from kw/bb-binaries to master August 31, 2023 10:46
@TomAFrench TomAFrench force-pushed the monorepo-backend branch 2 times, most recently from 07f9686 to 8e7078d Compare August 31, 2023 13:08
@TomAFrench TomAFrench marked this pull request as ready for review August 31, 2023 14:01
@TomAFrench TomAFrench requested a review from kobyhallx August 31, 2023 14:01
Copy link
Contributor

@kobyhallx kobyhallx left a comment

Choose a reason for hiding this comment

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

lgtm for temp situation

@TomAFrench TomAFrench added this pull request to the merge queue Aug 31, 2023
Merged via the queue into master with commit 825600a Aug 31, 2023
@TomAFrench TomAFrench deleted the monorepo-backend branch August 31, 2023 15:27
TomAFrench added a commit that referenced this pull request Sep 1, 2023
* master: (47 commits)
  fix: Initialize structs during def collection, not name resolution (#2528)
  feat: Apply optimizations to unconstrained code (#2348)
  chore(ci): Distinguish between expected failures and compiler panics in `compile_failure` tests. (#2518)
  chore: improve types in `acvm-backend-barretenberg` (#2516)
  feat(aztec_noir): abstract kernel return types (#2521)
  chore: remove usage of `Backend` trait (#2514)
  chore: delete `ProveAndVerifyCommand` (#2520)
  chore: Remove dead code from `acvm_backend_barretenberg` (#2512)
  chore: only install `tokio-util` dependency on windows (#2425)
  chore(aztec_noir):  imply the open keyword (#2508)
  chore: pull `acvm-backend-barretenberg` into main Noir repo (#2495)
  chore: clippy fix (#2507)
  chore: check if the noir aztec library is installed (#2505)
  chore: update ACIR artifacts (#2503)
  chore!: Update to `acvm-backend-barretenberg` v0.12.0 (#2377)
  fix: Bring back accidentally deleted double_verify_proof test. (#2501)
  chore(aztec_noir): import aztec library if not found yet (#2492)
  chore(abi)!: Replace struct name with fully qualified struct path (#2374)
  chore!: Remove keys from preprocessed artifacts (#2283)
  chore(noir): Release 0.10.5 (#2482)
  ...
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.

Pull acvm-backend-barretenberg into this noir repo
2 participants