[runtime] Move binary operator fallbacks into the runtime.

src/compiler/interpreter-assembler.h

Index: src/compiler/interpreter-assembler.h
diff --git a/src/compiler/interpreter-assembler.h b/src/compiler/interpreter-assembler.h
index d6a75db89f840dfc7b367c0ae8dddd9e0f47461e..4fbb19cf8f0cc860e8c38c92093b716407a8e29a 100644
--- a/src/compiler/interpreter-assembler.h
+++ b/src/compiler/interpreter-assembler.h
@@ -12,6 +12,7 @@
 #include "src/builtins.h"
 #include "src/frames.h"
 #include "src/interpreter/bytecodes.h"
+#include "src/runtime/runtime.h"
 
 namespace v8 {
 namespace internal {
@@ -86,9 +87,8 @@ class InterpreterAssembler {
   Node* CallIC(CallInterfaceDescriptor descriptor, Node* target, Node* arg1,
                Node* arg2, Node* arg3, Node* arg4, Node* arg5);
 
-  // Call JS builtin.
-  Node* CallJSBuiltin(int context_index, Node* receiver);
-  Node* CallJSBuiltin(int context_index, Node* receiver, Node* arg1);
+  // Call runtime function.
+  Node* CallRuntime(Runtime::FunctionId function_id, Node* arg1, Node* arg2);
 
   // Returns from the function.
   void Return();
@@ -116,6 +116,9 @@ class InterpreterAssembler {
   // Returns a tagged pointer to the current context.
   Node* ContextTaggedPointer();
 
+  // Returns an empty frame (used as a placeholder until we can actuall deopt).

[rmcilroy, 2015/09/10 10:28:25]

We will never be deopting the interpreter bytecode handlers, so this isn't a
placeholder. We should really only be calling runtime functions which can't
cause deopts - what is the situation for the new runtime functions, why do they
need FrameStates? Could we special case them in Linkage::FrameStateInputCount so
that they don't require frame states? 

[Benedikt Meurer, 2015/09/10 10:31:12]

Hm, you're right, the frame state can be empty here, because it's only required
to fullfil the API needs. So what you're suggesting is basically a special
GetRuntimeCallDescriptorForInterpreter, which avoids the frame state input?

[rmcilroy, 2015/09/10 10:43:22]

If it's safe to do this, then yeah I would be fine with it (it could also just
be an option passed to GetRuntimeCallDescriptor since that function already
potentially returns a CallDescriptor::kNoFlags if Linkage::FrameStateInputCount
says the function doesn't need a frame state.

+  Node* EmptyFrameState();
+
   // Returns the offset of register |index| relative to RegisterFilePointer().
   Node* RegisterFrameOffset(Node* index);