Title: [224485] trunk/PerformanceTests
Revision
224485
Author
[email protected]
Date
2017-11-06 04:56:51 -0800 (Mon, 06 Nov 2017)

Log Message

PerformanceTests/TailBench9000/merge-sort.js does not actually sort any of the large arrays it allocates
https://bugs.webkit.org/show_bug.cgi?id=178817

Reviewed by Saam Barati.

* TailBench9000/merge-sort.js:
(TEST_mergeSort):

Modified Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (224484 => 224485)


--- trunk/PerformanceTests/ChangeLog	2017-11-06 08:31:44 UTC (rev 224484)
+++ trunk/PerformanceTests/ChangeLog	2017-11-06 12:56:51 UTC (rev 224485)
@@ -1,3 +1,13 @@
+2017-11-06  Robin Morisset  <[email protected]>
+
+        PerformanceTests/TailBench9000/merge-sort.js does not actually sort any of the large arrays it allocates
+        https://bugs.webkit.org/show_bug.cgi?id=178817
+
+        Reviewed by Saam Barati.
+
+        * TailBench9000/merge-sort.js:
+        (TEST_mergeSort):
+
 2017-11-02  Jon Lee  <[email protected]>
 
         Add license for MotionMark

Modified: trunk/PerformanceTests/TailBench9000/merge-sort.js (224484 => 224485)


--- trunk/PerformanceTests/TailBench9000/merge-sort.js	2017-11-06 08:31:44 UTC (rev 224484)
+++ trunk/PerformanceTests/TailBench9000/merge-sort.js	2017-11-06 12:56:51 UTC (rev 224485)
@@ -134,8 +134,6 @@
     let arrays = [
         buildArray(10, x => x),
         buildArray(10, x => -x),
-        buildArray(1000, x => x),
-        buildArray(10000, x => -x),
         buildArray(10000, x => random())
     ];
     
@@ -148,10 +146,11 @@
         let sorted = mergeSorted(array, (a, b) => a < b ? -1 : a > b ? 1 : 0);
         checkSorted(sorted);
         checkSpectrum(array, sorted);
+        test(index + 1)
     }
     
     test(0);
 }
 
-for (var i = 0; i < 3000; ++i)
+for (var i = 0; i < 100; ++i)
     TEST_mergeSort();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to