Add rought version of VirtualMachineError::Hint

lambdaclass · Dec 29, 2022 · a9045fa · a9045fa · github-actions · Dec 29, 2022
1 parent bd6111a
commit a9045fa
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/vm/errors/vm_errors.rs b/src/vm/errors/vm_errors.rs
@@ -240,4 +240,6 @@ pub enum VirtualMachineError {
     InvalidArgCount(usize, usize),
     #[error("{0}, {1}")]
     ErrorMessageAttribute(String, Box<VirtualMachineError>),
+    #[error("{1}")]
+    Hint(usize, Box<VirtualMachineError>),
 }
diff --git a/src/vm/vm_core.rs b/src/vm/vm_core.rs
@@ -525,8 +525,10 @@ impl VirtualMachine {
         constants: &HashMap<String, BigInt>,
     ) -> Result<(), VirtualMachineError> {
         if let Some(hint_list) = hint_data_dictionary.get(&self.run_context.pc.offset) {
-            for hint_data in hint_list.iter() {
-                hint_executor.execute_hint(self, exec_scopes, hint_data, constants)?
+            for (hint_index, hint_data) in hint_list.iter().enumerate() {
+                hint_executor
+                    .execute_hint(self, exec_scopes, hint_data, constants)
+                    .map_err(|err| VirtualMachineError::Hint(hint_index, Box::new(err)))?
             }
         }
         Ok(())
Benchmark suite	Current: `a9045fa`	Previous: `34e2a4a`	Ratio
`cairo_run(cairo_programs/benchmarks/memory_integration_benchmark.json`	`1431519588` ns/iter (`± 66079514`)	`1098971820` ns/iter (`± 8313907`)	`1.30`
`cairo_run(cairo_programs/benchmarks/uint256_integration_benchmark.json`	`3321351738` ns/iter (`± 69346913`)	`2491213460` ns/iter (`± 6794363`)	`1.33`