Bug 202189

Summary: [JSC] We should handle RegExpConstructor in DFG
Product: WebKit Reporter: Yusuke Suzuki <ysuzuki>
Component: JavaScriptCoreAssignee: Yusuke Suzuki <ysuzuki>
Status: NEW    
Severity: Normal CC: ews-watchlist, keith_miller, mark.lam, msaboff, saam, tzagallo
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Yusuke Suzuki
Reported 2019-09-25 00:56:22 PDT
JetStream2/string-unpack-code-SP's hottest part is `new RegExp(...)` where parameters are already proven (LazyJSConstant string and String constant). We can completely eliminate this thing's RegExp allocation (RegExpObject could be allocated, but RegExp can be cached if we handle RegExpConstructor InternalFunction call in DFG).
Attachments
Patch (38.40 KB, patch)
2019-09-29 01:00 PDT, Yusuke Suzuki 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yusuke Suzuki
Comment 1 2019-09-29 01:00:29 PDT
Created attachment 379806 [details] Patch
Note You need to log in before you can comment on or make changes to this bug.