[wasm] asm.js - Remove Wasm.instantiateModuleFromAsm, use asm.js directly.

test/mjsunit/regress/regress-592352.js

Index: test/mjsunit/regress/regress-592352.js
diff --git a/test/mjsunit/regress/regress-592352.js b/test/mjsunit/regress/regress-592352.js
index f3477c038dde3611b8aadc5dc79e4795296a76dc..b4e32c7743251e79d5cb8150e97aca7d2a716552 100644
--- a/test/mjsunit/regress/regress-592352.js
+++ b/test/mjsunit/regress/regress-592352.js
@@ -2,7 +2,7 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-// Flags: --expose-wasm --allow-natives-syntax
+// Flags: --allow-natives-syntax --validate-asm
 
 function __f_76() {
   "use asm";
@@ -13,7 +13,8 @@ function __f_76() {
 }
 
 try {
-  Wasm.instantiateModuleFromAsm( __f_76.toString());
+  assertTrue(%IsAsmWasmCode(__f_76) ||
+             %GetOptimizationStatus(__f_76) === 3);

[titzer, 2016/08/22 08:37:51]

Maybe we want a runtime call that encapsulates this predicate, since it's
proliferated into a lot of regression tests?

[Michael Starzinger, 2016/08/22 11:45:59]

+1. Also if/when we change the pipeline to avoid optimizing asm.js module
instantiation code via --always-opt in the first place, we don't want to touch
all this places again. Could we just make the %IsAsmWasmCode predicate return
true in --always-opt mode for now? You can leave a TODO(mstarzinger) there.

[bradnelson, 2016/08/23 04:04:55]

Done.

[bradnelson, 2016/08/23 04:04:55]

Done.

   assertTrue(false);
 } catch (e) {
   print("Caught: " + e);