perf: skip unnecessary eth_getCode requests (#761)

- Disable eth_getCode request on every storage request
- Only fetch code when code hash is non-nullish in the account proof

Author: karen-sarkisyan

core/src/execution/providers/rpc.rs

@@ -7,7 +7,7 @@ use alloy::{
         primitives::HeaderResponse, BlockResponse, ReceiptResponse, TransactionBuilder,
         TransactionResponse,
     },
-    primitives::{Address, B256, U256},
+    primitives::{Address, Bytes, B256, U256},
     providers::{Provider, ProviderBuilder, RootProvider},
     rlp,
     rpc::{
@@ -16,7 +16,7 @@ use alloy::{
     },
     transports::layers::RetryBackoffLayer,
 };
-use alloy_trie::TrieAccount;
+use alloy_trie::{TrieAccount, KECCAK_EMPTY};
 use async_trait::async_trait;
 use eyre::{eyre, Result};
 use futures::future::{join_all, try_join_all};
@@ -224,9 +224,13 @@ impl<N: NetworkSpec, B: BlockProvider<N>, H: HistoricalBlockProvider<N>> Account
         verify_storage_proof(&proof)?;
 
         let code = if with_code {
-            let code = self.provider.get_code_at(address).await?;
-            verify_code_hash_proof(&proof, &code)?;
-            Some(code)
+            if proof.code_hash == KECCAK_EMPTY || proof.code_hash == B256::ZERO {
+                Some(Bytes::new())
+            } else {
+                let code = self.provider.get_code_at(address).await?;
+                verify_code_hash_proof(&proof, &code)?;
+                Some(code)
+            }
         } else {
             None
         };

revm-utils/src/proof_db.rs

@@ -82,7 +82,7 @@ impl<N: NetworkSpec, E: ExecutionProvider<N>> EvmState<N, E> {
                     let slot_bytes = B256::from(slot);
                     let account = self
                         .execution
-                        .get_account(address, &[slot_bytes], true, self.block)
+                        .get_account(address, &[slot_bytes], false, self.block)
                         .await?;
 
                     if let Some(stored_account) = self.accounts.get_mut(&address) {
@@ -124,6 +124,12 @@ impl<N: NetworkSpec, E: ExecutionProvider<N>> EvmState<N, E> {
             .and_then(|overrides| overrides.get(&address));
 
         if let Some(account) = self.accounts.get_mut(&address) {
+            let code_is_overriden = override_opt.and_then(|o| o.code.as_ref()).is_some();
+            if account.code.is_none() && !code_is_overriden {
+                self.access = Some(StateAccess::Basic(address));
+                return Err(DatabaseError::StateMissing);
+            }
+
             if let Some(override_opt) = override_opt {
                 apply_account_overrides(account, override_opt)?;
             }