REGRESSION(r237547): Test failures on 32-bit JSC since the JIT was disabled
authortzagallo@apple.com <tzagallo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2018 09:05:15 +0000 (09:05 +0000)
committertzagallo@apple.com <tzagallo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Nov 2018 09:05:15 +0000 (09:05 +0000)
https://bugs.webkit.org/show_bug.cgi?id=191184

Reviewed by Saam Barati.

JSTests:

Most tests were failing due to timeouts, since they are too slow to
run on CLoop. The exceptions are:

proxy-get-set-correct-receiver.js: Had to reduce the recursion depth not to overflow on CLoop
dont-crash-on-stack-overflow-when-parsing-builtin.js and
dont-crash-on-stack-overflow-when-parsing-default-constructor.js: had
to change the stack size since CLoop requires it to be page aligned.

* microbenchmarks/array-push-1.js:
* microbenchmarks/array-push-2.js:
* microbenchmarks/elidable-new-object-dag.js:
* microbenchmarks/elidable-new-object-roflcopter.js:
* microbenchmarks/elidable-new-object-tree.js:
* microbenchmarks/getter-richards.js:
* microbenchmarks/sinkable-new-object-dag.js:
* microbenchmarks/string-concat-long-convert.js:
* microbenchmarks/typed-array-get-set-by-val-profiling.js:
* slowMicrobenchmarks/array-push-3.js:
* slowMicrobenchmarks/large-map-iteration-with-additions.js:
* slowMicrobenchmarks/spread-small-array.js:
* slowMicrobenchmarks/undefined-property-access.js:
* stress/activation-sink-default-value-tdz-error.js:
* stress/activation-sink-default-value.js:
* stress/activation-sink-osrexit-default-value-tdz-error.js:
* stress/activation-sink-osrexit-default-value.js:
* stress/activation-sink-osrexit.js:
* stress/activation-sink.js:
* stress/allow-math-ic-b3-code-duplication.js:
* stress/array-push-multiple-int32.js:
* stress/arrowfunction-activation-sink-osrexit-default-value-tdz-error.js:
* stress/arrowfunction-lexical-this-activation-sink-osrexit.js:
* stress/arrowfunction-lexical-this-activation-sink.js:
* stress/dont-crash-on-stack-overflow-when-parsing-builtin.js:
* stress/dont-crash-on-stack-overflow-when-parsing-default-constructor.js:
* stress/elide-new-object-dag-then-exit.js:
* stress/materialize-regexp-cyclic.js:
* stress/new-regex-inline.js:
* stress/op_add.js:
* stress/op_bitand.js:
* stress/op_bitor.js:
* stress/op_bitxor.js:
* stress/op_div-ConstVar.js:
* stress/op_div-VarConst.js:
* stress/op_div-VarVar.js:
* stress/op_lshift-ConstVar.js:
* stress/op_lshift-VarConst.js:
* stress/op_lshift-VarVar.js:
* stress/op_mod-ConstVar.js:
* stress/op_mod-VarConst.js:
* stress/op_mod-VarVar.js:
* stress/op_mul-ConstVar.js:
* stress/op_mul-VarConst.js:
* stress/op_mul-VarVar.js:
* stress/op_rshift-ConstVar.js:
* stress/op_rshift-VarConst.js:
* stress/op_rshift-VarVar.js:
* stress/op_sub-ConstVar.js:
* stress/op_sub-VarConst.js:
* stress/op_sub-VarVar.js:
* stress/op_urshift-ConstVar.js:
* stress/op_urshift-VarConst.js:
* stress/op_urshift-VarVar.js:
* stress/proxy-get-set-correct-receiver.js:
* stress/regress-179562.js:
* stress/rest-parameter-many-arguments.js:
* stress/sampling-profiler-richards.js:
* stress/splay-flash-access-1ms.js:
* stress/tailCallForwardArguments.js:
* stress/typed-array-get-by-val-profiling.js:
* typeProfiler/getter-richards.js:

PerformanceTests:

Skip test when the JIT is disabled, it takes too long to run on CLoop.

* JetStream/cdjs/main.js:

Source/JavaScriptCore:

Fix API test on CLoop: we can only disable the LLInt when the JIT is enabled.

