Bug 142874

Summary:

run-jsc-benchmarks should only report preferred mean

Product:

WebKit

Reporter:

Ryosuke Niwa <rniwa>

Component:

Tools / Tests

Assignee:

Ryosuke Niwa <rniwa>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

barraclough, fpizlo, ggaren, msaboff

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Fixes the bug	none

Ryosuke Niwa

Reported 2015-03-19 13:03:38 PDT

Don't report non-preferred means.

Attachments
Fixes the bug (2.34 KB, patch) 2015-03-19 13:05 PDT, Ryosuke Niwa	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Ryosuke Niwa

Comment 1 2015-03-19 13:05:42 PDT

Created attachment 249052 [details] Fixes the bug

Geoffrey Garen

Comment 2 2015-03-19 15:19:35 PDT

Comment on attachment 249052 [details] Fixes the bug r=me Does the output text still specify the name of the mean (harmonic, geometric, arithmetic)? I think it does -- just want to double-check.

Ryosuke Niwa

Comment 3 2015-03-19 15:42:26 PDT

(In reply to comment #2) > Comment on attachment 249052 [details] > Fixes the bug > > r=me > > Does the output text still specify the name of the mean (harmonic, > geometric, arithmetic)? I think it does -- just want to double-check. Yes. It'll be like <artihmetic> just without * since there will be exactly one mean per test.

Ryosuke Niwa

Comment 4 2015-03-19 15:47:31 PDT

Comment on attachment 249052 [details] Fixes the bug Clearing flags on attachment: 249052 Committed r181764: <http://trac.webkit.org/changeset/181764>

Ryosuke Niwa

Comment 5 2015-03-19 15:47:35 PDT

All reviewed patches have been landed. Closing bug.

Ryosuke Niwa

Comment 6 2015-03-19 17:54:26 PDT

Fixed the script in r181774.

Note You need to log in before you can comment on or make changes to this bug.