Skip to content

Remove clones in step operations #2301

@gabrielbosio

Description

@gabrielbosio

Line 336 clones every Res::Op1 case:

Res::Op1 => Ok(Some(op1.clone()))

Line 349 clones every AssertEq:

(Opcode::AssertEq, Some(res)) => res.clone()

Lines 680 and 3227-3230 clone operand deductions:

Some(op0.clone())

We should replace each clone with references or restructure to avoid cloning.

Measure performance improvement of this change.

Metadata

Metadata

Assignees

No one assigned

    Labels

    performancePerformance-related improvements or regressionsresearchOpen questions requiring either research or requirements gathering

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions