[Wasm-References] Disable references by default
authorjustin_michaud@apple.com <justin_michaud@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Jul 2019 05:14:17 +0000 (05:14 +0000)
committerjustin_michaud@apple.com <justin_michaud@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 2 Jul 2019 05:14:17 +0000 (05:14 +0000)
https://bugs.webkit.org/show_bug.cgi?id=199390

Reviewed by Saam Barati.

JSTests:

* wasm/references-spec-tests/ref_is_null.js:
* wasm/references-spec-tests/ref_null.js:
* wasm/references/anyref_globals.js:
* wasm/references/anyref_modules.js:
* wasm/references/anyref_table.js:
* wasm/references/anyref_table_import.js:
* wasm/references/element_parsing.js:
* wasm/references/func_ref.js:
* wasm/references/is_null.js:
* wasm/references/multitable.js:
* wasm/references/table_misc.js:
* wasm/references/validation.js:

Source/JavaScriptCore:

* runtime/Options.h:

Tools:

* Scripts/run-jsc-stress-tests:

LayoutTests:

* workers/wasm-references.html:

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

19 files changed:
JSTests/ChangeLog
JSTests/wasm/references-spec-tests/ref_is_null.js
JSTests/wasm/references-spec-tests/ref_null.js
JSTests/wasm/references/anyref_globals.js
JSTests/wasm/references/anyref_modules.js
JSTests/wasm/references/anyref_table.js
JSTests/wasm/references/anyref_table_import.js
JSTests/wasm/references/element_parsing.js
JSTests/wasm/references/func_ref.js
JSTests/wasm/references/is_null.js
JSTests/wasm/references/multitable.js
JSTests/wasm/references/table_misc.js
JSTests/wasm/references/validation.js
LayoutTests/ChangeLog
LayoutTests/workers/wasm-references.html
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/runtime/Options.h
Tools/ChangeLog
Tools/Scripts/run-jsc-stress-tests

index c0126b3..60e854a 100644 (file)
@@ -1,3 +1,23 @@
+2019-07-01  Justin Michaud  <justin_michaud@apple.com>
+
+        [Wasm-References] Disable references by default
+        https://bugs.webkit.org/show_bug.cgi?id=199390
+
+        Reviewed by Saam Barati.
+
+        * wasm/references-spec-tests/ref_is_null.js:
+        * wasm/references-spec-tests/ref_null.js:
+        * wasm/references/anyref_globals.js:
+        * wasm/references/anyref_modules.js:
+        * wasm/references/anyref_table.js:
+        * wasm/references/anyref_table_import.js:
+        * wasm/references/element_parsing.js:
+        * wasm/references/func_ref.js:
+        * wasm/references/is_null.js:
+        * wasm/references/multitable.js:
+        * wasm/references/table_misc.js:
+        * wasm/references/validation.js:
+
 2019-07-01  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r246946.
index 83ab049..ab9f64c 100644 (file)
@@ -1,4 +1,4 @@
-
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 'use strict';
 let console = { log: print }
 let hostrefs = {};
index d025904..1cf8da0 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 'use strict';
 
 let console = { log: print }
