Changeset 253008 in webkit for trunk/Source/JavaScriptCore/tools

Timestamp:

Dec 2, 2019, 2:16:42 PM (5 years ago)

Author:

[email protected]

Message:

mozilla-tests.yaml/js1_5/Array/regress-101964.js is frequently failing on JSC EWS bots.
​https://bugs.webkit.org/show_bug.cgi?id=200789
<rdar://problem/54361916>

Reviewed by Keith Miller.

JSTests:

The prevailing theory is that this test is being pre-empted and not getting the
CPU time it needs to complete. As a result, the wall clock time period for
running the test exceeds the expected time. This patch tests this theory by
changing the time measurement to use CPU time instead.

• mozilla/js1_5/Array/regress-101964.js:

Source/JavaScriptCore:

• tools/JSDollarVM.cpp:

(JSC::functionCurrentCPUTime):
(JSC::JSDollarVM::finishCreation):

File:

• trunk/Source/JavaScriptCore/tools/JSDollarVM.cpp (modified) (3 diffs)

trunk/Source/JavaScriptCore/tools/JSDollarVM.cpp

@@ -54 +54 @@
 #include "WasmCapabilities.h"
 #include <wtf/Atomics.h>
+#include <wtf/CPUTime.h>
 #include <wtf/DataLog.h>
 #include <wtf/ProcessID.h>
@@ -2676 +2677 @@
 }
 
+static EncodedJSValue JSC_HOST_CALL functionCurrentCPUTime(JSGlobalObject*, CallFrame*)
+{
+    DollarVMAssertScope assertScope;
+    return JSValue::encode(jsNumber(CPUTime::forCurrentThread().value()));
+}
+
 static EncodedJSValue JSC_HOST_CALL functionTotalGCTime(JSGlobalObject* globalObject, CallFrame*)
 {
@@ -2833 +2840 @@
     addFunction(vm, "deltaBetweenButterflies", functionDeltaBetweenButterflies, 2);
     
+    addFunction(vm, "currentCPUTime", functionCurrentCPUTime, 0);
     addFunction(vm, "totalGCTime", functionTotalGCTime, 0);