FFI Compiler based on code stub assembler

src/compiler/ffi-compiler.cc

Index: src/compiler/ffi-compiler.cc
diff --git a/src/compiler/ffi-compiler.cc b/src/compiler/ffi-compiler.cc
new file mode 100644
index 0000000000000000000000000000000000000000..586894a0b2879ae47060361c613a0fa3d5e1b3b8
--- /dev/null
+++ b/src/compiler/ffi-compiler.cc
@@ -0,0 +1,107 @@
+// Copyright 2016 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.
+
+#include "src/compiler/ffi-compiler.h"
+
+#include "src/isolate-inl.h"
+
+#include "src/api.h"
+#include "src/code-factory.h"
+#include "src/code-stub-assembler.h"
+#include "src/compilation-info.h"
+#include "src/compiler/linkage.h"
+
+namespace v8 {
+namespace internal {
+namespace compiler {
+
+// TODO(mattloring): find a better home for this.
+static void Setup(Isolate* isolate) {
+  Handle<Context> context(isolate->context());
+  if (!context->get(Context::NATIVE_FUNCTION_MAP_INDEX)->IsMap()) {
+    // TODO(mattloring): move this to boostrapper.cc??
+    Handle<Map> prev_map = Handle<Map>(context->sloppy_function_map(), isolate);
+
+    InstanceType instance_type = prev_map->instance_type();
+    int internal_fields = JSObject::GetInternalFieldCount(*prev_map);
+    CHECK_EQ(0, internal_fields);
+    int pre_allocated =
+        prev_map->GetInObjectProperties() - prev_map->unused_property_fields();
+    int instance_size;
+    int in_object_properties;
+    JSFunction::CalculateInstanceSizeHelper(instance_type, internal_fields, 0,
+                                            &instance_size,
+                                            &in_object_properties);
+    int unused_property_fields = in_object_properties - pre_allocated;
+    Handle<Map> map = Map::CopyInitialMap(
+        prev_map, instance_size, in_object_properties, unused_property_fields);
+    context->set_native_function_map(*map);
+  }
+}
+
+Node* ToJS(CodeStubAssembler* assembler, Node* node, Node* context,
+           MachineType type) {
+  UNREACHABLE();
+  // TODO(mattloring): Needs to be implemented.
+  return nullptr;
+}
+
+Node* FromJS(CodeStubAssembler* assembler, Node* node, Node* context,
+             MachineType type) {
+  UNREACHABLE();
+  // TODO(mattloring): Needs to be implemented.
+  return nullptr;
+}
+
+Handle<JSFunction> CompileJSToNativeWrapper(Isolate* isolate,
+                                            Handle<String> name,
+                                            ffi::NativeFunction func) {
+  Setup(isolate);
+
+  int params = static_cast<int>(func.sig->parameter_count());
+  int returns = static_cast<int>(func.sig->return_count());
+  Handle<SharedFunctionInfo> shared = isolate->factory()->NewSharedFunctionInfo(
+      name, MaybeHandle<Code>(), false);
+  shared->set_length(params);
+  shared->set_internal_formal_parameter_count(params);
+  Handle<JSFunction> function = isolate->factory()->NewFunction(
+      isolate->native_function_map(), name, MaybeHandle<Code>());
+  function->set_shared(*shared);
+
+  Zone zone(isolate->allocator(), ZONE_NAME);
+  int params_with_recv = params + 1;
+  CodeAssemblerState state(isolate, &zone, params_with_recv,
+                           Code::ComputeFlags(Code::FUNCTION), "js-to-native");
+  CodeStubAssembler assembler(&state);
+
+  CallDescriptor* cdesc = Linkage::GetSimplifiedCDescriptor(&zone, func.sig);
+  ApiFunction api_func(func.start);
+  ExternalReference ref(&api_func, ExternalReference::DIRECT_API_CALL, isolate);
+
+  Node* context_param = assembler.Parameter(
+      Linkage::GetJSCallContextParamIndex(params_with_recv));

[Michael Starzinger, 2017/01/16 12:32:00]

Once the FFI-compiler moves to outside of the "compiler" directory, there won't
be a clean way to compute this parameter index. The {FastAccessorAssembler}
already ran into a similar problem. There is a TODO in that regard there:

  // TODO(vogelheim): There is currently no clean way to retrieve the context
  //     parameter for a stub and the implementation details are hidden in
  //     compiler/*. The context_paramter is computed as:
  //       Linkage::GetJSCallContextParamIndex(descriptor->JSParameterCount())

I would be fine with adding a {CodeAssembler::GetJSContextParameter} helper that
just returns the appropriate parameter node. This helper should DCHECK whether
we are generating code with a JS-linkage and then base the index computation on
the parameter-count stored in that linkage.

+
+  Node** inputs = zone.NewArray<Node*>(params + 1);
+  int input_count = 0;
+  inputs[input_count++] = assembler.ExternalConstant(ref);
+  for (int i = 0; i < params; i++) {
+    inputs[input_count++] = FromJS(&assembler, assembler.Parameter(i),
+                                   context_param, func.sig->GetParam(i));
+  }
+
+  Node* call = assembler.CallCFunctionN(cdesc, input_count, inputs);
+  Node* return_val = assembler.UndefinedConstant();
+  if (returns == 1) {
+    return_val = ToJS(&assembler, call, context_param, func.sig->GetReturn());
+  }
+  assembler.Return(return_val);
+
+  Handle<Code> code = CodeAssembler::GenerateCode(&state);
+  function->set_code(*code);
+  return function;
+}
+
+}  // namespace compiler
+}  // namespace internal
+}  // namespace v8