Skip to content

Commit

Permalink
clippy
Browse files Browse the repository at this point in the history
  • Loading branch information
zhiyong1997 committed Sep 4, 2024
1 parent e6397f0 commit 762a757
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 18 deletions.
4 changes: 1 addition & 3 deletions src/prover/gkr.rs
Original file line number Diff line number Diff line change
@@ -1,13 +1,11 @@
//! This module implements the core GKR IOP.
use std::ops::Mul;

use arith::{Field, SimdField};
use ark_std::{end_timer, start_timer};

use crate::{
eq_eval_at, sumcheck_prove_gkr_layer, Circuit, GKRConfig, GkrScratchpad, MultiLinearPoly,
SumcheckGkrHelper, Transcript,
sumcheck_prove_gkr_layer, Circuit, GKRConfig, GkrScratchpad, MultiLinearPoly, Transcript,
};

// FIXME
Expand Down
5 changes: 1 addition & 4 deletions src/prover/sumcheck.rs
Original file line number Diff line number Diff line change
Expand Up @@ -119,10 +119,7 @@ pub fn sumcheck_prove_gkr_square_layer<C: GKRConfig>(

#[cfg(test)]
mod tests {
use crate::{
sumcheck_prove_gkr_layer, utils::KECCAK_M31_CIRCUIT, BN254ConfigKeccak, Circuit, GKRConfig,
GkrScratchpad, Keccak256hasher, RawCommitment, Transcript,
};
use crate::BN254ConfigKeccak;

type C = BN254ConfigKeccak;

Expand Down
6 changes: 3 additions & 3 deletions src/prover/sumcheck_helper.rs
Original file line number Diff line number Diff line change
Expand Up @@ -448,15 +448,15 @@ impl<'a, C: GKRConfig> SumcheckGkrHelper<'a, C> {

pub(crate) fn prepare_simd(&mut self) {
eq_eval_at(
&self.r_simd0,
self.r_simd0,
&C::ChallengeField::one(),
&mut self.sp.eq_evals_at_r_simd0,
&mut self.sp.eq_evals_first_half,
&mut self.sp.eq_evals_second_half,
);

eq_eval_at(
&self.r_simd1,
self.r_simd1,
&C::ChallengeField::one(),
&mut self.sp.eq_evals_at_r_simd1,
&mut self.sp.eq_evals_first_half,
Expand Down Expand Up @@ -570,7 +570,7 @@ impl<'a, C: GKRConfig> SumcheckGkrHelper<'a, C> {
&C::ChallengeField::one(),
&mut self.sp.eq_evals_at_r_simd0,
&mut self.sp.eq_evals_first_half,
&mut &mut self.sp.eq_evals_second_half,
&mut self.sp.eq_evals_second_half,
);
let eq_evals_at_r_simdx = C::Field::pack(&self.sp.eq_evals_at_r_simd0);

Expand Down
16 changes: 8 additions & 8 deletions src/verifier.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
use std::{io::Cursor, vec};

use arith::{ExtensionField, Field, SimdField};
use arith::{ExtensionField, Field};
use ark_std::{end_timer, iterable::Iterable, start_timer};

#[cfg(feature = "grinding")]
Expand Down Expand Up @@ -165,10 +165,10 @@ fn sumcheck_verify_gkr_layer<C: GKRConfig>(
r_simd1,
alpha,
beta,
&vec![],
&vec![],
&vec![],
&vec![],
&[],
&[],
&[],
&[],
);

let mut rx = vec![];
Expand Down Expand Up @@ -196,9 +196,9 @@ fn sumcheck_verify_gkr_layer<C: GKRConfig>(
alpha,
beta,
&rx,
&vec![],
&[],
&r_simdx,
&vec![],
&[],
);
transcript.append_challenge_f::<C>(&vx_claim);

Expand All @@ -215,7 +215,7 @@ fn sumcheck_verify_gkr_layer<C: GKRConfig>(
== vx_claim
* vy_claim
* eval_sparse_circuit_connect_poly(
&layer.mul, rz0, rz1, r_simd0, r_simd1, alpha, beta, &rx, &ry, &r_simd0, &r_simd1,
&layer.mul, rz0, rz1, r_simd0, r_simd1, alpha, beta, &rx, &ry, r_simd0, r_simd1,
);
transcript.append_challenge_f::<C>(&vy_claim);
(verified, rx, ry, r_simdx, r_simdy, vx_claim, vy_claim)
Expand Down

0 comments on commit 762a757

Please sign in to comment.