feat(acvm)!: have all black box functions return Result<OpcodeResolution, OpcodeResolutionError>

[TomAFrench]

Related issue(s)

(If it does not already exist, first create a GitHub issue that describes the problem this Pull Request (PR) solves before creating the PR and link it here.)

Resolves (link to issue)

Description

Summary of changes

For consistency, all bb function implementations inside ACVM pass back a Result<OpcodeResolution, OpcodeResolutionError> so backends don't need to do anything other than forward the return value back into the ACVM.

Dependency additions / changes

(If applicable.)

Test additions / changes

(If applicable.)

Checklist

• I have tested the changes locally.
• I have formatted the changes with Prettier and/or cargo fmt with default settings.
• I have linked this PR to the issue(s) that it resolves.
• I have reviewed the changes on GitHub, line by line.
• I have ensured all changes are covered in the description.

Additional context

(If applicable.)