First part of static call cleanup: store static call targets into code object, referenced by code-o…

runtime/vm/native_entry_test.cc

Index: runtime/vm/native_entry_test.cc
===================================================================
--- runtime/vm/native_entry_test.cc	(revision 15037)
+++ runtime/vm/native_entry_test.cc	(working copy)
@@ -58,14 +58,18 @@
 // Test code patching.
 void TestStaticCallPatching(Dart_NativeArguments args) {
   Dart_EnterScope();
-  uword target_address = 0;
   Function& target_function = Function::Handle();
   DartFrameIterator iterator;
   iterator.NextFrame();  // Skip native call.
   StackFrame* static_caller_frame = iterator.NextFrame();
+  uword target_address = 0;

[siva, 2012/11/16 23:15:23]

TODO that you will replace GetStaticCallAt with
Code::GetStaticCallTaregtAtOffset(...)

One performance point here, it would be more optimal to
get the code object from the stack frame than by using
Code::LookupCode(pc) inside Code::GetStaticCallTargetAt(..).

You could write this code as:
const Code& code = Code::Handle(static_call_frame->LookupDartCode());
uword offset = (static_caller_frame->pc() - code.EntryPoint());
const Function& target_function_2 =
Function::Handle(code.GetStaticCallTargetAtOffset(offset));

[srdjan, 2012/11/17 00:47:24]

Addressed as discussed

   CodePatcher::GetStaticCallAt(static_caller_frame->pc(),
                                &target_function,
                                &target_address);
+  const Function& target_function_2 =
+      Function::Handle(Code::GetStaticCallTargetAt(static_caller_frame->pc()));
+  EXPECT(!target_function_2.IsNull());
+  EXPECT_EQ(target_function_2.raw(), target_function.raw());
   EXPECT(String::Handle(target_function.name()).
       Equals(String::Handle(String::New("NativePatchStaticCall"))));
   const uword function_entry_address =