Changeset 201158 in webkit

Timestamp:

May 19, 2016 5:03:16 AM (8 years ago)

Author:

Csaba Osztrogonác

Message:

Slow JSC stress tests times out in 32 bit debug mode
​https://bugs.webkit.org/show_bug.cgi?id=154964

Reviewed by Michael Catanzaro.

• Scripts/run-javascriptcore-tests:

(runJSCStressTests): Pass through the --debug option.

• Scripts/run-jsc-stress-tests: Increase the timeout for debug builds.

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/run-javascriptcore-tests (modified) (1 diff)
• Scripts/run-jsc-stress-tests (modified) (1 diff)

trunk/Tools/ChangeLog

@@ +1 @@
+2016-05-19  Csaba Osztrogonác  <ossy@webkit.org>
+
+        Slow JSC stress tests times out in 32 bit debug mode
+        https://bugs.webkit.org/show_bug.cgi?id=154964
+
+        Reviewed by Michael Catanzaro.
+
+        * Scripts/run-javascriptcore-tests:
+        (runJSCStressTests): Pass through the --debug option.
+        * Scripts/run-jsc-stress-tests: Increase the timeout for debug builds.
+
 2016-05-18  Aakash Jain  <aakash_jain@apple.com>
 

trunk/Tools/Scripts/run-javascriptcore-tests

@@ -266 +266 @@
     }
 
+    if (configuration() eq "Debug") {
+        push(@jscStressDriverCmd, "--debug");
+    }
+
     if ($envVars ne "") {
             push(@jscStressDriverCmd, "--env-vars");

trunk/Tools/Scripts/run-jsc-stress-tests

@@ -2013 +2013 @@
 end
 
-if $enableFTL and ENV["JSCTEST_timeout"]
+if $enableFTL and ENV["JSCTEST_timeout"] or !ifJSCArgIsntProvidedAreWeReleaseBuild
     # Currently, using the FTL is a performance regression particularly in real
     # (i.e. non-loopy) benchmarks. Account for this in the timeout.
+    # Increase the timeout for debug builds too. (--debug command line option)
     ENV["JSCTEST_timeout"] = (ENV["JSCTEST_timeout"].to_i * 2).to_s
 end