Remove evm debug tracer

lambdaclass · Aug 20, 2024 · c40ac2d · c40ac2d
1 parent 167c581
commit c40ac2d
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 126 deletions.
diff --git a/core/lib/multivm/src/versions/vm_latest/tracers/default_tracers.rs b/core/lib/multivm/src/versions/vm_latest/tracers/default_tracers.rs
@@ -13,7 +13,7 @@ use zk_evm_1_5_0::{
     zkevm_opcode_defs::{decoding::EncodingModeProduction, Opcode, RetOpcode},
 };
 
-use super::{EvmDebugTracer, EvmDeployTracer, PubdataTracer};
+use super::{EvmDeployTracer, PubdataTracer};
 use crate::{
     glue::GlueInto,
     interface::{
@@ -64,7 +64,7 @@ pub struct DefaultExecutionTracer<S: WriteStorage, H: HistoryMode> {
     // take into account e.g circuits produced by the initial bootloader memory commitment.
     pub(crate) circuits_tracer: CircuitsTracer<S, H>,
 
-    pub(crate) evm_tracer: Option<EvmDebugTracer<S, H>>,
+    // This tracer is responsible for handling EVM deployments and providing the data to the code decommitter.
     pub(crate) evm_deploy_tracer: EvmDeployTracer<S>,
 
     subversion: MultiVMSubversion,
@@ -98,7 +98,6 @@ impl<S: WriteStorage, H: HistoryMode> DefaultExecutionTracer<S, H> {
             circuits_tracer: CircuitsTracer::new(),
             evm_deploy_tracer: EvmDeployTracer::new(),
             storage,
-            evm_tracer: None,
             _phantom: PhantomData,
         }
     }
@@ -177,9 +176,6 @@ macro_rules! dispatch_tracers {
         if let Some(tracer) = &mut $self.pubdata_tracer {
             tracer.$function($( $params ),*);
         }
-        if let Some(tracer) = &mut $self.evm_tracer {
-            tracer.$function($( $params ),*);
-        }
         $self.circuits_tracer.$function($( $params ),*);
         $self.evm_deploy_tracer.$function($( $params ),*);
     };

diff --git a/core/lib/multivm/src/versions/vm_latest/tracers/evm_debug_tracer.rs b/core/lib/multivm/src/versions/vm_latest/tracers/evm_debug_tracer.rs
diff --git a/core/lib/multivm/src/versions/vm_latest/tracers/mod.rs b/core/lib/multivm/src/versions/vm_latest/tracers/mod.rs
@@ -1,14 +1,12 @@
 pub(crate) use circuits_tracer::CircuitsTracer;
 pub(crate) use default_tracers::DefaultExecutionTracer;
-pub(crate) use evm_debug_tracer::EvmDebugTracer;
 pub(crate) use evm_deploy_tracer::EvmDeployTracer;
 pub(crate) use pubdata_tracer::PubdataTracer;
 pub(crate) use refunds::RefundsTracer;
 pub(crate) use result_tracer::ResultTracer;
 
 pub(crate) mod circuits_tracer;
 pub(crate) mod default_tracers;
-pub(crate) mod evm_debug_tracer;
 pub(crate) mod evm_deploy_tracer;
 pub(crate) mod pubdata_tracer;
 pub(crate) mod refunds;

diff --git a/core/node/eth_watch/src/client.rs b/core/node/eth_watch/src/client.rs
@@ -70,7 +70,6 @@ impl EthHttpQueryClient {
             diamond_proxy_addr,
             governance_address
         );
-        dbg!(&state_transition_manager_contract());
         Self {
             client: client.for_component("watch"),
             topics: Vec::new(),