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

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 263 matching lines @@
 
   // Store the top value on the virtual frame into a local frame slot.  The
   // value is left in place on top of the frame.
   void StoreToLocalAt(int index) {
     StoreToFrameSlotAt(local0_index() + index);
   }
 
   // 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.
     ASSERT(index < parameter_count_);
     return Operand(ebp, (1 + parameter_count_ - index) * kPointerSize);
   }
 
   // Push a copy of the value of a parameter frame slot on top of the frame.
   void LoadParameterAt(int index) {
@@ skipping 23 matching lines @@
   // removes from) the top of the physical frame.
   void CallStub(CodeStub* stub, int frame_arg_count);
   Result CallStub(CodeStub* stub, Result* arg, int frame_arg_count);
   Result CallStub(CodeStub* stub,
                   Result* arg0,
                   Result* arg1,
                   int frame_arg_count);
 
   // 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.
   void InvokeBuiltin(Builtins::JavaScript id,
                      InvokeFlag flag,
                      int frame_arg_count);
 
   // Call into a JS code object, given the number of arguments it expects on
   // (and removes from) the top of the physical frame.
   void CallCodeObject(Handle<Code> ic,
@@ skipping 163 matching lines @@
   // Called after all register-to-memory and register-to-register
   // moves have been made.  After this function returns, the frames
   // should be equal.
   void MergeMoveMemoryToRegisters(VirtualFrame *expected);
 };
 
 
 } }  // namespace v8::internal
 
 #endif  // V8_VIRTUAL_FRAME_IA32_H_