Title: [209273] trunk/PerformanceTests
Revision
209273
Author
[email protected]
Date
2016-12-02 14:44:38 -0800 (Fri, 02 Dec 2016)

Log Message

ES6SampleBench should report an average for Steady State so that all of the numbers are comparable
https://bugs.webkit.org/show_bug.cgi?id=165325

Reviewed by Saam Barati.

This makes all of the numbers that ES6SampleBench reports comparable to each other: they all speak of the time
it took to run an iteration of something.

* ES6SampleBench/results.js:
(Results.prototype.reportResult):

Modified Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (209272 => 209273)


--- trunk/PerformanceTests/ChangeLog	2016-12-02 22:41:39 UTC (rev 209272)
+++ trunk/PerformanceTests/ChangeLog	2016-12-02 22:44:38 UTC (rev 209273)
@@ -1,3 +1,16 @@
+2016-12-02  Filip Pizlo  <[email protected]>
+
+        ES6SampleBench should report an average for Steady State so that all of the numbers are comparable
+        https://bugs.webkit.org/show_bug.cgi?id=165325
+
+        Reviewed by Saam Barati.
+
+        This makes all of the numbers that ES6SampleBench reports comparable to each other: they all speak of the time
+        it took to run an iteration of something.
+
+        * ES6SampleBench/results.js:
+        (Results.prototype.reportResult):
+
 2016-11-18  Jer Noble  <[email protected]>
 
         Add some performance tests for various aspects of media loading and playback.

Modified: trunk/PerformanceTests/ES6SampleBench/results.js (209272 => 209273)


--- trunk/PerformanceTests/ES6SampleBench/results.js	2016-12-02 22:41:39 UTC (rev 209272)
+++ trunk/PerformanceTests/ES6SampleBench/results.js	2016-12-02 22:44:38 UTC (rev 209273)
@@ -93,7 +93,7 @@
         this.firstIteration.add(times[0]);
         let steadyTimes = times.slice(1);
         this.averageWorstCase.add(averageAbovePercentile(steadyTimes, 98));
-        this.steadyState.add(steadyTimes.reduce((previous, current) => previous + current));
+        this.steadyState.add(steadyTimes.reduce((previous, current) => previous + current) / steadyTimes.length);
         this.reportDone();
     }
     
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to