[Pcre-svn] [822] code/trunk/src/pcre2_jit_compile.c: Improve…

Top Page
Delete this message
Author: Subversion repository
Date:  
To: pcre-svn
Subject: [Pcre-svn] [822] code/trunk/src/pcre2_jit_compile.c: Improve char range match in JIT.
Revision: 822
          http://www.exim.org/viewvc/pcre2?view=rev&revision=822
Author:   zherczeg
Date:     2017-06-08 13:07:05 +0100 (Thu, 08 Jun 2017)
Log Message:
-----------
Improve char range match in JIT.


Modified Paths:
--------------
    code/trunk/src/pcre2_jit_compile.c


Modified: code/trunk/src/pcre2_jit_compile.c
===================================================================
--- code/trunk/src/pcre2_jit_compile.c    2017-06-06 11:32:25 UTC (rev 821)
+++ code/trunk/src/pcre2_jit_compile.c    2017-06-08 12:07:05 UTC (rev 822)
@@ -5783,9 +5783,10 @@
 uint8_t byte;
 sljit_s32 type;
 int i, j, k, len, c;
-struct sljit_jump *jump;
-jump_list *found = NULL;


+if (!sljit_has_cpu_feature(SLJIT_HAS_CMOV))
+ return FALSE;
+
if (invert)
nclass = !nclass;

@@ -5831,23 +5832,28 @@
     }
   }


-jump = NULL;
+i = 0;
j = 0;

-for (i = 0; i < len; i++)
+if (char_list[0] == 0)
   {
+  i++;
+  OP2(SLJIT_SUB | SLJIT_SET_Z, SLJIT_UNUSED, 0, TMP1, 0, SLJIT_IMM, 0);
+  OP_FLAGS(SLJIT_MOV, TMP2, 0, SLJIT_ZERO);
+  }
+else
+  OP1(SLJIT_MOV, TMP2, 0, SLJIT_IMM, 0);
+
+while (i < len)
+  {
   if ((char_list[i] & 0x100) != 0)
     j++;
-  else 
+  else
     {
-    type = SLJIT_EQUAL;
-    if (!nclass && j == 0 && i + 1 == len)
-      type = SLJIT_NOT_EQUAL;
-
-    jump = CMP(type, TMP1, 0, SLJIT_IMM, char_list[i]);
-
-    add_jump(compiler, (nclass || type == SLJIT_NOT_EQUAL) ? backtracks : &found, jump);
+    OP2(SLJIT_SUB | SLJIT_SET_Z, SLJIT_UNUSED, 0, TMP1, 0, SLJIT_IMM, char_list[i]);
+    CMOV(SLJIT_ZERO, TMP2, TMP1, 0);
     }
+  i++;
   }


 if (j != 0)
@@ -5858,17 +5864,13 @@
     if ((char_list[i] & 0x100) != 0)
       {
       j--;
-
-      type = SLJIT_EQUAL;
-      if (!nclass && j == 0)
-        type = SLJIT_NOT_EQUAL;
-
-      jump = CMP(type, TMP1, 0, SLJIT_IMM, char_list[i] & 0xff);
-      add_jump(compiler, (nclass || type == SLJIT_NOT_EQUAL) ? backtracks : &found, jump);
+      OP2(SLJIT_SUB | SLJIT_SET_Z, SLJIT_UNUSED, 0, TMP1, 0, SLJIT_IMM, char_list[i] & 0xff);
+      CMOV(SLJIT_ZERO, TMP2, TMP1, 0);
       }
   }


-set_jumps(found, LABEL());
+type = nclass ? SLJIT_NOT_EQUAL : SLJIT_EQUAL;
+add_jump(compiler, backtracks, CMP(type, TMP2, 0, SLJIT_IMM, 0));
return TRUE;
}