From 07ab0c197438b0275757ec3880a2b85a8f7a3ad2 Mon Sep 17 00:00:00 2001
From: Mario Rugiero <mrugiero@gmail.com>
Date: Tue, 4 Jul 2023 18:33:30 -0300
Subject: [PATCH] perf: avoid iterators for keeping rc limits

---
 CHANGELOG.md         |  2 ++
 vm/src/vm/vm_core.rs | 11 +++--------
 2 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 8a55baf3ac..0558e9f7ce 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,8 @@
 
 #### Upcoming Changes
 
+* perf: remove pointless iterator from rc limits tracking [#1316](https://github.com/lambdaclass/cairo-vm/pull/1316)
+
 * fix: specify resolver version 2 in the virtual workspace's manifest [#1311](https://github.com/lambdaclass/cairo-vm/pull/1311)
 
 * feat: add `lambdaworks-felt` feature to `cairo-vm-cli` [#1308](https://github.com/lambdaclass/cairo-vm/pull/1308)
diff --git a/vm/src/vm/vm_core.rs b/vm/src/vm/vm_core.rs
index c34244308c..f11c949714 100644
--- a/vm/src/vm/vm_core.rs
+++ b/vm/src/vm/vm_core.rs
@@ -405,15 +405,10 @@ impl VirtualMachine {
             instruction.off1 + (1_isize << (OFFSET_BITS - 1)),
             instruction.off2 + (1_isize << (OFFSET_BITS - 1)),
         );
+        let (min, max) = self.rc_limits.unwrap_or((off0, off0));
         self.rc_limits = Some((
-            [self.rc_limits.unwrap_or((off0, off0)).0, off0, off1, off2]
-                .into_iter()
-                .min()
-                .unwrap(),
-            [self.rc_limits.unwrap_or((off0, off0)).1, off0, off1, off2]
-                .into_iter()
-                .max()
-                .unwrap(),
+            min.min(off0).min(off1).min(off2),
+            max.max(off0).max(off1).max(off2),
         ));
 
         self.segments