index 2d3ca7b..61e27d1 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index ed10096..38c1734 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index f3dd01b..c8beaee 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index 2fc601e..62f3a04 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index bc27eac..93be1fd 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 
 function module(bytes, valid = true) {
index ee5187e..9265e63 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index 1c7bc57..6358f45 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index 784ea5d..7ef7130 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index 964c62b..5c65716 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index a288c8b..19bddf9 100644 (file)
@@ -1,3 +1,4 @@
+//@ runWebAssemblySuite("--useWebAssemblyReferences=true")
 import * as assert from '../assert.js';
 import Builder from '../Builder.js';
 
index 81897b5..9f54506 100644 (file)
@@ -1,3 +1,12 @@
+2019-07-01  Justin Michaud  <justin_michaud@apple.com>
+
+        [Wasm-References] Disable references by default
+        https://bugs.webkit.org/show_bug.cgi?id=199390
+
+        Reviewed by Saam Barati.
+
+        * workers/wasm-references.html:
+
 2019-07-01  Devin Rousso  <drousso@apple.com>
 
         Web Inspector: REGRESSION(r245498): Timelines: CPU: discontinuities are filled in by the next record
index 348241b..5c2abf1 100644 (file)
@@ -1,3 +1,4 @@
+<!-- webkit-test-runner [ jscOptions=--useWebAssemblyReferences=true ] -->
 <html>
 <head>
 <script src="../resources/js-test-pre.js"></script>
index d1fb074..a249548 100644 (file)
@@ -1,3 +1,12 @@
+2019-07-01  Justin Michaud  <justin_michaud@apple.com>
+
+        [Wasm-References] Disable references by default
+        https://bugs.webkit.org/show_bug.cgi?id=199390
+
+        Reviewed by Saam Barati.
+
+        * runtime/Options.h:
+
 2019-07-01  Ryan Haddad  <ryanhaddad@apple.com>
 
         Unreviewed, rolling out r246946.
index c42a341..cf96aa9 100644 (file)
@@ -502,7 +502,7 @@ constexpr bool enableWebAssemblyStreamingApi = false;
     v(bool, useWebAssemblyStreamingApi, enableWebAssemblyStreamingApi, Normal, "Allow to run WebAssembly's Streaming API") \
     v(bool, useCallICsForWebAssemblyToJSCalls, true, Normal, "If true, we will use CallLinkInfo to inline cache Wasm to JS calls.") \
     v(bool, useEagerWebAssemblyModuleHashing, false, Normal, "Unnamed WebAssembly modules are identified in backtraces through their hash, if available.") \
-    v(bool, useWebAssemblyReferences, true, Normal, "Allow types from the wasm references spec.") \
+    v(bool, useWebAssemblyReferences, false, Normal, "Allow types from the wasm references spec.") \
     v(bool, useBigInt, false, Normal, "If true, we will enable BigInt support.") \
     v(bool, useIntlNumberFormatToParts, enableIntlNumberFormatToParts, Normal, "If true, we will enable Intl.NumberFormat.prototype.formatToParts") \
     v(bool, useIntlPluralRules, enableIntlPluralRules, Normal, "If true, we will enable Intl.PluralRules.") \
index ca5bab0..3545ca9 100644 (file)
@@ -1,3 +1,12 @@
+2019-07-01  Justin Michaud  <justin_michaud@apple.com>
+
+        [Wasm-References] Disable references by default
+        https://bugs.webkit.org/show_bug.cgi?id=199390
+
+        Reviewed by Saam Barati.
+
+        * Scripts/run-jsc-stress-tests:
+
 2019-07-01  Joonghun Park  <pjh0718@gmail.com>
 
         Unreviewed. Revert r246965 to fix build break
index a033656..eea17fb 100755 (executable)
@@ -1090,21 +1090,21 @@ def runWebAssembly
     end
 end
 
-def runWebAssemblySuite
+def runWebAssemblySuite(*optionalTestSpecificOptions)
     return if !$jitTests
     return if !$isFTLPlatform
     modules = Dir[WASMTESTS_PATH + "*.js"].map { |f| File.basename(f) }
     prepareExtraAbsoluteFiles(WASMTESTS_PATH, ["wasm.json"])
     prepareExtraRelativeFiles(modules.map { |f| "../" + f }, $collection)
-    run("default-wasm", "-m", *FTL_OPTIONS)
+    run("default-wasm", "-m", *(FTL_OPTIONS + optionalTestSpecificOptions))
     if $mode != "quick"
-        run("wasm-no-cjit-yes-tls-context", "-m", "--useFastTLSForWasmContext=true", *(FTL_OPTIONS + NO_CJIT_OPTIONS))
-        run("wasm-eager-jettison", "-m", "--forceCodeBlockToJettisonDueToOldAge=true", *FTL_OPTIONS)
-        run("wasm-no-call-ic", "-m", "--useCallICsForWebAssemblyToJSCalls=false", *FTL_OPTIONS)
-        run("wasm-no-tls-context", "-m", "--useFastTLSForWasmContext=false", *FTL_OPTIONS)
-        run("wasm-slow-memory", "-m", "--useWebAssemblyFastMemory=false", *FTL_OPTIONS)
-        run("wasm-no-air", "-m", "--wasmBBQUsesAir=false", *FTL_OPTIONS)
-        run("wasm-collect-continuously", "-m", "--collectContinuously=true", *FTL_OPTIONS) if shouldCollectContinuously?
+        run("wasm-no-cjit-yes-tls-context", "-m", "--useFastTLSForWasmContext=true", *(FTL_OPTIONS + NO_CJIT_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-eager-jettison", "-m", "--forceCodeBlockToJettisonDueToOldAge=true", *(FTL_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-no-call-ic", "-m", "--useCallICsForWebAssemblyToJSCalls=false", *(FTL_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-no-tls-context", "-m", "--useFastTLSForWasmContext=false", *(FTL_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-slow-memory", "-m", "--useWebAssemblyFastMemory=false", *(FTL_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-no-air", "-m", "--wasmBBQUsesAir=false", *(FTL_OPTIONS + optionalTestSpecificOptions))
+        run("wasm-collect-continuously", "-m", "--collectContinuously=true", *(FTL_OPTIONS + optionalTestSpecificOptions)) if shouldCollectContinuously?
     end
 end