Don't skip tests unnecessarily on arm64
authorsbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Mar 2020 23:53:40 +0000 (23:53 +0000)
committersbarati@apple.com <sbarati@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 2 Mar 2020 23:53:40 +0000 (23:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=208470
<rdar://problem/59968869>

Reviewed by Tadeu Zagallo.

JSTests:

* microbenchmarks/bit-test-constant.js:
* microbenchmarks/bit-test-nonconstant.js:
* microbenchmarks/memcpy-typed-loop-large.js:
* microbenchmarks/memcpy-typed-loop-small.js:
* microbenchmarks/mul-immediate-sub.js:
* stress/dont-range-based-iterate-vector-that-is-mutated.js:
* stress/dont-reserve-huge-capacity-lexer.js:
* stress/new-largeish-contiguous-array-with-size.js:
* stress/sampling-profiler-richards.js:
* typeProfiler/deltablue-for-of.js:
* typeProfiler/getter-richards.js:
* wasm/regress/llint-callee-saves-with-fast-memory.js:

Tools:

* Scripts/run-jsc-stress-tests:

LayoutTests:

* js/script-tests/array-from.js:
* js/script-tests/function-apply-many-args.js:

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

18 files changed:
JSTests/ChangeLog
JSTests/microbenchmarks/bit-test-constant.js
JSTests/microbenchmarks/bit-test-nonconstant.js
JSTests/microbenchmarks/memcpy-typed-loop-large.js
JSTests/microbenchmarks/memcpy-typed-loop-small.js
JSTests/microbenchmarks/mul-immediate-sub.js
JSTests/stress/dont-range-based-iterate-vector-that-is-mutated.js
JSTests/stress/dont-reserve-huge-capacity-lexer.js
JSTests/stress/new-largeish-contiguous-array-with-size.js
JSTests/stress/sampling-profiler-richards.js
JSTests/typeProfiler/deltablue-for-of.js
JSTests/typeProfiler/getter-richards.js
JSTests/wasm/regress/llint-callee-saves-with-fast-memory.js
LayoutTests/ChangeLog
LayoutTests/js/script-tests/array-from.js
LayoutTests/js/script-tests/function-apply-many-args.js
Tools/ChangeLog
Tools/Scripts/run-jsc-stress-tests

index ab8062b..9d1270b 100644 (file)
@@ -1,3 +1,24 @@
+2020-03-02  Saam Barati  <sbarati@apple.com>
+
+        Don't skip tests unnecessarily on arm64
+        https://bugs.webkit.org/show_bug.cgi?id=208470
+        <rdar://problem/59968869>
+
+        Reviewed by Tadeu Zagallo.
+
+        * microbenchmarks/bit-test-constant.js:
+        * microbenchmarks/bit-test-nonconstant.js:
+        * microbenchmarks/memcpy-typed-loop-large.js:
+        * microbenchmarks/memcpy-typed-loop-small.js:
+        * microbenchmarks/mul-immediate-sub.js:
+        * stress/dont-range-based-iterate-vector-that-is-mutated.js:
+        * stress/dont-reserve-huge-capacity-lexer.js:
+        * stress/new-largeish-contiguous-array-with-size.js:
+        * stress/sampling-profiler-richards.js:
+        * typeProfiler/deltablue-for-of.js:
+        * typeProfiler/getter-richards.js:
+        * wasm/regress/llint-callee-saves-with-fast-memory.js:
+
 2020-03-02  Justin Michaud  <justin_michaud@apple.com>
 
         Delete by val caching does not keep the subscript alive
index 3f97305..a2cdba3 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
+//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
 let glob = 0
 
 function doTest(number) {
index f4740f6..a04e1c2 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $model == "Apple Watch Series 3" or (not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
+//@ skip if $model == "Apple Watch Series 3" or (not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
 let glob = 0
 
 function doTest(number, bit) {
index ef028d4..11cf326 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
+//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
 function doTest(arr1, arr2) {
     if (arr1.length != arr2.length)
         return []
index ccde893..682dfc8 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
+//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
 function doTest(arr1, arr2) {
     if (arr1.length != arr2.length)
         return []
index 2037cd0..fd16d49 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
+//@ skip if $model == "Apple Watch Series 3" or $model == "Apple Watch Series 4" or (not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux") # added by mark-jsc-stress-test.py
 function doTest(max) {
     let sum = 0
     for (let i=0; i<max; ++i) {
index 53cdc04..51c0c41 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux"
+//@ skip if not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux"
 // Should not crash when run with ASAN.
 
 function foo(arr1, arr2) {
index 6b56962..628d8c6 100644 (file)
@@ -1,4 +1,4 @@
-//@ if ($architecture != "x86-64") or $memoryLimited then skip else runDefault end
+//@ if $memoryLimited then skip else runDefault end
 
 var fe="f";                                                                         
 try
index e119442..fba335c 100644 (file)
@@ -1,6 +1,6 @@
 // We only need one run of this with any GC or JIT strategy. This test is not particularly fast.
 // Unfortunately, it needs to run for a while to test the thing it's testing.
-//@ if $architecture =~ /arm|mips/ then skip else runWithRAMSize(10000000) end
+//@ if $architecture =~ /(^arm$)|mips/ then skip else runWithRAMSize(10000000) end
 //@ slow!
 
 function foo(x) {
index 54457a3..4280124 100644 (file)
@@ -1,5 +1,5 @@
 //@ skip if $architecture == "mips" and $hostOS == "linux"
-//@ skip if not $jitTests and $architecture =~ /arm|mips/ and $hostOS == "linux"
+//@ skip if not $jitTests and $architecture =~ /(^arm$)|mips/ and $hostOS == "linux"
 //@ skip if $architecture == "x86"
 //@ runDefault("--collectContinuously=1", "--useSamplingProfiler=1", "--collectSamplingProfilerDataForJSCShell=1")
 
index 89f820c..152e326 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "debug" or $architecture =~ /arm|mips/ then skip else runTypeProfiler end
+//@ if $buildType == "debug" or $architecture =~ /(^arm$)|mips/ then skip else runTypeProfiler end
 
 // Copyright 2008 the V8 project authors. All rights reserved.
 // Copyright 1996 John Maloney and Mario Wolczko.
index 220bb69..7974674 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "debug" or not $jitTests or $architecture =~ /arm|mips/ then skip else runTypeProfiler end
+//@ if $buildType == "debug" or not $jitTests or $architecture =~ /(^arm$)|mips/ then skip else runTypeProfiler end
 
 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Copyright 2014 Apple Inc.
index c1076d7..aa6bb00 100644 (file)
@@ -1,6 +1,4 @@
-//@ skip if $architecture != "x86-64"
 //@ requireOptions("--useWebAssemblyFastMemory=true")
-// FIXME: Stop skipping when we enable fast memory for iOS. https://bugs.webkit.org/show_bug.cgi?id=170774
 
 import { instantiate } from '../wabt-wrapper.js';
 
index e9bd8e4..824b254 100644 (file)
@@ -1,3 +1,14 @@
+2020-03-02  Saam Barati  <sbarati@apple.com>
+
+        Don't skip tests unnecessarily on arm64
+        https://bugs.webkit.org/show_bug.cgi?id=208470
+        <rdar://problem/59968869>
+
+        Reviewed by Tadeu Zagallo.
+
+        * js/script-tests/array-from.js:
+        * js/script-tests/function-apply-many-args.js:
+
 2020-03-02  Jason Lawrence  <lawrence.j@apple.com>
 
         [ Mac wk2 ] is css3/shapes/shape-outside/shape-image/shape-image-025.html flaky failing.
index f339604..c63cf3a 100644 (file)
@@ -1,6 +1,3 @@
-// [ARM] jsc-layout-tests.yaml/js/script-tests/array-from.js fails on Aarch64
-// https://bugs.webkit.org/show_bug.cgi?id=142272
-//@ skip if $architecture == "arm64"
 description("Tests for Array.from");
 
 function section(title) {
index 2653ec3..6d7c70b 100644 (file)
@@ -1,4 +1,4 @@
-//@ skip if $architecture !~ /x86/i and $hostOS == "darwin"
+//@ skip if $memoryLimited and $hostOS == "darwin"
 
 description("Tests that we throw an error when passing a number of arguments beyond a certain threshold.");
 
index faf0efc..6144dca 100644 (file)
@@ -1,3 +1,13 @@
+2020-03-02  Saam Barati  <sbarati@apple.com>
+
+        Don't skip tests unnecessarily on arm64
+        https://bugs.webkit.org/show_bug.cgi?id=208470
+        <rdar://problem/59968869>
+
+        Reviewed by Tadeu Zagallo.
+
+        * Scripts/run-jsc-stress-tests:
+
 2020-03-02  Andres Gonzalez  <andresg_22@apple.com>
 
         Fix for LayoutTests/accessibility/mac/search-text/search-text.html in IsolatedTree mode.
index bdb1a3b..2fbe3b4 100755 (executable)
@@ -698,12 +698,6 @@ def runBytecodeCacheNoAssertion(*optionalTestSpecificOptions)
 end
 
 def runBigIntEnabled(*optionalTestSpecificOptions)
-    # FIXME: <rdar://problem/40331121>
-    if $remote or ($architecture !~ /x86/i and $hostOS == "darwin")
-        skip
-        return
-    end
-
     run("big-int-enabled", "--useBigInt=true" , *(FTL_OPTIONS + optionalTestSpecificOptions))
 end
 
@@ -930,7 +924,7 @@ def defaultNoSamplingProfilerRun
 end
 
 def runProfiler
-    if $remote or ($architecture !~ /x86/i and $hostOS == "darwin") or ($hostOS == "windows") or ($hostOS == "playstation")
+    if $remote or $memoryLimited or ($hostOS == "windows") or ($hostOS == "playstation")
         skip
         return
     end