Bug 69207 - Bencher script makes it difficult to do automated performance testing
Summary: Bencher script makes it difficult to do automated performance testing
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 69208
  Show dependency tree / graph
 
Reported: 2011-10-01 00:40 PDT by Filip Pizlo
Modified: 2011-10-01 15:01 PDT (History)
2 users (show)

See Also:

Attachments
the patch (12.88 KB, patch)
2011-10-01 00:50 PDT, Filip Pizlo 		sam: review+
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 Filip Pizlo 2011-10-01 00:40:44 PDT 
The Tools/Scripts/bencher script simplifies performance testing.  Sometimes it would be useful to use bencher itself within a script, and read out the results - for example when varying some configuration parameter of a particular build of a VM.  But bencher has no way of computing a single aggregate result for all benchmark suites, and no way of printing it.  It also gets confused if a VM is modified to print additional output beyond jsc's print statement, which makes it impossible to use bencher with certain profiling features enabled.
Comment 1 Filip Pizlo 2011-10-01 00:50:40 PDT 
Created attachment 109402 [details]
the patch
Comment 2 Chris Rogers 2011-10-01 10:48:52 PDT 
Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?
Comment 3 Sam Weinig 2011-10-01 13:24:14 PDT 
(In reply to comment #2)
> Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?

My guess is off by one from Tools/Tests.  Fixed.
Comment 4 Filip Pizlo 2011-10-01 14:52:42 PDT 
(In reply to comment #3)
> (In reply to comment #2)
> > Hi Filip, I'm not sure why this issue is associated with the "Web Audio" component?
> 
> My guess is off by one from Tools/Tests.  Fixed.

Thanks!
Comment 5 Filip Pizlo 2011-10-01 15:01:59 PDT 
Landed in r96462.