Skip to content

Commit 0d18909

Browse files
author
notnotraju
committed
a comment for the third term was accidentally placed under the second term, fixed.
1 parent a68c3e8 commit 0d18909

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

barretenberg/cpp/src/barretenberg/relations/ecc_vm/ecc_set_relation_impl.hpp

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,17 @@ namespace bb {
3535
* Input source: ECCVMPointTableRelation
3636
* Output source: ECCVMTranscriptRelation
3737
*
38+
* Note that, from the latter table, this is only turned on when we are at a `mul` instruction. Similarly, from the
39+
* former table, this is only turned on when `precompute_point_transition == 1`. THIRD TERM: tuple of (pc, P.x, P.y,
40+
* msm-size) from ECCVMMSMRelation, to link the output of the MSM computation from the MSM table to the values in the
41+
* Transcript tables.
42+
*
43+
* Input source: ECCVMMSMRelation
44+
* Output source: ECCVMTranscriptRelation
3845
* Note that, from the latter table, this is only turned on when we are at an MSM transition, so we don't record the
3946
* "intermediate" `transcript_pc` values from the Transcript columns. This is compatible with the fact that the `msm_pc`
4047
* values are _constant_ on a fixed MSM.
4148
*
42-
* THIRD TERM: tuple of (pc, P.x, P.y, msm-size) from ECCVMMSMRelation, to link the output of the MSM computation from
43-
* the MSM table to the values in the Transcript tables.
44-
*
45-
* Input source: ECCVMMSMRelation
46-
* Output source: ECCVMTranscriptRelation
4749
*
4850
* @tparam FF
4951
* @tparam AccumulatorTypes

0 commit comments

Comments
 (0)