WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
122333
Delete baseline optimizer
https://bugs.webkit.org/show_bug.cgi?id=122333
Summary
Delete baseline optimizer
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
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug