Revert "Fix #81424: PCRE2 10.35 JIT performance regression"

This reverts commit a247138. Fixing the performance regression, apparently fixes a functional regression[1], so we revert for now. [1] <PCRE2Project/pcre2#21> Closes GH-7556.
php · Oct 5, 2021 · a19f655 · a19f655
1 parent 4c01bd2
commit a19f655
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/NEWS b/NEWS
@@ -17,9 +17,6 @@ PHP                                                                        NEWS
   . Fixed bug #61700 (FILTER_FLAG_IPV6/FILTER_FLAG_NO_PRIV|RES_RANGE failing).
     (cmb, Nikita)
 
-- PCRE:
-  . Fixed bug #81424 (PCRE2 10.35 JIT performance regression). (cmb)
-
 - SPL:
   . Fixed bug #80663 (Recursive SplFixedArray::setSize() may cause double-free).
     (cmb, Nikita, Tyson Andre)

diff --git a/ext/pcre/pcre2lib/pcre2_jit_compile.c b/ext/pcre/pcre2lib/pcre2_jit_compile.c
@@ -11152,7 +11152,7 @@ early_fail_type = (early_fail_ptr & 0x7);
 early_fail_ptr >>= 3;
 
 /* During recursion, these optimizations are disabled. */
-if (common->early_fail_start_ptr == 0 && common->fast_forward_bc_ptr == NULL)
+if (common->early_fail_start_ptr == 0)
   {
   early_fail_ptr = 0;
   early_fail_type = type_skip;