Use perfect hashing instead of string matching in SyscallHintProcessor::execute_syscall_hint

Cargo.toml

@@ -56,6 +56,8 @@ starknet = { workspace = true }
 base64 = { version = "0.21.0", default-features = false, features = ["alloc"] }
 flate2 = "1.0.25"
 serde_json_pythonic = { git = "https://github.com/xJonathanLEI/serde_json_pythonic", tag = "v0.1.2"}
+phf = "0.11"
+phf_macros = "0.11"
 
 [dev-dependencies]
 assert_matches = "1.5.0"
@@ -65,3 +67,6 @@ coverage-helper = "0.1.0"
 path = "bench/internals.rs"
 name = "internals"
 harness = false
+
+[build-dependencies]
+phf_codegen = "0.11"

rpc_state_reader/src/lib.rs

@@ -329,7 +329,6 @@ impl RpcState {
             "params": [self.block.to_value()],
             "id": 1
         });
-
         let block_info: serde_json::Value = self.rpc_call(&get_block_info_params).unwrap();
 
         starknet_in_rust::state::BlockInfo {
@@ -584,12 +583,9 @@ mod tests {
             RpcChain::TestNet2,
             BlockValue::Number(serde_json::to_value(838683).unwrap()),
         );
-
         let tx_hash_str = "19feb888a2d53ffddb7a1750264640afab8e9c23119e648b5259f1b5e7d51bc";
         let tx_hash = felt_str!(format!("{}", tx_hash_str), 16);
-
         let tx_trace = state_reader.get_transaction_trace(tx_hash);
-
         assert_eq!(
             tx_trace.signature,
             vec![
@@ -630,6 +626,7 @@ mod tests {
                 felt_str!("38bd34c31a0a5c", 16),
             ]
         );
+
         assert_eq!(tx_trace.validate_invocation.retdata, vec![]);
         assert_eq!(
             tx_trace.validate_invocation.execution_resources,
@@ -671,6 +668,7 @@ mod tests {
                 felt_str!("38bd34c31a0a5c", 16),
             ]
         );
+
         assert_eq!(tx_trace.function_invocation.retdata, vec![0.into()]);
         assert_eq!(
             tx_trace.function_invocation.execution_resources,
@@ -683,6 +681,7 @@ mod tests {
                 ]),
             }
         );
+
         assert_eq!(tx_trace.function_invocation.internal_calls.len(), 1);
         assert_eq!(
             tx_trace.function_invocation.internal_calls[0]
@@ -708,6 +707,7 @@ mod tests {
                 felt_str!("0", 16),
             ]
         );
+
         assert_eq!(tx_trace.fee_transfer_invocation.retdata, vec![1.into()]);
         assert_eq!(
             tx_trace.fee_transfer_invocation.execution_resources,
@@ -753,7 +753,9 @@ mod transaction_tests {
 
         // Instantiate the RPC StateReader and the CachedState
         let block = BlockValue::Number(serde_json::to_value(block_number).unwrap());
+
         let rpc_state = Arc::new(RpcState::new(network, block));
+
         let mut state = CachedState::new(rpc_state.clone(), HashMap::new());
 
         let fee_token_address = Address(felt_str!(
@@ -843,7 +845,6 @@ mod transaction_tests {
             838683,
             2917470325,
         );
-
         dbg!(&result.actual_resources);
         dbg!(&result.actual_fee); // test=7252831227950, explorer=7207614784695, diff=45216443255 (0.06%)
         dbg!(&result.call_info.clone().unwrap().execution_resources); // Ok with explorer
@@ -907,7 +908,6 @@ mod transaction_tests {
             155054,
             33977120598,
         );
-
         dbg!(&result.actual_resources);
         dbg!(&result.actual_fee); // test=6361070805216, explorer=47292465953700, diff=5888146145679 (0.13%)
         dbg!(&result.call_info.clone().unwrap().execution_resources); // Ok with explorer

src/lib.rs

@@ -25,6 +25,7 @@ use utils::Address;
 #[cfg(test)]
 #[macro_use]
 extern crate assert_matches;
+extern crate phf_macros;
 
 // Re-exports
 pub use crate::services::api::contract_classes::deprecated_contract_class::{