Skip to content

Commit c293cb1

Browse files
Gilad ChaseAlon-Ti
authored andcommitted
Bump stwo version.
1 parent f0a68be commit c293cb1

File tree

6 files changed

+13
-45
lines changed

6 files changed

+13
-45
lines changed

Cargo.lock

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ serde_json = "1.0.1"
2828
sonic-rs = "0.3.10"
2929
starknet-ff = "0.3.7"
3030
# TODO(ShaharS): take stwo version from the source repository.
31-
stwo-prover = { git = "https://github.com/starkware-libs/stwo", rev = "bede666c" }
31+
stwo-prover = { git = "https://github.com/starkware-libs/stwo", rev = "81d1fe3" }
3232
thiserror = "1.0.63"
3333
tracing = "0.1.40"
3434
tracing-subscriber = "0.3.18"

crates/prover/src/cairo_air/mod.rs

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
use itertools::{chain, Itertools};
22
use num_traits::Zero;
33
use serde::{Deserialize, Serialize};
4-
use stwo_prover::constraint_framework::preprocessed_columns::gen_is_first;
54
use stwo_prover::constraint_framework::{Relation, TraceLocationAllocator};
65
use stwo_prover::core::air::{Component, ComponentProver};
76
use stwo_prover::core::backend::simd::SimdBackend;
@@ -136,7 +135,7 @@ impl CairoComponents {
136135
interaction_elements.addr_to_value_lookup.clone(),
137136
interaction_claim.clone(),
138137
),
139-
(interaction_claim.claimed_sum, None),
138+
interaction_claim.claimed_sum,
140139
)
141140
})
142141
.collect_vec();
@@ -148,7 +147,7 @@ impl CairoComponents {
148147
interaction_elements.addr_to_value_lookup.clone(),
149148
interaction_claim.addr_to_value.clone(),
150149
),
151-
(interaction_claim.addr_to_value.clone().claimed_sum, None),
150+
interaction_claim.addr_to_value.clone().claimed_sum,
152151
);
153152
Self {
154153
ret: ret_components,
@@ -254,21 +253,6 @@ pub fn prove_cairo(input: CairoInput) -> Result<CairoProof<Blake2sMerkleHasher>,
254253
interaction_claim.mix_into(channel);
255254
tree_builder.commit(channel);
256255

257-
// Fixed trace.
258-
let mut tree_builder = commitment_scheme.tree_builder();
259-
let addr_to_value_constant_trace =
260-
gen_is_first::<SimdBackend>(claim.addr_to_value.log_sizes()[2][0]);
261-
let range_check9_9_constant_trace = gen_is_first::<SimdBackend>(18);
262-
tree_builder.extend_evals(
263-
[
264-
vec![addr_to_value_constant_trace],
265-
vec![range_check9_9_constant_trace],
266-
]
267-
.into_iter()
268-
.flatten(),
269-
);
270-
tree_builder.commit(channel);
271-
272256
// Component provers.
273257
let component_builder = CairoComponents::new(&claim, &interaction_elements, &interaction_claim);
274258
let components = component_builder.provers();

crates/prover/src/components/add_mul_opcode/component.rs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
use itertools::{chain, Itertools};
22
use num_traits::One;
33
use serde::{Deserialize, Serialize};
4-
use stwo_prover::constraint_framework::logup::LogupSums;
54
use stwo_prover::constraint_framework::{
65
EvalAtRow, FrameworkComponent, FrameworkEval, RelationEntry,
76
};
87
use stwo_prover::core::backend::simd::m31::LOG_N_LANES;
98
use stwo_prover::core::channel::Channel;
109
use stwo_prover::core::fields::m31::M31;
10+
use stwo_prover::core::fields::qm31::SecureField;
1111
use stwo_prover::core::fields::secure_column::SECURE_EXTENSION_DEGREE;
1212
use stwo_prover::core::pcs::TreeVec;
1313

@@ -176,15 +176,10 @@ impl Claim {
176176
#[derive(Clone, Serialize, Deserialize)]
177177
pub struct InteractionClaim {
178178
pub log_size: u32,
179-
pub logup_sums: LogupSums,
179+
pub claimed_sum: SecureField,
180180
}
181181
impl InteractionClaim {
182182
pub fn mix_into(&self, channel: &mut impl Channel) {
183-
let (total_sum, claimed_sum) = self.logup_sums;
184-
channel.mix_felts(&[total_sum]);
185-
if let Some(claimed_sum) = claimed_sum {
186-
channel.mix_felts(&[claimed_sum.0]);
187-
channel.mix_u64(claimed_sum.1 as u64);
188-
}
183+
channel.mix_felts(&[self.claimed_sum]);
189184
}
190185
}

crates/prover/src/components/add_mul_opcode/mod.rs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ mod tests {
1111
use num_traits::Zero;
1212
use rand::rngs::SmallRng;
1313
use rand::{Rng, SeedableRng};
14-
use stwo_prover::constraint_framework::preprocessed_columns::gen_is_first;
1514
use stwo_prover::constraint_framework::{
1615
FrameworkComponent, FrameworkEval, TraceLocationAllocator,
1716
};
@@ -111,10 +110,8 @@ mod tests {
111110
config, &twiddles,
112111
);
113112

114-
// Preprocessed trace.
115-
let mut tree_builder = commitment_scheme.tree_builder();
116-
let range_check_preprocessed_trace = gen_is_first::<SimdBackend>(LOG_HEIGHT);
117-
tree_builder.extend_evals([range_check_preprocessed_trace]);
113+
// Preprocessed.
114+
let tree_builder = commitment_scheme.tree_builder();
118115
tree_builder.commit(channel);
119116

120117
let mut tree_builder = commitment_scheme.tree_builder();
@@ -137,7 +134,7 @@ mod tests {
137134
let component = FrameworkComponent::new(
138135
trace_location_allocator,
139136
Eval::new(claim, memory_relation, state_relation),
140-
interaction_claim.logup_sums,
137+
interaction_claim.claimed_sum,
141138
);
142139

143140
let trace_polys = commitment_scheme
@@ -151,7 +148,7 @@ mod tests {
151148
|eval| {
152149
component.evaluate(eval);
153150
},
154-
interaction_claim.logup_sums,
151+
interaction_claim.claimed_sum,
155152
)
156153
}
157154
}

crates/prover/src/components/add_mul_opcode/prover.rs

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -138,20 +138,12 @@ impl InteractionClaimGenerator {
138138
}
139139
col2.finalize_col();
140140

141-
let (trace, total_sum, claimed_sum) = if self.n_calls == 1 << log_size {
142-
let (trace, claimed_sum) = logup_gen.finalize_last();
143-
(trace, claimed_sum, None)
144-
} else {
145-
let (trace, [total_sum, claimed_sum]) =
146-
logup_gen.finalize_at([(1 << log_size) - 1, self.n_calls - 1]);
147-
(trace, total_sum, Some((claimed_sum, self.n_calls - 1)))
148-
};
149-
141+
let (trace, claimed_sum) = logup_gen.finalize_last();
150142
tree_builder.extend_evals(trace);
151143

152144
InteractionClaim {
153145
log_size,
154-
logup_sums: (total_sum, claimed_sum),
146+
claimed_sum,
155147
}
156148
}
157149
}

0 commit comments

Comments
 (0)