From 95ba74d9342cd38edf57c3c3b2af43a70837f46e Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 28 Apr 2025 20:41:26 +0200 Subject: [PATCH] LibJS: Remove redundant VERIFY in run_executable() Getting the running_execution_context() already verifies that the execution context stack is non-empty, we don't need to do it separately here as well. --- Libraries/LibJS/Bytecode/Interpreter.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/Libraries/LibJS/Bytecode/Interpreter.cpp b/Libraries/LibJS/Bytecode/Interpreter.cpp index d7e6a4f39b0..960c85fcdb1 100644 --- a/Libraries/LibJS/Bytecode/Interpreter.cpp +++ b/Libraries/LibJS/Bytecode/Interpreter.cpp @@ -726,8 +726,6 @@ Interpreter::ResultAndReturnRegister Interpreter::run_executable(Executable& exe TemporaryChange restore_global_object { m_global_object, GC::Ptr { m_realm->global_object() } }; TemporaryChange restore_global_declarative_environment { m_global_declarative_environment, GC::Ptr { m_realm->global_environment().declarative_record() } }; - VERIFY(!vm().execution_context_stack().is_empty()); - auto& running_execution_context = vm().running_execution_context(); u32 registers_and_constants_and_locals_count = executable.number_of_registers + executable.constants.size() + executable.local_variable_names.size(); VERIFY(registers_and_constants_and_locals_count <= running_execution_context.registers_and_constants_and_locals_and_arguments_span().size());