From b467a13d91ae6e491fdea3750d9432ba7e204350 Mon Sep 17 00:00:00 2001 From: Nicholas Ward Date: Thu, 7 Jul 2022 09:28:34 -0700 Subject: [PATCH] fix --- evm/src/all_stark.rs | 4 +++- evm/src/memory/memory_stark.rs | 7 ++++--- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/evm/src/all_stark.rs b/evm/src/all_stark.rs index d9469ecc..c986ab86 100644 --- a/evm/src/all_stark.rs +++ b/evm/src/all_stark.rs @@ -195,7 +195,7 @@ mod tests { num_memory_ops: usize, memory_stark: &MemoryStark, rng: &mut R, - ) -> Vec> { + ) -> (Vec>, usize) { let memory_ops = generate_random_memory_ops(num_memory_ops, rng); memory_stark.generate_trace(memory_ops) } @@ -327,6 +327,8 @@ mod tests { let keccak_trace = make_keccak_trace(num_keccak_perms, &keccak_stark, &mut rng); let logic_trace = make_logic_trace(num_logic_rows, &logic_stark, &mut rng); let mem_trace = make_memory_trace(num_memory_ops, &memory_stark, &mut rng); + let mut memory_trace = mem_trace.0; + let num_memory_ops = mem_trace.1; let cpu_trace = make_cpu_trace( num_keccak_perms, num_logic_rows, diff --git a/evm/src/memory/memory_stark.rs b/evm/src/memory/memory_stark.rs index b485bc69..5f4e5599 100644 --- a/evm/src/memory/memory_stark.rs +++ b/evm/src/memory/memory_stark.rs @@ -226,7 +226,7 @@ pub fn generate_range_check_value( range_check.push(F::ZERO); - (range_check, max_timestamp_diff.try_into().unwrap()) + (range_check, max_timestamp_diff as usize) } impl, const D: usize> MemoryStark { @@ -352,7 +352,7 @@ impl, const D: usize> MemoryStark { } } - pub fn generate_trace(&self, memory_ops: Vec>) -> Vec> { + pub fn generate_trace(&self, memory_ops: Vec>) -> (Vec>, usize) { let mut timing = TimingTree::new("generate trace", log::Level::Debug); // Generate the witness. @@ -361,6 +361,7 @@ impl, const D: usize> MemoryStark { "generate trace rows", self.generate_trace_rows(memory_ops) ); + let num_ops = trace_rows.len(); let trace_polys = timed!( &mut timing, @@ -369,7 +370,7 @@ impl, const D: usize> MemoryStark { ); timing.print(); - trace_polys + (trace_polys, num_ops) } }