fix log_pc

qwang98 · qwang98 · commit f7cfa9ef77fe · 2025-10-29T16:10:46.000+08:00
diff --git a/crates/vm/src/arch/interpreter.rs b/crates/vm/src/arch/interpreter.rs
@@ -592,7 +592,7 @@ unsafe fn execute_trampoline<F: PrimeField32, Ctx: ExecutionCtxTrait>(
         .as_ref()
         .is_ok_and(|exit_code| exit_code.is_none())
     {
-        exec_state.log_pc();
+        tracing::trace!(pc = self.pc, "executing instruction");
         if Ctx::should_suspend(instret, pc, arg, exec_state) {
             break;
         }
diff --git a/crates/vm/src/arch/state.rs b/crates/vm/src/arch/state.rs
@@ -57,11 +57,6 @@ impl<F: Clone, MEM> VmState<F, MEM> {
         }
     }
 
-    #[inline(always)]
-    pub fn log_pc(&self) {
-        tracing::trace!(pc = self.pc, "executing instruction");
-    }
-
     pub fn set_instret_and_pc(&mut self, instret: u64, pc: u32) {
         self.instret = instret;
         self.pc = pc;

Original file line number	Diff line number	Diff line change
`@@ -592,7 +592,7 @@ unsafe fn execute_trampoline<F: PrimeField32, Ctx: ExecutionCtxTrait>(`
`592`	`592`	`.as_ref()`
`593`	`593`	`.is_ok_and(\|exit_code\| exit_code.is_none())`
`594`	`594`	`{`
`595`		`- exec_state.log_pc();`
	`595`	`+ tracing::trace!(pc = self.pc, "executing instruction");`
`596`	`596`	`if Ctx::should_suspend(instret, pc, arg, exec_state) {`
`597`	`597`	`break;`
`598`	`598`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,11 +57,6 @@ impl<F: Clone, MEM> VmState<F, MEM> {`
`57`	`57`	`}`
`58`	`58`	`}`
`59`	`59`
`60`		`- #[inline(always)]`
`61`		`- pub fn log_pc(&self) {`
`62`		`- tracing::trace!(pc = self.pc, "executing instruction");`
`63`		`- }`
`64`		`-`
`65`	`60`	`pub fn set_instret_and_pc(&mut self, instret: u64, pc: u32) {`
`66`	`61`	`self.instret = instret;`
`67`	`62`	`self.pc = pc;`