Bug 122333

Summary: Delete baseline optimizer
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, bfulgham, commit-queue, dpranke, glenn, kling, ossy, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
The baseline optimizer and its dependencies are removed. none

Ryosuke Niwa
Reported 2013-10-04 11:26:39 PDT
At this point, we only have 4 ports that run layout tests. We should get rid of this broken feature from webkitpy. It only confuses hell out of me and other contributors because the current behavior is almost a random shuffling of expected results.
Attachments
The baseline optimizer and its dependencies are removed. (32.53 KB, patch)
2013-11-20 08:28 PST, Berta József 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Berta József
Comment 1 2013-11-20 08:28:20 PST
Created attachment 217437 [details] The baseline optimizer and its dependencies are removed.
Zan Dobersek
Comment 2 2013-11-20 08:44:54 PST
akling was using webkit-patch optimize-baselines lately, I think with success. Is this really good to delete? I guess it's a bit bloat-y, but presumably it still works, and is used.
Ryosuke Niwa
Comment 3 2013-11-20 16:25:02 PST
(In reply to comment #2) > akling was using webkit-patch optimize-baselines lately, I think with success. > > Is this really good to delete? I guess it's a bit bloat-y, but presumably it still works, and is used. It's actually broken in some unknown ways :(
WebKit Commit Bot
Comment 4 2013-11-20 16:51:44 PST
Comment on attachment 217437 [details] The baseline optimizer and its dependencies are removed. Clearing flags on attachment: 217437 Committed r159595: <http://trac.webkit.org/changeset/159595>
WebKit Commit Bot
Comment 5 2013-11-20 16:51:46 PST
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.