RESOLVED FIXED Bug 38898
Remove unnecessary jump in Yarr JIT's code 
https://bugs.webkit.org/show_bug.cgi?id=38898
Summary Remove unnecessary jump in Yarr JIT's code
Peter Varga
Reported 2010-05-11 05:03:20 PDT
The Yarr JIT generates an unnecessary jump in case of greedy character and characterclass matching when it checks the count of repetitions. It should be removed, because it is followed by a direct jump instead of a branch if the greedy term's quantityCount doesn't have max limit.
Attachments
proposed patch (2.53 KB, patch)
2010-05-11 05:08 PDT, Peter Varga 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Peter Varga
Comment 1 2010-05-11 05:08:29 PDT
Created attachment 55689 [details] proposed patch
Zoltan Horvath
Comment 2 2010-05-12 04:40:25 PDT
Comment on attachment 55689 [details] proposed patch Setting cq+.
Zoltan Horvath
Comment 3 2010-05-12 04:49:50 PDT
Comment on attachment 55689 [details] proposed patch Clearing flags on attachment: 55689 Committed r59222: <http://trac.webkit.org/changeset/59222>
Zoltan Horvath
Comment 4 2010-05-12 04:49:59 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.