Make VirtualFrame::CallStub on IA32 responsible for moving arguments...

src/virtual-frame-ia32.h

 // Copyright 2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 223 matching lines @@
   void StoreToParameterAt(int index) {
     StoreToFrameSlotAt(param0_index() + index);
   }
 
   // The receiver frame slot.
   Operand Receiver() const { return ParameterAt(-1); }
 
   // Push a try-catch or try-finally handler on top of the virtual frame.
   void PushTryHandler(HandlerType type);
 
-  // Call a code stub, given the number of arguments it expects on (and
-  // removes from) the top of the physical frame.
+  // Call stub given the number of arguments it expects on (and
+  // removes from) the stack.
   Result CallStub(CodeStub* stub, int arg_count);
-  Result CallStub(CodeStub* stub, Result* arg, int arg_count);
-  Result CallStub(CodeStub* stub, Result* arg0, Result* arg1, int arg_count);
 
-  // Call the runtime, given the number of arguments expected on (and
-  // removed from) the top of the physical frame.
+  // Call stub that takes a single argument passed in eax.  The
+  // argument is given as a result which does not have to be eax or
+  // even a register.  The argument is consumed by the call.
+  Result CallStub(CodeStub* stub, Result* arg);
+
+  // Call stub that takes a pair of arguments passed in edx (arg0) and
+  // eax (arg1).  The arguments are given as results which do not have
+  // to be in the proper registers or even in registers.  The
+  // arguments are consumed by the call.
+  Result CallStub(CodeStub* stub, Result* arg0, Result* arg1);
+
+  // Call runtime given the number of arguments expected on (and
+  // removed from) the stack.
   Result CallRuntime(Runtime::Function* f, int arg_count);
   Result CallRuntime(Runtime::FunctionId id, int arg_count);
 
-  // Invoke a builtin, given the number of arguments it expects on (and
-  // removes from) the top of the physical frame.
+  // Invoke builtin given the number of arguments it expects on (and
+  // removes from) the stack.
   Result InvokeBuiltin(Builtins::JavaScript id,
                        InvokeFlag flag,
                        int arg_count);
 
   // Call load IC.  Name and receiver are found on top of the frame.
   // Receiver is not dropped.
   Result CallLoadIC(RelocInfo::Mode mode);
 
   // Call keyed load IC.  Key and receiver are found on top of the
   // frame.  They are not dropped.
@@ skipping 203 matching lines @@
   Result RawCallCodeObject(Handle<Code> code, RelocInfo::Mode rmode);
 
   bool Equals(VirtualFrame* other);
 
   friend class JumpTarget;
 };
 
 } }  // namespace v8::internal
 
 #endif  // V8_VIRTUAL_FRAME_IA32_H_