Bug 172326 - Add SixSpeed benchmark to PerformanceTests
Summary: Add SixSpeed benchmark to PerformanceTests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Yusuke Suzuki
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-05-18 17:43 PDT by Yusuke Suzuki
Modified: 2017-05-19 06:59 PDT (History)
5 users (show)

See Also:

Attachments
Patch (41.47 KB, patch)
2017-05-19 01:26 PDT, Yusuke Suzuki 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Yusuke Suzuki 2017-05-18 17:43:14 PDT 
While arewefastyet repository's SixSpeed does not include license text, the original SixSpeed is released under MIT license.
We can just import it to WebKit tree.

Maybe, adjusting the wrapper script to be fit to the original repository is necessary (since the current wrapper script is written for the version in arewefastyet).
Comment 1 Yusuke Suzuki 2017-05-18 17:43:43 PDT 
https://github.com/kpdecker/six-speed
Comment 2 Yusuke Suzuki 2017-05-19 01:26:44 PDT 
Created attachment 310633 [details]
Patch
Comment 3 Yusuke Suzuki 2017-05-19 06:30:17 PDT 
Comment on attachment 310633 [details]
Patch

Thanks, let's land it!
Comment 4 WebKit Commit Bot 2017-05-19 06:59:29 PDT 
Comment on attachment 310633 [details]
Patch

Clearing flags on attachment: 310633

Committed r217124: <http://trac.webkit.org/changeset/217124>
Comment 5 WebKit Commit Bot 2017-05-19 06:59:31 PDT 
All reviewed patches have been landed.  Closing bug.