Bladeren bron

LibRegex: Remove redundant VERIFY in OpCode::argument()

The bounds check here is already performed by the at() we're calling,
and removing it appears to have non-trivial performance impact.
Andreas Kling 2 jaren geleden
bovenliggende
commit
17bff999c8
1 gewijzigde bestanden met toevoegingen van 0 en 1 verwijderingen
  1. 0 1
      Userland/Libraries/LibRegex/RegexByteCode.h

+ 0 - 1
Userland/Libraries/LibRegex/RegexByteCode.h

@@ -559,7 +559,6 @@ public:
 
     ALWAYS_INLINE ByteCodeValueType argument(size_t offset) const
     {
-        VERIFY(state().instruction_position + offset <= m_bytecode->size());
         return m_bytecode->at(state().instruction_position + 1 + offset);
     }