Skip to content

fix(ssa): Handle array get result under predicate#8356

Closed
vezenovm wants to merge 1 commit intomasterfrom
mv/handle_array_get_res_side_effects
Closed

fix(ssa): Handle array get result under predicate#8356
vezenovm wants to merge 1 commit intomasterfrom
mv/handle_array_get_res_side_effects

Conversation

@vezenovm
Copy link
Contributor

@vezenovm vezenovm commented May 5, 2025

Description

Problem*

Resolves #8236

Alternative to #8355

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.

@vezenovm
Copy link
Contributor Author

vezenovm commented May 6, 2025

Closing for now. This was an initial naive attempt at an optimization. For now we can go forward with #8355 which fixes the linked issue and look at bringing back the array get optimization later.

@vezenovm vezenovm closed this May 6, 2025
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.

ACIR != Brillig: passing unused parameters

1 participant