Skip to content

Commit cdc2ac7

Browse files
committed
Fix non-parallel bugs.
1 parent 8c41adb commit cdc2ac7

File tree

3 files changed

+4
-5
lines changed

3 files changed

+4
-5
lines changed

crates/stwo/src/prover/backend/simd/grind.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ pub mod poseidon252 {
146146
]);
147147
#[cfg(not(feature = "parallel"))]
148148
let res = (0..)
149-
.find_map(|hi| grind_poseidon(digest, hi, pow_bits))
149+
.find_map(|hi| grind_poseidon(prefixed_digest, hi, pow_bits))
150150
.expect("Grind failed to find a solution.");
151151

152152
#[cfg(feature = "parallel")]

crates/stwo/src/prover/backend/simd/poseidon252_lifted.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ use crate::core::vcs_lifted::merkle_hasher::MerkleHasherLifted;
99
use crate::core::vcs_lifted::poseidon252_merkle::{
1010
poseidon_finalize, poseidon_update, Poseidon252MerkleHasher, ELEMENTS_IN_BUFFER,
1111
};
12-
#[cfg(feature = "parallel")]
1312
use crate::prover::backend::simd::m31::N_LANES;
1413
use crate::prover::backend::simd::SimdBackend;
1514
use crate::prover::backend::{Col, Column, CpuBackend};

crates/stwo/src/prover/backend/simd/quotients.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -93,10 +93,10 @@ impl QuotientOps for SimdBackend {
9393
// Populate `quotients`.
9494
// TODO(Leo): make chunk size configurable.
9595
#[cfg(not(feature = "parallel"))]
96-
let iter = quotients.iter_mut(1).enumerate();
96+
let iter = quotients.chunks_mut(1).enumerate();
9797

9898
#[cfg(feature = "parallel")]
99-
let iter = quotients.chunks_mut(1).enumerate();
99+
let iter = quotients.par_chunks_mut(1).enumerate();
100100

101101
iter.for_each(|(domain_idx, mut value_dst)| {
102102
let mut quotient = PackedSecureField::zero();
@@ -147,7 +147,7 @@ fn denominator_inverses(
147147
let domain_points = CircleDomainBitRevIterator::new(domain);
148148

149149
#[cfg(not(feature = "parallel"))]
150-
let iter = domain_points.iter();
150+
let iter = domain_points;
151151

152152
#[cfg(feature = "parallel")]
153153
let iter = domain_points.par_iter();

0 commit comments

Comments
 (0)