Bug 186477

Summary: Test js/regexp-zero-length-alternatives.html fails when RegExpJIT is disabled
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: JavaScriptCoreAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, ews-watchlist, fpizlo, keith_miller, mark.lam, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 185989    
Attachments:
Description Flags
Patch none

Description Michael Saboff 2018-06-10 22:30:16 PDT
A couple of subtests of subtests of js/regexp-zero-length-alternatives.html fail when using the YARR interpreter.

These subtests all involve the RegExp: /(?:a*?|b*?)*/.

var re32 = new RegExp(/(?:a*?|b*?)*/);
"aaaa".match(re32); // Hangs, should return "aaaa"
"abab".match(re32); // Hangs, should return "abab"
"ab".match(re32); // Returns "", should return "ab"
"xabx".match(re32); // Hangs, should return "ab"
Comment 1 Radar WebKit Bug Importer 2018-06-10 22:30:49 PDT
<rdar://problem/40992533>
Comment 2 Michael Saboff 2018-06-11 08:01:00 PDT
Created attachment 342429 [details]
Patch
Comment 3 WebKit Commit Bot 2018-06-11 09:32:27 PDT
Comment on attachment 342429 [details]
Patch

Clearing flags on attachment: 342429

Committed r232718: <https://trac.webkit.org/changeset/232718>
Comment 4 WebKit Commit Bot 2018-06-11 09:32:29 PDT
All reviewed patches have been landed.  Closing bug.