Refactoring various pieces of post interceptor caching.

src/ia32/macro-assembler-ia32.h

Index: src/ia32/macro-assembler-ia32.h
diff --git a/src/ia32/macro-assembler-ia32.h b/src/ia32/macro-assembler-ia32.h
index 02f342eebbb4864d4ac65ee9d560562387fd413f..9e67295afaaf1e7732f966592765f8a22ae580b1 100644
--- a/src/ia32/macro-assembler-ia32.h
+++ b/src/ia32/macro-assembler-ia32.h
@@ -124,6 +124,9 @@ class MacroAssembler: public Assembler {
                       const ParameterCount& actual,
                       InvokeFlag flag);
 
+  void InvokeFunction(JSFunction* function,

[Vitaly Repeshko, 2010/02/12 20:32:40]

I think this should either be named JumpFunction or take an InvokeFlag. Also
this needs a comment.

[Mads Ager (chromium), 2010/02/15 08:04:01]

Agreed.  The usual naming is CallSomething and TailCallSomething.  If you pass
in an InvokeFlag you can call it InvokeFunction.

[antonm, 2010/02/15 08:54:26]

Thanks a lot, Vitaly and Mads, flag added.  Submitting after William's change.

+                      const ParameterCount& actual);
+
   // Invoke specified builtin JavaScript function. Adds an entry to
   // the unresolved list if the name does not resolve.
   void InvokeBuiltin(Builtins::JavaScript id, InvokeFlag flag);
@@ -353,6 +356,9 @@ class MacroAssembler: public Assembler {
   // Convenience function: Same as above, but takes the fid instead.
   void CallRuntime(Runtime::FunctionId id, int num_arguments);
 
+  // Convenience function: call an external reference.
+  void CallExternalReference(ExternalReference ref, int num_arguments);
+
   // Convenience function: Same as above, but takes the fid instead.
   Object* TryCallRuntime(Runtime::FunctionId id, int num_arguments);