Title: [238568] trunk
Revision
238568
Author
[email protected]
Date
2018-11-27 13:03:29 -0800 (Tue, 27 Nov 2018)

Log Message

Unreviewed, rolling out r238509.

Causes JSC tests to fail on iOS.

Reverted changeset:

"NaNs read from Wasm code needs to be be purified."
https://bugs.webkit.org/show_bug.cgi?id=191056
https://trac.webkit.org/changeset/238509

Modified Paths

Removed Paths

Diff

Modified: trunk/JSTests/ChangeLog (238567 => 238568)


--- trunk/JSTests/ChangeLog	2018-11-27 20:40:40 UTC (rev 238567)
+++ trunk/JSTests/ChangeLog	2018-11-27 21:03:29 UTC (rev 238568)
@@ -1,3 +1,15 @@
+2018-11-27  Ryan Haddad  <[email protected]>
+
+        Unreviewed, rolling out r238509.
+
+        Causes JSC tests to fail on iOS.
+
+        Reverted changeset:
+
+        "NaNs read from Wasm code needs to be be purified."
+        https://bugs.webkit.org/show_bug.cgi?id=191056
+        https://trac.webkit.org/changeset/238509
+
 2018-11-26  Caio Lima  <[email protected]>
 
         Re-introduce op_bitnot

Deleted: trunk/JSTests/wasm/regress/regress-191056.js (238567 => 238568)


--- trunk/JSTests/wasm/regress/regress-191056.js	2018-11-27 20:40:40 UTC (rev 238567)
+++ trunk/JSTests/wasm/regress/regress-191056.js	2018-11-27 21:03:29 UTC (rev 238568)
@@ -1,15 +0,0 @@
-var importObject = {
-    env: {
-        print_number: function (number) {
-            print('[+] importObject callback.');
-            print(number);
-        }
-    }
-};
-var wasmCode = new Uint8Array([0x0,0x61,0x73,0x6d,0x1,0x0,0x0,0x0,0x1,0x7,0x1,0x60,0x2,0x7e,0x7f,0x1,0x7f,0x3,0x2,0x1,0x0,0x4,0x4,0x1,0x70,0x0,0x4,0x6,0x2e,0x5,0x7f,0x0,0x41,0x2a,0xb,0x7d,0x0,0x43,0x74,0x0,0x0,0x4d,0xb,0x7c,0x0    ,0x44,0x83,0x88,0x88,0x00,0x0,0x0,0xff,0xff,0xb,0x7d,0x0,0x43,0x0,0x0,0x9,0x7f,0xb,0x7c,0x0,0x44,0x0,0x3c,0x0,0x0,0x0,0x0,0xf8,0x7f,0xb,0x7,0x40,0x7,0x5,0x74,0x61,0x62,0x3f,0x45,0x1,0x0,0x3,0x7d,0x75,0x6d,0x0,0x0,0x6,0x61,0x3d,    0x22,0x0,0xd,0x72,0x3,0x0,0x7,0x4f,0x7c,0x68,0x77,0x65,0x20,0x31,0x3,0x1,0x7,0x41,0x41,0x41,0x41,0x41,0x41,0x41,0x3,0x2,0x7,0x61,0x6e,0x73,0x77,0x65,0x72,0x3c,0x3,0x3,0x7,0x61,0x6e,0x73,0x77,0x65,0x72,0x34,0x3,0x4,0x9,0x7,0x1,0x0,0x41,0x0,0xb,0x1,0x0,0xa,0x9,0x1,0x7,0x0,0x20,0x1,0x0,0x1,0x0,0xb,]);
-var wasmModule = new WebAssembly.Module(wasmCode);
-var wasmInstance = new WebAssembly.Instance(wasmModule, importObject);
-
-var res = wasmInstance.exports.AAAAAAA;
-res = res + 'string';
-

Modified: trunk/Source/_javascript_Core/ChangeLog (238567 => 238568)


--- trunk/Source/_javascript_Core/ChangeLog	2018-11-27 20:40:40 UTC (rev 238567)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-11-27 21:03:29 UTC (rev 238568)
@@ -1,3 +1,15 @@
+2018-11-27  Ryan Haddad  <[email protected]>
+
+        Unreviewed, rolling out r238509.
+
+        Causes JSC tests to fail on iOS.
+
+        Reverted changeset:
+
+        "NaNs read from Wasm code needs to be be purified."
+        https://bugs.webkit.org/show_bug.cgi?id=191056
+        https://trac.webkit.org/changeset/238509
+
 2018-11-27  Mark Lam  <[email protected]>
 
         Introducing a ENABLE_SEPARATED_WX_HEAP macro.

Modified: trunk/Source/_javascript_Core/wasm/js/WebAssemblyModuleRecord.cpp (238567 => 238568)


--- trunk/Source/_javascript_Core/wasm/js/WebAssemblyModuleRecord.cpp	2018-11-27 20:40:40 UTC (rev 238567)
+++ trunk/Source/_javascript_Core/wasm/js/WebAssemblyModuleRecord.cpp	2018-11-27 21:03:29 UTC (rev 238568)
@@ -388,11 +388,11 @@
                 return;
 
             case Wasm::F32:
-                exportedValue = jsNumber(purifyNaN(m_instance->instance().loadF32Global(exp.kindIndex)));
+                exportedValue = JSValue(m_instance->instance().loadF32Global(exp.kindIndex));
                 break;
 
             case Wasm::F64:
-                exportedValue = jsNumber(purifyNaN(m_instance->instance().loadF64Global(exp.kindIndex)));
+                exportedValue = JSValue(m_instance->instance().loadF64Global(exp.kindIndex));
                 break;
 
             default:
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to