[wasm] Allow import function to be any kind of callables.

test/mjsunit/wasm/ffi.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --expose-wasm
 
 load("test/mjsunit/wasm/wasm-constants.js");
 load("test/mjsunit/wasm/wasm-module-builder.js");
 
 function testCallFFI(func, check) {
@@ skipping 36 matching lines @@
     assertEquals(a - b | 0, r);
     assertTrue(was_called);
 //    assertEquals(global, params[0]);  // sloppy mode
     assertEquals(a, params[1]);
     assertEquals(b, params[2]);
     was_called = false;
 }
 
 testCallFFI(FOREIGN_SUB, check_FOREIGN_SUB);
 
+var proxy_sub = new Proxy(FOREIGN_SUB, {
+});
+testCallFFI(proxy_sub, check_FOREIGN_SUB);

[Benedikt Meurer, 2016/08/03 17:14:43]

I'd like to see appropriate tests for all the special cases, i.e. bound
functions, class constructors, special callables (propably requires a cctest in
test-api.cc), and also native functions, i.e. our EcmaScript builtins making
sure they get undefined for receiver and not some global proxy.

 
 function FOREIGN_ABCD(a, b, c, d) {
   print("FOREIGN_ABCD(" + a + ", " + b + ", " + c + ", " + d + ")");
   was_called = true;
   params[0] = this;
   params[1] = a;
   params[2] = b;
   params[3] = c;
   params[4] = d;
   return (a * b * 6) | 0;
@@ skipping 184 matching lines @@
       kExprCallImport, kArity1, 1   // --
     ])        // --
     .exportFunc()
 
   var main = builder.instantiate({print: print}).exports.main;
   for (var i = -9; i < 900; i += 6.125) main(i);
 }
 
 testCallPrint();
 testCallPrint();