feat(optimization): Don't defunctionalize acir functions

[jfecher]

Description

Problem*

Summary*

This shouldn't be necessary know that we separate constrained-ness during monomorphization. Not sure if it'd result in any optimization changes yet though.

This seems to be causing a panic currently passing a function to brillig. We could maybe just change the translator to translate function inputs as fields.

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.

[github-actions]

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Execution Memory'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite	Current: 8a95db8	Previous: dc06d8e	Ratio
rollup-base-public	1880 MB	750.19 MB	2.51
rollup-base-private	1060 MB	610.93 MB	1.74
private-kernel-reset	460 MB	246.6 MB	1.87
private-kernel-inner	270.47 MB	210.12 MB	1.29

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench