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

Description Ryosuke Niwa 2015-03-19 13:03:38 PDT

Don't report non-preferred means.

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

Created attachment 249052 [details]
Fixes the bug

Comment 2 Geoffrey Garen 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.

Comment 3 Ryosuke Niwa 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.

Comment 4 Ryosuke Niwa 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>

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

All reviewed patches have been landed.  Closing bug.

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

Fixed the script in r181774.