* API/tests/PingPongStackOverflowTest.cpp:
(testPingPongStackOverflow):

LayoutTests:

Skip test when the JIT is disabled since it's too slow to run on CLoop.

* js/script-tests/regress-139548.js:

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

69 files changed:
JSTests/ChangeLog
JSTests/microbenchmarks/array-push-1.js
JSTests/microbenchmarks/array-push-2.js
JSTests/microbenchmarks/elidable-new-object-dag.js
JSTests/microbenchmarks/elidable-new-object-roflcopter.js
JSTests/microbenchmarks/elidable-new-object-tree.js
JSTests/microbenchmarks/getter-richards.js
JSTests/microbenchmarks/sinkable-new-object-dag.js
JSTests/microbenchmarks/string-concat-long-convert.js
JSTests/microbenchmarks/typed-array-get-set-by-val-profiling.js
JSTests/slowMicrobenchmarks/array-push-3.js
JSTests/slowMicrobenchmarks/large-map-iteration-with-additions.js
JSTests/slowMicrobenchmarks/spread-small-array.js
JSTests/slowMicrobenchmarks/undefined-property-access.js
JSTests/stress/activation-sink-default-value-tdz-error.js
JSTests/stress/activation-sink-default-value.js
JSTests/stress/activation-sink-osrexit-default-value-tdz-error.js
JSTests/stress/activation-sink-osrexit-default-value.js
JSTests/stress/activation-sink-osrexit.js
JSTests/stress/activation-sink.js
JSTests/stress/allow-math-ic-b3-code-duplication.js
JSTests/stress/array-push-multiple-int32.js
JSTests/stress/arrowfunction-activation-sink-osrexit-default-value-tdz-error.js
JSTests/stress/arrowfunction-lexical-this-activation-sink-osrexit.js
JSTests/stress/arrowfunction-lexical-this-activation-sink.js
JSTests/stress/dont-crash-on-stack-overflow-when-parsing-builtin.js
JSTests/stress/dont-crash-on-stack-overflow-when-parsing-default-constructor.js
JSTests/stress/elide-new-object-dag-then-exit.js
JSTests/stress/materialize-regexp-cyclic.js
JSTests/stress/new-regex-inline.js
JSTests/stress/op_add.js
JSTests/stress/op_bitand.js
JSTests/stress/op_bitor.js
JSTests/stress/op_bitxor.js
JSTests/stress/op_div-ConstVar.js
JSTests/stress/op_div-VarConst.js
JSTests/stress/op_div-VarVar.js
JSTests/stress/op_lshift-ConstVar.js
JSTests/stress/op_lshift-VarConst.js
JSTests/stress/op_lshift-VarVar.js
JSTests/stress/op_mod-ConstVar.js
JSTests/stress/op_mod-VarConst.js
JSTests/stress/op_mod-VarVar.js
JSTests/stress/op_mul-ConstVar.js
JSTests/stress/op_mul-VarConst.js
JSTests/stress/op_mul-VarVar.js
JSTests/stress/op_rshift-ConstVar.js
JSTests/stress/op_rshift-VarConst.js
JSTests/stress/op_rshift-VarVar.js
JSTests/stress/op_sub-ConstVar.js
JSTests/stress/op_sub-VarConst.js
JSTests/stress/op_sub-VarVar.js
JSTests/stress/op_urshift-ConstVar.js
JSTests/stress/op_urshift-VarConst.js
JSTests/stress/op_urshift-VarVar.js
JSTests/stress/proxy-get-set-correct-receiver.js
JSTests/stress/regress-179562.js
JSTests/stress/rest-parameter-many-arguments.js
JSTests/stress/sampling-profiler-richards.js
JSTests/stress/splay-flash-access-1ms.js
JSTests/stress/tailCallForwardArguments.js
JSTests/stress/typed-array-get-by-val-profiling.js
JSTests/typeProfiler/getter-richards.js
LayoutTests/ChangeLog
LayoutTests/js/script-tests/regress-139548.js
PerformanceTests/ChangeLog
PerformanceTests/JetStream/cdjs/main.js
Source/JavaScriptCore/API/tests/PingPongStackOverflowTest.cpp
Source/JavaScriptCore/ChangeLog

index 30a4003..2f60247 100644 (file)
@@ -1,3 +1,81 @@
+2018-11-07  Tadeu Zagallo  <tzagallo@apple.com>
+
+        REGRESSION(r237547): Test failures on 32-bit JSC since the JIT was disabled
+        https://bugs.webkit.org/show_bug.cgi?id=191184
+
+        Reviewed by Saam Barati.
+
+        Most tests were failing due to timeouts, since they are too slow to
+        run on CLoop. The exceptions are:
+
+        proxy-get-set-correct-receiver.js: Had to reduce the recursion depth not to overflow on CLoop
+        dont-crash-on-stack-overflow-when-parsing-builtin.js and
+        dont-crash-on-stack-overflow-when-parsing-default-constructor.js: had
+        to change the stack size since CLoop requires it to be page aligned.
+
+        * microbenchmarks/array-push-1.js:
+        * microbenchmarks/array-push-2.js:
+        * microbenchmarks/elidable-new-object-dag.js:
+        * microbenchmarks/elidable-new-object-roflcopter.js:
+        * microbenchmarks/elidable-new-object-tree.js:
+        * microbenchmarks/getter-richards.js:
+        * microbenchmarks/sinkable-new-object-dag.js:
+        * microbenchmarks/string-concat-long-convert.js:
+        * microbenchmarks/typed-array-get-set-by-val-profiling.js:
+        * slowMicrobenchmarks/array-push-3.js:
+        * slowMicrobenchmarks/large-map-iteration-with-additions.js:
+        * slowMicrobenchmarks/spread-small-array.js:
+        * slowMicrobenchmarks/undefined-property-access.js:
+        * stress/activation-sink-default-value-tdz-error.js:
+        * stress/activation-sink-default-value.js:
+        * stress/activation-sink-osrexit-default-value-tdz-error.js:
+        * stress/activation-sink-osrexit-default-value.js:
+        * stress/activation-sink-osrexit.js:
+        * stress/activation-sink.js:
+        * stress/allow-math-ic-b3-code-duplication.js:
+        * stress/array-push-multiple-int32.js:
+        * stress/arrowfunction-activation-sink-osrexit-default-value-tdz-error.js:
+        * stress/arrowfunction-lexical-this-activation-sink-osrexit.js:
+        * stress/arrowfunction-lexical-this-activation-sink.js:
+        * stress/dont-crash-on-stack-overflow-when-parsing-builtin.js:
+        * stress/dont-crash-on-stack-overflow-when-parsing-default-constructor.js:
+        * stress/elide-new-object-dag-then-exit.js:
+        * stress/materialize-regexp-cyclic.js:
+        * stress/new-regex-inline.js:
+        * stress/op_add.js:
+        * stress/op_bitand.js:
+        * stress/op_bitor.js:
+        * stress/op_bitxor.js:
+        * stress/op_div-ConstVar.js:
+        * stress/op_div-VarConst.js:
+        * stress/op_div-VarVar.js:
+        * stress/op_lshift-ConstVar.js:
+        * stress/op_lshift-VarConst.js:
+        * stress/op_lshift-VarVar.js:
+        * stress/op_mod-ConstVar.js:
+        * stress/op_mod-VarConst.js:
+        * stress/op_mod-VarVar.js:
+        * stress/op_mul-ConstVar.js:
+        * stress/op_mul-VarConst.js:
+        * stress/op_mul-VarVar.js:
+        * stress/op_rshift-ConstVar.js:
+        * stress/op_rshift-VarConst.js:
+        * stress/op_rshift-VarVar.js:
+        * stress/op_sub-ConstVar.js:
+        * stress/op_sub-VarConst.js:
+        * stress/op_sub-VarVar.js:
+        * stress/op_urshift-ConstVar.js:
+        * stress/op_urshift-VarConst.js:
+        * stress/op_urshift-VarVar.js:
+        * stress/proxy-get-set-correct-receiver.js:
+        * stress/regress-179562.js:
+        * stress/rest-parameter-many-arguments.js:
+        * stress/sampling-profiler-richards.js:
+        * stress/splay-flash-access-1ms.js:
+        * stress/tailCallForwardArguments.js:
+        * stress/typed-array-get-by-val-profiling.js:
+        * typeProfiler/getter-richards.js:
+
 2018-11-06  Michael Saboff  <msaboff@apple.com>
 
         Multiple stress/regexp-compile-oom.js tests are failing on High Sierra Debug and Release JSC testers.
