Experimental: begin using the register allocator across entering and...

src/virtual-frame-ia32.h

Index: src/virtual-frame-ia32.h
===================================================================
--- src/virtual-frame-ia32.h	(revision 1026)
+++ src/virtual-frame-ia32.h	(working copy)
@@ -281,9 +281,12 @@
   // The function frame slot.
   Operand Function() const { return Operand(ebp, kFunctionOffset); }
 
-  // The context frame slot.
-  Operand Context() const { return Operand(ebp, kContextOffset); }
+  // Save the value of the esi register in the context frame slot.

[William Hesse, 2009/01/06 12:04:08]

Comment should say that that esi is not synced to memory until spilled.
// Save the esi register to the virtual context frame slot.
// The value is not written to memory until synced.
Should the code optimize the case where the context slot is already esi?

[Kevin Millikin (Chromium), 2009/01/06 12:10:28]

Comment changed.

+  void SaveContextRegister();
 
+  // Restore the esi register with the value of the frame context slot.
+  void RestoreContextRegister();
+
   // A parameter as an assembly operand.
   Operand ParameterAt(int index) const {
     ASSERT(-1 <= index);  // -1 is the receiver.
@@ -326,8 +329,8 @@
 
   // Call the runtime, given the number of arguments expected on (and
   // removed from) the top of the physical frame.
-  void CallRuntime(Runtime::Function* f, int frame_arg_count);
-  void CallRuntime(Runtime::FunctionId id, int frame_arg_count);
+  Result CallRuntime(Runtime::Function* f, int frame_arg_count);
+  Result CallRuntime(Runtime::FunctionId id, int frame_arg_count);
 
   // Invoke a builtin, given the number of arguments it expects on (and
   // removes from) the top of the physical frame.