REGRESSION: 3 stress/op_mod (and op_div) tests timing out on Debug JSC bots.
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Sep 2017 19:46:41 +0000 (19:46 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Sep 2017 19:46:41 +0000 (19:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=176630

Reviewed by JF Bastien.

Debug builds are just slow, and these tests do a lot.  They pass when I run them
locally on my MacBook Pro.  So, I'm bumping their timing multiplier to 2.0x as
a speculative fix for the bots that are seeing these fail.

I also undid the skipping of the op_mod tests for debug builds.

* stress/op_div-ConstVar.js:
* stress/op_div-VarConst.js:
* stress/op_div-VarVar.js:
* stress/op_mod-ConstVar.js:
* stress/op_mod-VarConst.js:
* stress/op_mod-VarVar.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@221935 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JSTests/ChangeLog
JSTests/stress/op_div-ConstVar.js
JSTests/stress/op_div-VarConst.js
JSTests/stress/op_div-VarVar.js
JSTests/stress/op_mod-ConstVar.js
JSTests/stress/op_mod-VarConst.js
JSTests/stress/op_mod-VarVar.js

index 3aaa4f3..bd19cd2 100644 (file)
@@ -1,3 +1,23 @@
+2017-09-12  Mark Lam  <mark.lam@apple.com>
+
+        REGRESSION: 3 stress/op_mod (and op_div) tests timing out on Debug JSC bots.
+        https://bugs.webkit.org/show_bug.cgi?id=176630
+
+        Reviewed by JF Bastien.
+
+        Debug builds are just slow, and these tests do a lot.  They pass when I run them
+        locally on my MacBook Pro.  So, I'm bumping their timing multiplier to 2.0x as
+        a speculative fix for the bots that are seeing these fail.
+
+        I also undid the skipping of the op_mod tests for debug builds.
+
+        * stress/op_div-ConstVar.js:
+        * stress/op_div-VarConst.js:
+        * stress/op_div-VarVar.js:
+        * stress/op_mod-ConstVar.js:
+        * stress/op_mod-VarConst.js:
+        * stress/op_mod-VarVar.js:
+
 2017-09-12  Ryan Haddad  <ryanhaddad@apple.com>
 
         Skip stress/value-to-boolean.js on Debug bots.
index 0800093..7dc565c 100644 (file)
@@ -1,4 +1,4 @@
-//@ runFTLNoCJIT("--timeoutMultiplier=1.5")
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.
index 06a029e..72eb2b5 100644 (file)
@@ -1,4 +1,4 @@
-//@ runFTLNoCJIT("--timeoutMultiplier=1.5")
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.
index 6ff0d22..05fd85f 100644 (file)
@@ -1,4 +1,4 @@
-//@ runFTLNoCJIT("--timeoutMultiplier=1.5")
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.
index 794ef05..b54e020 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "release" then runFTLNoCJIT("--timeoutMultiplier=1.5") else skip end
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.
index 406e0e5..2144ac0 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "release" then runFTLNoCJIT("--timeoutMultiplier=1.5") else skip end
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.
index 3110733..57aea24 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "release" then runFTLNoCJIT("--timeoutMultiplier=1.5") else skip end
+//@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
 // errors. See binary-op-test.js for debugging options if needed.