Bug 170569

Summary: Linear scan should run liveness only once
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, buildbot, cdumez, cmarcelo, dbates, ggaren, jfbastien, keith_miller, mark.lam, msaboff, saam
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
it's a start
none
the patch keith_miller: review+

Filip Pizlo
Reported 2017-04-06 15:01:13 PDT
Patch forthcoming.
Attachments
it's a start (36.18 KB, patch)
2017-04-06 15:01 PDT, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
the patch (42.64 KB, patch)
2017-04-06 16:50 PDT, Filip Pizlo 		keith_miller: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2017-04-06 15:01:50 PDT
Created attachment 306430 [details] it's a start
Filip Pizlo
Comment 2 2017-04-06 16:50:27 PDT
Created attachment 306437 [details] the patch another 10% -O1 compile time speed-up
Filip Pizlo
Comment 3 2017-04-06 17:06:57 PDT
(In reply to Filip Pizlo from comment #2) > Created attachment 306437 [details] > the patch > > another 10% -O1 compile time speed-up Actually, further testing shows that it's only a 6% speed-up. Still good though.
Keith Miller
Comment 4 2017-04-06 17:09:33 PDT
Comment on attachment 306437 [details] the patch r=me.
Filip Pizlo
Comment 5 2017-04-06 17:11:39 PDT
Landed in https://trac.webkit.org/changeset/215071/webkit
Note You need to log in before you can comment on or make changes to this bug.