[wasm] Support validation of asm.js modules with != 3 args.

src/runtime/runtime.h

Index: src/runtime/runtime.h
diff --git a/src/runtime/runtime.h b/src/runtime/runtime.h
index 2dd9d08bc55e12c675ecae29848d8187c041730c..20362c5decd3a1f0017cdb97662cc5b2b596744f 100644
--- a/src/runtime/runtime.h
+++ b/src/runtime/runtime.h
@@ -130,7 +130,7 @@ namespace internal {
   F(CompileForOnStackReplacement, 1, 1)   \
   F(TryInstallOptimizedCode, 1, 1)        \
   F(ResolvePossiblyDirectEval, 6, 1)      \
-  F(InstantiateAsmJs, 4, 1)
+  F(InstantiateAsmJs, -1, 1)

[Michael Starzinger, 2016/08/09 15:48:09]

I would very much vote for keeping the arguments count to this runtime function
fixed to 4 and instead pass sentinel values (e.g. Smi(0) or undefined) when the
builtin cannot provide the values. The implementation of the runtime function is
already resilient enough to deal with arbitrary values for arguments 1, 2 and 3
AFAICT.

[bradn, 2016/08/11 09:19:33]

Indeed.
Done.

 
 #define FOR_EACH_INTRINSIC_DATE(F) \
   F(IsDate, 1, 1)                  \
@@ -880,7 +880,8 @@ namespace internal {
   F(HasFixedFloat32Elements, 1, 1)            \
   F(HasFixedFloat64Elements, 1, 1)            \
   F(HasFixedUint8ClampedElements, 1, 1)       \
-  F(SpeciesProtector, 0, 1)
+  F(SpeciesProtector, 0, 1)                   \
+  F(IsAsmWasmCode, 1, 1)
 
 #define FOR_EACH_INTRINSIC_TYPEDARRAY(F)     \
   F(ArrayBufferGetByteLength, 1, 1)          \