Skip to content

Commit 240beb6

Browse files
committed
wip3
1 parent 33679c0 commit 240beb6

File tree

3 files changed

+9
-9
lines changed

3 files changed

+9
-9
lines changed

extensions/native/circuit/src/extension/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ impl<F: PrimeField32> VmExecutionExtension<F> for Native {
179179
inventory.add_executor(
180180
tower_verify,
181181
[SumcheckOpcode::SUMCHECK_LAYER_EVAL.global_opcode()],
182-
);
182+
)?;
183183

184184
inventory.add_phantom_sub_executor(
185185
NativeHintInputSubEx,

extensions/native/circuit/src/sumcheck/execution.rs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,7 @@ unsafe fn execute_e12_impl<F: PrimeField32, CTX: ExecutionCtxTrait>(
215215
let c1: [F; EXT_DEG] = challenges[EXT_DEG..EXT_DEG * 2].try_into().unwrap();
216216
let c2: [F; EXT_DEG] = challenges[EXT_DEG * 2..EXT_DEG * 3].try_into().unwrap();
217217

218-
let mut height = 0;
218+
let mut height = 1;
219219
let mut alpha_acc = elem_to_ext(F::ONE);
220220
let mut eval_acc = elem_to_ext(F::ZERO);
221221

@@ -265,8 +265,8 @@ unsafe fn execute_e12_impl<F: PrimeField32, CTX: ExecutionCtxTrait>(
265265
.vm_read(NATIVE_AS, ctx_ptr_u32 + 8 + num_prod_spec + i)
266266
.map(|x: F| x.as_canonical_u32());
267267
let start = calculate_3d_ext_idx(
268-
prod_specs_inner_inner_len,
269-
prod_specs_inner_len,
268+
logup_specs_inner_len,
269+
logup_specs_inner_inner_len,
270270
i,
271271
round,
272272
0,
@@ -330,6 +330,9 @@ unsafe fn execute_e12_impl<F: PrimeField32, CTX: ExecutionCtxTrait>(
330330
height += 1;
331331
}
332332

333+
*pc += DEFAULT_PC_STEP;
334+
*instret += 1;
335+
333336
exec_state.vm_write(NATIVE_AS, r_evals_ptr_u32, &eval_acc);
334337
// return height delta
335338
height

extensions/native/recursion/tests/sumcheck.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ use openvm_stark_sdk::{
3030
},
3131
engine::StarkFriEngine,
3232
p3_baby_bear::BabyBear,
33-
utils::{create_seeded_rng, ProofInputForTest},
3433
};
3534

3635
pub type F = BabyBear;
@@ -42,15 +41,11 @@ fn test_sumcheck_layer_eval() {
4241

4342
build_test_program(&mut builder);
4443

45-
// Fill in test program logic
46-
builder.halt();
47-
4844
let compilation_options = CompilerOptions::default().with_cycle_tracker();
4945
let mut compiler = AsmCompiler::new(compilation_options.word_size);
5046
compiler.build(builder.operations);
5147
let asm_code = compiler.code();
5248

53-
// let program = Program::from_instructions(&instructions);
5449
let program: Program<_> = convert_program(asm_code, compilation_options);
5550
let sumcheck_max_constraint_degree = 3;
5651
let fri_params = if matches!(std::env::var("OPENVM_FAST_TEST"), Ok(x) if &x == "1") {
@@ -1268,4 +1263,6 @@ fn build_test_program<C: Config>(builder: &mut Builder<C>) {
12681263
&logup_spec_evals,
12691264
&next_layer_evals,
12701265
);
1266+
1267+
builder.halt();
12711268
}

0 commit comments

Comments
 (0)