Changeset 34386 in webkit for trunk/JavaScriptCore/pcre

Timestamp:

Jun 5, 2008, 3:40:23 PM (17 years ago)

Author:

[email protected]

Message:

2008-06-05 Geoffrey Garen <​[email protected]>

Reviewed by Stephanie Lewis (or maybe the other way around).

Minor change to PCRE to help out certain compilers.

SunSpider reports no change, maybe a small speedup.

• pcre/pcre_exec.cpp: (match): Use instructionPtr++ a little less, to avoid confusing the optimizer.

File:

• trunk/JavaScriptCore/pcre/pcre_exec.cpp (modified) (1 diff)

trunk/JavaScriptCore/pcre/pcre_exec.cpp

@@ -1243 +1243 @@
                 if (stack.currentFrame->args.subjectPtr >= md.endSubject)
                     RRETURN_NO_MATCH;
-                stack.currentFrame->args.instructionPtr++;
+                int b = stack.currentFrame->args.instructionPtr[1];
                 int c = *stack.currentFrame->args.subjectPtr++;
+                stack.currentFrame->args.instructionPtr += 2;
                 if (md.ignoreCase) {
                     if (c < 128)
                         c = toLowerCase(c);
-                    if (toLowerCase(*stack.currentFrame->args.instructionPtr++) == c)
+                    if (toLowerCase(b) == c)
                         RRETURN_NO_MATCH;
                 } else {
-                    if (*stack.currentFrame->args.instructionPtr++ == c)
+                    if (b == c)
                         RRETURN_NO_MATCH;
                 }