[Webkit-unassigned] [Bug 16186] New: Reduce number of c-stack-based MatchFrames

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=16186

           Summary: Reduce number of c-stack-based MatchFrames
           Product: WebKit
           Version: 523.x (Safari 3)
          Platform: Macintosh
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: JavaScriptCore
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: eric at webkit.org
                CC: mjs at apple.com, darin at apple.com, ggaren at apple.com
 BugsThisDependsOn: 16185


Reduce number of c-stack-based MatchFrames

I believe the credit for this idea goes to Darin, I heard about it through
Maciej.  Currently we allocate space for 16 match frames every time we call
into match().  That's excessive, since very very very few regexps ever go
beyond a few frames.

Supposedly darin tried this and found that reducing to 4 frames was a big
speedup.  I expect we could probably reduce to 8 frames w/o worrying too much
about affecting real-world performance.  Until bug 16185 is fixed, we always
use 2 frames for every regexp. :(


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.