Skip to content

Commit 83cf5bb

Browse files
authored
perf(mpcs): copy jagged q from committed column MLEs (#62)
* fix: align jagged q layout with committed rows * fix(mpcs): compact jagged q over occupied rows * rollback unnessesary comments * rollback unnessesary change
1 parent 8b847f4 commit 83cf5bb

1 file changed

Lines changed: 2 additions & 9 deletions

File tree

crates/mpcs/src/jagged/mod.rs

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -279,19 +279,12 @@ pub fn jagged_commit<E: ExtensionField, InnerPcs: PolynomialCommitmentScheme<E>>
279279
// The start position in `concatenated` for this matrix's block of polynomials.
280280
let start = cumulative_heights[poly_idx];
281281

282-
// Step 3: Transpose — write each column j of `rmm` (= one polynomial)
283-
// into its corresponding contiguous slice in `concatenated`.
282+
// Step 3: write each committed column prefix into its compact q slice.
284283
(0..n_cols)
285284
.into_par_iter()
286285
.zip(concatenated[start..start + n_cells].par_chunks_mut(n_rows))
287286
.for_each(|(j, chunk)| {
288-
rmm.values
289-
.iter()
290-
.take(n_cells)
291-
.skip(j)
292-
.step_by(n_cols)
293-
.zip_eq(chunk.iter_mut())
294-
.for_each(|(v, out)| *out = *v);
287+
chunk.copy_from_slice(&polys[poly_idx + j].get_base_field_vec()[..n_rows]);
295288
});
296289

297290
poly_idx += n_cols;

0 commit comments

Comments
 (0)