Open adria0 opened 2 months ago
Attention: Patch coverage is 81.97065%
with 86 lines
in your changes missing coverage. Please review.
Project coverage is 81.81%. Comparing base (
32599e8
) to head (c8c7ff9
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
NOTE: Relevant files to review are
halo2_frontend/src/plonk/circuit/{expession.rs, arena.rs}
This an experiment about making expression copiable. To implement this we go with this strategy:
ExprRef<F>
that contains the index (usize) of an expression stored in an specific arena.Expression
to useExprRef
instead the a recursiveExpression
that makes it non-copiable.FieldFront
. Mainly we change all frontend fromField
toFieldFront
, making arena accessible everywhere.This means:
<Field>
for<FieldFront>
and automatically all theirExpressions
are automatically copiable. This is thread-safe.