perf(vm): Remove unnecessary check when calling closures

Author: Marwes

Cargo.toml

@@ -145,8 +145,8 @@ path = "examples/lisp/main.rs"
 [package.metadata.docs.rs]
 features = ["docs_rs"]
 
-# [profile.bench]
-# debug = 2
-#
-# [profile.release]
-# debug = 2
+[profile.bench]
+debug = 2
+
+[profile.release]
+debug = 2

vm/src/thread.rs

@@ -1815,22 +1815,18 @@ impl<'b> OwnedContext<'b> {
                 }) => {
                     let instruction_index = *instruction_index;
 
-                    if context.stack.stack().get_frames().len() == 0 {
-                        return Ok(Some(self)).into();
-                    } else {
-                        debug!(
-                            "Continue with {}\nAt: {}/{}\n{:?}",
-                            closure.function.name,
-                            instruction_index,
-                            closure.function.instructions.len(),
-                            &context.stack[..]
-                        );
+                    debug!(
+                        "Continue with {}\nAt: {}/{}\n{:?}",
+                        closure.function.name,
+                        instruction_index,
+                        closure.function.instructions.len(),
+                        &context.stack[..]
+                    );
 
-                        let closure_context = context.from_state();
-                        match ready!(closure_context.execute_())? {
-                            Some(new_context) => context = new_context,
-                            None => return Ok(None).into(),
-                        }
+                    let closure_context = context.from_state();
+                    match ready!(closure_context.execute_())? {
+                        Some(new_context) => context = new_context,
+                        None => return Ok(None).into(),
                     }
                 }
             };