Store arguments descriptor in ICData. Remove loading of arguments descriptor at unoptimized call si…

runtime/vm/code_patcher_ia32_test.cc

Index: runtime/vm/code_patcher_ia32_test.cc
===================================================================
--- runtime/vm/code_patcher_ia32_test.cc	(revision 24193)
+++ runtime/vm/code_patcher_ia32_test.cc	(working copy)
@@ -56,13 +56,15 @@
                     true, false, false, false, owner_class, 0));
 
   const String& target_name = String::Handle(String::New("targetFunction"));
-  const ICData& ic_data =
-      ICData::ZoneHandle(ICData::New(function, target_name, 15, 1));
   const Array& args_descriptor =
       Array::ZoneHandle(ArgumentsDescriptor::New(1, Array::Handle()));

[siva, 2013/06/19 23:07:15]

Ditto comment about ZoneHandle.

[srdjan, 2013/06/20 00:12:34]

Done.

+  const ICData& ic_data = ICData::ZoneHandle(ICData::New(function,
+                                                         target_name,
+                                                         args_descriptor,
+                                                         15,
+                                                         1));
 
   __ LoadObject(ECX, ic_data);
-  __ LoadObject(EDX, args_descriptor);
   ExternalLabel target_label(
       "InlineCache", StubCode::OneArgCheckInlineCacheEntryPoint());
   __ call(&target_label);
@@ -73,7 +75,7 @@
 ASSEMBLER_TEST_RUN(IcDataAccess, test) {
   uword return_address = test->entry() + CodePatcher::InstanceCallSizeInBytes();
   ICData& ic_data = ICData::Handle();
-  CodePatcher::GetInstanceCallAt(return_address, test->code(), &ic_data, NULL);
+  CodePatcher::GetInstanceCallAt(return_address, test->code(), &ic_data);
   EXPECT_STREQ("targetFunction",
       String::Handle(ic_data.target_name()).ToCString());
   EXPECT_EQ(1, ic_data.num_args_tested());