Skip to content

Commit

Permalink
Remove a sanity check that calls to_matrices before inlining
Browse files Browse the repository at this point in the history
  • Loading branch information
weikengchen authored and Pratyush committed Nov 12, 2020
1 parent 0380bdb commit 6065113
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions src/ahp/constraint_systems.rs
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,6 @@ pub(crate) fn num_non_zero<F: PrimeField>(matrices: &ConstraintMatrices<F>) -> u

pub(crate) fn make_matrices_square_for_indexer<F: PrimeField>(cs: ConstraintSystemRef<F>) {
let num_variables = cs.num_instance_variables() + cs.num_witness_variables();
let matrices = cs.to_matrices().unwrap();
let num_non_zero_val = num_non_zero::<F>(&matrices);
let matrix_dim = padded_matrix_dim(num_variables, cs.num_constraints());
make_matrices_square(cs.clone(), num_variables);
assert_eq!(
Expand All @@ -62,11 +60,6 @@ pub(crate) fn make_matrices_square_for_indexer<F: PrimeField>(cs: ConstraintSyst
matrix_dim,
"padding does not result in expected matrix size!"
);
assert_eq!(
num_non_zero::<F>(&matrices),
num_non_zero_val,
"padding changed matrix density"
);
}

/// This must *always* be in sync with `make_matrices_square`.
Expand Down

0 comments on commit 6065113

Please sign in to comment.