index 5e6d9e6..2e8d680 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function arrayPush1() {
   var ret = [1];
   ret.push(1);
index cfd8f8a..3e82efd 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function arrayPush2() {
   var ret = [1];
   ret.push(1, 2);
index c2a77f6..35de670 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function sumOfArithSeries(limit) {
     return limit * (limit + 1) / 2;
 }
index 294a56b..96590cf 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function sumOfArithSeries(limit) {
     return limit * (limit + 1) / 2;
 }
index 21fe9c0..a54e18a 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function sumOfArithSeries(limit) {
     return limit * (limit + 1) / 2;
 }
index 288ed86..4eb348a 100644 (file)
@@ -1,6 +1,4 @@
-// FIXME: unskip when this is solved
-// https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "mips" or $architecture == "arm"
+//@ skip if not $jitTests
 //@ defaultQuickRun
 
 // Copyright 2006-2008 the V8 project authors. All rights reserved.
index bc0b923..5462252 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function sumOfArithSeries(limit) {
     return limit * (limit + 1) / 2;
 }
index 44a9cf8..28df77d 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function test(a, b, c, d, e)
 {
     return a.concat(b, c, d, e);
index 8e12c67..5d49d2b 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 // The type of arrayObject is polymorphic, but the access we do on it are not.
 function nonPolymorphicUint8ClampedArraySetter(arrayObject, isTypedArray) {
     if (isTypedArray) {
index 8c3694d..d65c148 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function arrayPush3() {
   var ret = [1];
   ret.push(1, 2, 3);
index fa2d35e..dfc5ff1 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 let counter = 0;
 function bar(map) {
     for (let [key, value] of map) {
index a68ee6c..ab82645 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function foo(arg) {
     return [...arg];
 }
index c9765bc..fb70067 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 var someGlobal;
 
 // This is a simple speed test. It should go fast.
index cc8b280..c780b38 100644 (file)
@@ -1,4 +1,5 @@
-//@ skip if $buildType == "debug"
+//@ skip if $buildType == "debug" or $architecture == "x86"
+
 "use strict";
 
 var n = 10000000;
index 0e660d5..4bb5277 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 var n = 10000000;
 
 function bar(f) { f(10); }
index 9d3c4d3..bd5a2af 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 var n = 10000000;
 
 function bar() { }
index bf00022..ccde211 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 var n = 10000000;
 
 function bar(f) { f(10); }
index 2059b18..0dc1827 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function test1() {
     var o1;
     for (let i = 0; i < 1000000; ++i) {
index 553ee58..f4164d8 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function shouldBe(actual, expected)
 {
     if (actual !== expected)
index 606ef4b..4254ec8 100644 (file)
@@ -1,4 +1,4 @@
-//@ runDefault("--softReservedZoneSize=16384", "--reservedZoneSize=0", "--useJIT=0", "--validateBytecode=1", "--maxPerThreadStackUsage=500000")
+//@ runDefault("--softReservedZoneSize=16384", "--reservedZoneSize=0", "--useJIT=0", "--validateBytecode=1", "--maxPerThreadStackUsage=499712")
 
 function f() {
     try {
index 4f91cbe..5545441 100644 (file)
@@ -1,4 +1,4 @@
-//@ runDefault("--softReservedZoneSize=16384", "--reservedZoneSize=0", "--useJIT=0", "--validateBytecode=1", "--maxPerThreadStackUsage=500000")
+//@ runDefault("--softReservedZoneSize=16384", "--reservedZoneSize=0", "--useJIT=0", "--validateBytecode=1", "--maxPerThreadStackUsage=499712")
 
 function runNearStackLimit(f) {
     function t() {
index fbe99a5..a2ed594 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function sumOfArithSeries(limit) {
     return limit * (limit + 1) / 2;
 }
index 1097041..1787639 100644 (file)
@@ -1,3 +1,5 @@
+//@ slow!
+
 function shouldBe(actual, expected)
 {
     if (actual !== expected)
index 22feeb5..53d8047 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 function assert(a) {
     if (!a)
         throw Error("bad assertion");
index 65679b4..c629040 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 1bc61a0..a458603 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 685e236..2668a99 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index e05fa5d..769c552 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 7dc565c..4508654 100644 (file)
@@ -1,3 +1,4 @@
+//@ skip if $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 72eb2b5..91ff2f1 100644 (file)
@@ -1,3 +1,4 @@
+//@ skip if $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 05fd85f..828a203 100644 (file)
@@ -1,3 +1,4 @@
+//@ skip if $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=2.0")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 0feac84..79b46c1 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 3345eee..da7e06f 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 5c20cc3..26b8688 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index f7d439f..6052bb2 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=1.5")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 97bdcf2..3ecc67d 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=1.5")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 4d32170..1f09be8 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT("--timeoutMultiplier=1.5")
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 934926a..f1ca9b5 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index e7c617b..1de41fe 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index ec1225b..fd9f1d0 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index b6d65bc..721856c 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index ab46fcd..b2bf2da 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 02c5b41..a6a1bfc 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 9e94c2b..f651df2 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 5223846..f590d0e 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index e8bbc15..f194861 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 82312fe..d427c1f 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 491153b..1bf063c 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index 5b6f5cb..bc22b99 100644 (file)
@@ -1,6 +1,6 @@
 // FIXME: unskip when this is solved
 // https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "arm"
+//@ skip if $architecture == "arm" or $architecture == "x86"
 //@ runFTLNoCJIT
 
 // If all goes well, this test module will terminate silently. If not, it will print
index e7fc18c..6708ab1 100644 (file)
@@ -80,7 +80,7 @@ test(function() {
     let callCount = 0;
     let handler = {
         get(target, prop, receiver) {
-            if (callCount === 200)
+            if (callCount === 100)
                 assert(target === t);
             ++callCount;
             assert(receiver === proxy);
@@ -89,10 +89,10 @@ test(function() {
         }
     };
     let proxy = new Proxy(t, handler);
-    for (let i = 0; i < 200; ++i)
+    for (let i = 0; i < 100; ++i)
         proxy = new Proxy(proxy, handler);
     proxy.prop
-    assert(callCount === 201);
+    assert(callCount === 101);
 }, 10);
 
 test(function() {
@@ -100,7 +100,7 @@ test(function() {
     let callCount = 0;
     let handler = {
         set(target, prop, value, receiver) {
-            if (callCount === 200)
+            if (callCount === 100)
                 assert(target === t);
             ++callCount;
             assert(receiver === proxy);
@@ -110,8 +110,8 @@ test(function() {
         }
     };
     let proxy = new Proxy(t, handler);
-    for (let i = 0; i < 200; ++i)
+    for (let i = 0; i < 100; ++i)
         proxy = new Proxy(proxy, handler);
     proxy.prop = 20;
-    assert(callCount === 201);
+    assert(callCount === 101);
 }, 10);
index 287e0ed..488bf9b 100644 (file)
@@ -1,4 +1,4 @@
-//@ runFTLNoCJIT("--maxPerThreadStackUsage=400000")
+//@ runFTLNoCJIT("--maxPerThreadStackUsage=400000") if $jitTests
 
 function runNearStackLimit(f) {
     function t() {
index 9a2cb28..16a1f28 100644 (file)
@@ -1,3 +1,4 @@
+//@ skip if $architecture == "x86"
 //@ if $architecture == "x86" then defaultSpotCheckNoMaximalFlush else defaultRun end
 
 function assert(b) {
index 40af902..475063b 100644 (file)
@@ -1,6 +1,7 @@
 // [JSC] [Armv7] stress/sampling-profiler-richards.js crashes
 // https://bugs.webkit.org/show_bug.cgi?id=190426
 //@ skip if $architecture == "arm" and $hostOS == "linux"
+//@ skip if $architecture == "x86"
 //@ runDefault("--collectContinuously=1", "--useSamplingProfiler=1", "--collectSamplingProfilerDataForJSCShell=1")
 
 "use strict";
index 2010ec2..aa676ee 100644 (file)
@@ -1,4 +1,5 @@
 //@ skip if $memoryLimited
+//@ skip if $architecture == "x86"
 //@ runNoisyTestDefault
 //@ runNoisyTestNoCJIT
 
index f654f72..6972552 100644 (file)
@@ -1,3 +1,5 @@
+//@ skip if $architecture == "x86"
+
 var createBuiltin = $vm.createBuiltin;
 
 // This is pretty bad but I need a private name.
index b02f590..39a0394 100644 (file)
@@ -1,4 +1,4 @@
-//@ if $buildType == "debug" or $architecture =~ /arm|mips/ then skip else runTypeProfiler end
+//@ if $buildType == "debug" or not $jitTests then skip else runTypeProfiler end
 
 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Copyright 2014 Apple Inc.
index ee314dd..a458af8 100644 (file)
@@ -1,3 +1,14 @@
+2018-11-07  Tadeu Zagallo  <tzagallo@apple.com>
+
+        REGRESSION(r237547): Test failures on 32-bit JSC since the JIT was disabled
+        https://bugs.webkit.org/show_bug.cgi?id=191184
+
+        Reviewed by Saam Barati.
+
+        Skip test when the JIT is disabled since it's too slow to run on CLoop.
+
+        * js/script-tests/regress-139548.js:
+
 2018-11-06  Youenn Fablet  <youenn@apple.com>
 
         Reenable imported/w3c/web-platform-tests/webrtc/RTCPeerConnection-ontrack.https.html
index 6bba468..0a42e20 100644 (file)
@@ -1,6 +1,4 @@
-// FIXME: unskip when this is solved
-// https://bugs.webkit.org/show_bug.cgi?id=191163
-//@ skip if $architecture == "mips" or $architecture == "arm"
+//@ skip if not $jitTests
 //@ slow!
 //@ noEagerNoNoLLIntTestsRunLayoutTest
 
index 0455c8a..d4891c4 100644 (file)
@@ -1,3 +1,14 @@
+2018-11-07  Tadeu Zagallo  <tzagallo@apple.com>
+
+        REGRESSION(r237547): Test failures on 32-bit JSC since the JIT was disabled
+        https://bugs.webkit.org/show_bug.cgi?id=191184
+
+        Reviewed by Saam Barati.
+
+        Skip test when the JIT is disabled, it takes too long to run on CLoop.
+
+        * JetStream/cdjs/main.js:
+
 2018-11-05  Myles C. Maxfield  <mmaxfield@apple.com>
 
         Cache glyph paths and share underline skipping code between all the ports
index f4c8914..677d7fe 100644 (file)
@@ -25,6 +25,7 @@
 
 // This is run as a JSC stress test. Let the harness know that this is a slow test.
 //@ slow!
+//@ skip if $architecture == "x86"
 
 load("constants.js");
 load("util.js");
index 832bd14..fbe4687 100644 (file)
@@ -127,7 +127,7 @@ int testPingPongStackOverflow()
 
     Options::softReservedZoneSize() = 128 * KB;
     Options::reservedZoneSize() = 64 * KB;
-#if ENABLE(C_LOOP)
+#if ENABLE(JIT)
     // Normally, we want to disable the LLINT to force the use of JITted code which is necessary for
     // reproducing the regression in https://bugs.webkit.org/show_bug.cgi?id=148749. However, we only
     // want to do this if the LLINT isn't the only available execution engine.
index e815271..dfb6102 100644 (file)
@@ -1,3 +1,15 @@
+2018-11-07  Tadeu Zagallo  <tzagallo@apple.com>
+
+        REGRESSION(r237547): Test failures on 32-bit JSC since the JIT was disabled
+        https://bugs.webkit.org/show_bug.cgi?id=191184
+
+        Reviewed by Saam Barati.
+
+        Fix API test on CLoop: we can only disable the LLInt when the JIT is enabled.
+
+        * API/tests/PingPongStackOverflowTest.cpp:
+        (testPingPongStackOverflow):
+
 2018-11-06  Justin Fan  <justin_fan@apple.com>
 
         [WebGPU] Experimental prototype for WebGPURenderPipeline and WebGPUSwapChain