From feb02251f58fb320d00396d3153fc12ae625356b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aur=C3=A9lien=20Nicolas?= Date: Thu, 21 Nov 2024 13:00:53 +0100 Subject: [PATCH] feat/private-input: remove draft --- .../src/instructions/riscv/memory/store.rs | 1 + ceno_zkvm/src/tables/ram/ram_impl.rs | 17 ----------------- 2 files changed, 1 insertion(+), 17 deletions(-) diff --git a/ceno_zkvm/src/instructions/riscv/memory/store.rs b/ceno_zkvm/src/instructions/riscv/memory/store.rs index 18b134e57..c193e8ec6 100644 --- a/ceno_zkvm/src/instructions/riscv/memory/store.rs +++ b/ceno_zkvm/src/instructions/riscv/memory/store.rs @@ -65,6 +65,7 @@ impl Instruction ) -> Result { let rs1_read = UInt::new_unchecked(|| "rs1_read", circuit_builder)?; // unsigned 32-bit value let rs2_read = UInt::new_unchecked(|| "rs2_read", circuit_builder)?; + // Memory initialization is not guaranteed to contain u32. Decompose in [u16; 2] here. let prev_memory_value = UInt::new(|| "prev_memory_value", circuit_builder)?; let imm = circuit_builder.create_witin(|| "imm"); // signed 12-bit value diff --git a/ceno_zkvm/src/tables/ram/ram_impl.rs b/ceno_zkvm/src/tables/ram/ram_impl.rs index 06e69641b..47f665807 100644 --- a/ceno_zkvm/src/tables/ram/ram_impl.rs +++ b/ceno_zkvm/src/tables/ram/ram_impl.rs @@ -405,20 +405,3 @@ impl DynVolatileRamTableConfig Ok(final_table) } } - -#[allow(dead_code)] -/// DynUnConstrainRamTableConfig with unconstrain init value and final value -/// dynamic address as witin, relied on augment of knowledge to prove address form -/// do not check init_value -/// TODO implement DynUnConstrainRamTableConfig -#[derive(Clone, Debug)] -pub struct DynUnConstrainRamTableConfig { - addr: WitIn, - - init_v: Vec, - - final_v: Vec, - final_cycle: WitIn, - - phantom: PhantomData, -}