LibJS: Demote VERIFYs in Completion() to ASSERT

These were *extremely* hot in profiles (noticed when looking at
disassembly).

Now that we've made the special empty JS::Value much harder to create
accidentally, we can feel better about turning these into ASSERT and
catching them in debug builds.
This commit is contained in:
Andreas Kling 2025-04-04 23:23:39 +02:00
parent 00cc1bedd7
commit 90c0c4bee3

View file

@ -64,8 +64,8 @@ public:
: m_type(type)
, m_value(value)
{
VERIFY(type != Type::Empty);
VERIFY(!value.is_special_empty_value());
ASSERT(type != Type::Empty);
ASSERT(!value.is_special_empty_value());
}
Completion(ThrowCompletionOr<Value> const&);