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

fix(ssa refactor): Fix recursive call to create_value_from_type #1815

Merged
merged 1 commit into from
Jun 23, 2023

Conversation

jfecher
Copy link
Contributor

@jfecher jfecher commented Jun 23, 2023

Description

Problem*

The "missing assignment for witness index" error was caused by an incorrect call in the array-element case of create_value_from_type which should have been a recursive call but instead was still calling convert_ssa_block_param. This was an oversight from PR #1797.

Resolves #1799

Summary*

With this PR sha2_byte and sha2_block no longer hit this error and instead run into a "cannot satisfy all constraints" error as in #1800.

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.

@kevaundray kevaundray added this pull request to the merge queue Jun 23, 2023
Merged via the queue into master with commit 890a63b Jun 23, 2023
@kevaundray kevaundray deleted the jf/fix-1799 branch June 23, 2023 18:47
@ludamad
Copy link
Collaborator

ludamad commented Jun 23, 2023

Should we now copy sha2 bytes over?

@jfecher
Copy link
Contributor Author

jfecher commented Jun 23, 2023

@ludamad nope :/

With this PR sha2_byte and sha2_block no longer hit this error and instead run into a "cannot satisfy all constraints" error as in #1800.

Just more bugs hiding behind this bug

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.

Cannot solve opcode: missing assignment for witness index 100
3 participants