Title: [287025] trunk/Tools
Revision
287025
Author
[email protected]
Date
2021-12-14 08:43:21 -0800 (Tue, 14 Dec 2021)

Log Message

[JSC] Fix LocalJumpError in run-jsc-stress-tests
https://bugs.webkit.org/show_bug.cgi?id=234295

Reviewed by Michael Catanzaro.

Ruby code cannot return from a Proc; use an if/else instead. This fixes
the LocalJumpError reported in https://bugs.webkit.org/show_bug.cgi?id=233926.

* Scripts/run-jsc-stress-tests:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (287024 => 287025)


--- trunk/Tools/ChangeLog	2021-12-14 16:29:11 UTC (rev 287024)
+++ trunk/Tools/ChangeLog	2021-12-14 16:43:21 UTC (rev 287025)
@@ -1,3 +1,15 @@
+2021-12-14  Angelos Oikonomopoulos  <[email protected]>
+
+        [JSC] Fix LocalJumpError in run-jsc-stress-tests
+        https://bugs.webkit.org/show_bug.cgi?id=234295
+
+        Reviewed by Michael Catanzaro.
+
+        Ruby code cannot return from a Proc; use an if/else instead. This fixes
+        the LocalJumpError reported in https://bugs.webkit.org/show_bug.cgi?id=233926.
+
+        * Scripts/run-jsc-stress-tests:
+
 2021-12-14  Jean-Yves Avenard  <[email protected]>
 
         Rename SharedBuffer classes.

Modified: trunk/Tools/Scripts/run-jsc-stress-tests (287024 => 287025)


--- trunk/Tools/Scripts/run-jsc-stress-tests	2021-12-14 16:29:11 UTC (rev 287024)
+++ trunk/Tools/Scripts/run-jsc-stress-tests	2021-12-14 16:43:21 UTC (rev 287025)
@@ -1373,14 +1373,17 @@
         "NoLLInt",
         "no-llint",
         Proc.new { |*optionalTestSpecificOptions|
-            return nil unless $jitTests
-            {
-                :cfg => {
-                },
-                :testSpecificOptions => [
-                    "--useLLInt=false",
-                ] + optionalTestSpecificOptions
-            }
+            if $jitTests
+                {
+                    :cfg => {
+                    },
+                    :testSpecificOptions => [
+                        "--useLLInt=false",
+                    ] + optionalTestSpecificOptions
+                }
+            else
+                nil
+            end
         }
     ],
     [
@@ -1388,11 +1391,12 @@
         "default",
         Proc.new { |*optionalTestSpecificOptions|
             if $memoryLimited
-                return nil
+                nil
+            else
+                {
+                    :testSpecificOptions => optionalTestSpecificOptions
+                }
             end
-            {
-                :testSpecificOptions => optionalTestSpecificOptions
-            }
         }
     ]
 ]
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to