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

feat: add blake3 black box function #3249

Closed
wants to merge 4 commits into from
Closed

feat: add blake3 black box function #3249

wants to merge 4 commits into from

Conversation

TomAFrench
Copy link
Member

Description

Problem*

Resolves #3171

Summary*

This PR adds a black box function for the blake3 hash function.

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.

@TomAFrench
Copy link
Member Author

@kevaundray Do we want to push for this to get merged for kernel circuit reasons?

@kevaundray
Copy link
Contributor

@kevaundray Do we want to push for this to get merged for kernel circuit reasons?

Missed this -- this is not blocking for the kernel circuit (we just changed everything to use blake2s), though we could batch it in with #3340

@kevaundray
Copy link
Contributor

It would be good to come full circle on the blake3 issue and the hash_to_field issue too

TomAFrench and others added 2 commits November 22, 2023 10:49
* master: (148 commits)
  feat: add support for writing tracing debug info to file (#3790)
  chore: clippy fix (#3803)
  chore(ci): enforce timeouts on build and test workflows (#3804)
  chore!: remove unused `source-resolver` package (#3791)
  chore!: Make file manager read-only to the compiler (#3760)
  feat: Add some traits to the stdlib (#3796)
  fix: Stop issuing unused variable warnings for variables in trait definitions (#3797)
  fix(lsp): package resolution on save (#3794)
  chore: clippy fix (#3793)
  feat: Remove experimental feature warning for traits (#3783)
  fix: Allow trait method references from the trait name (#3774)
  chore: adds a new option only-acir (#3683)
  feat(lsp): add goto definition for structs (#3718)
  chore: disable code lens feature of lsp (#3789)
  chore: moving ordering to category jsons and frontmatters (#3777)
  chore(ci): use `actions/setup-node` for caching yarn dependencies (#2730)
  fix: remove `noirc_driver/aztec` feature flag in docker (#3784)
  chore: remove aztec compile time feature flag (#3596)
  chore: move debugger tests in submodule (#3780)
  feat: simplify explicit equality assertions to assert equality directly (#3708)
  ...
@TomAFrench
Copy link
Member Author

Closing as this PR is going to be done in the aztec-packages repo

@TomAFrench TomAFrench closed this Dec 19, 2023
@TomAFrench TomAFrench deleted the tf/blake3 branch January 14, 2024 22:05
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.

Add blake3 opcode into ACIR
2 participants