Skip to content

feat: add brillig support for blake3#3995

Closed
TomAFrench wants to merge 1 commit intomasterfrom
tf/codegen-blake3-brillig
Closed

feat: add brillig support for blake3#3995
TomAFrench wants to merge 1 commit intomasterfrom
tf/codegen-blake3-brillig

Conversation

@TomAFrench
Copy link
Member

Description

Problem*

Resolves #3987

Summary*

Allows blake3 in brillig.

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [Exceptional Case] 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.

@TomAFrench TomAFrench requested a review from sirasistant January 9, 2024 20:55
@TomAFrench
Copy link
Member Author

Will redo this PR on the aztec-packages side as it breaks serialisation. (We should probably rework this so that backends can just ignore brillig entirely.)

@TomAFrench
Copy link
Member Author

The above is doable with #3907

@TomAFrench
Copy link
Member Author

Closing in favour of AztecProtocol/aztec-packages#3913

@TomAFrench TomAFrench closed this Jan 10, 2024
@TomAFrench TomAFrench deleted the tf/codegen-blake3-brillig branch January 14, 2024 22:06
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.

Implement codegen for Blake3 in brillig

1 participant