From d0d87d3aedaac8ab178f58ffdd0805c6c09e3d7b Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 28 Apr 2025 01:53:44 +0200 Subject: [PATCH] LibJS: Demote some overly paranoid VERIFY()s in ESFO [[Call]] flow --- Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp b/Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp index c662f34580a..357f03decdb 100644 --- a/Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp +++ b/Libraries/LibJS/Runtime/ECMAScriptFunctionObject.cpp @@ -503,7 +503,7 @@ ThrowCompletionOr ECMAScriptFunctionObject::internal_call(ExecutionContex { auto& vm = this->vm(); - VERIFY(m_bytecode_executable); + ASSERT(m_bytecode_executable); // 1. Let callerContext be the running execution context. // NOTE: No-op, kept by the VM in its execution context stack. @@ -513,7 +513,7 @@ ThrowCompletionOr ECMAScriptFunctionObject::internal_call(ExecutionContex TRY(prepare_for_ordinary_call(vm, callee_context, nullptr)); // 3. Assert: calleeContext is now the running execution context. - VERIFY(&vm.running_execution_context() == &callee_context); + ASSERT(&vm.running_execution_context() == &callee_context); // 4. If F.[[IsClassConstructor]] is true, then if (is_class_constructor()) { @@ -543,7 +543,7 @@ ThrowCompletionOr ECMAScriptFunctionObject::internal_call(ExecutionContex return result.value(); // 9. Assert: result is a throw completion. - VERIFY(result.type() == Completion::Type::Throw); + ASSERT(result.type() == Completion::Type::Throw); // 10. Return ? result. return result.release_error();