Decode ic data and arguments descriptor passed in calls on ARM.

runtime/vm/instructions_arm_test.cc

Index: runtime/vm/instructions_arm_test.cc
===================================================================
--- runtime/vm/instructions_arm_test.cc	(revision 19822)
+++ runtime/vm/instructions_arm_test.cc	(working copy)
@@ -16,13 +16,16 @@
 
 ASSEMBLER_TEST_GENERATE(Call, assembler) {
   __ BranchLinkPatchable(&StubCode::InstanceFunctionLookupLabel());
-  // Do not emit any code after the call above, since we decode backwards
-  // starting from the return address, i.e. from the end of this code buffer.
+  __ Ret();
 }
 
 
 ASSEMBLER_TEST_RUN(Call, test) {
-  CallPattern call(test->entry() + test->code().Size(), test->code());
+  // The return address, which must be the address of an instruction contained
+  // in the code, points to the Ret instruction above, i.e. one instruction
+  // before the end of the code buffer.

[zra, 2013/03/11 22:36:08]

Check why this did not fail before.

[regis, 2013/03/11 23:16:30]

code_patcher_arm verifies the above statement with an assert, but not
instructions_arm. I have now added the assert.

+  CallPattern call(test->entry() + test->code().Size() - Instr::kInstrSize,
+                   test->code());
   EXPECT_EQ(StubCode::InstanceFunctionLookupLabel().address(),
             call.TargetAddress());
 }