Fix x64 patching of asllocation stub: the pool pointer must be set-up before patching jump is execu…

runtime/vm/stub_code_x64.cc

Index: runtime/vm/stub_code_x64.cc
===================================================================
--- runtime/vm/stub_code_x64.cc	(revision 40325)
+++ runtime/vm/stub_code_x64.cc	(working copy)
@@ -1067,8 +1067,16 @@
 // Input parameters:
 //   RSP + 8 : type arguments object (only if class is parameterized).
 //   RSP : points to return address.
-uword StubCode::GenerateAllocationStubForClass(Assembler* assembler,
-                                              const Class& cls) {
+void StubCode::GenerateAllocationStubForClass(
+    Assembler* assembler, const Class& cls,
+    uword* entry_patch_offset, uword* patch_code_pc_offset) {
+  // Must set-up a PP before being able to patch.
+  Register new_pp = R13;
+  Register new_pc = R12;
+
+  __ SetupPP(new_pp, new_pc);

[zra, 2014/09/16 22:18:01]

Looking at this more closely, I think this could actually just be:

__ LoadPoolPointer(new_pp)

The only reason EmitFrameEntry does something else is to avoid duplicating work
creating the PC marker. Sorry for not remembering this earlier.

[srdjan, 2014/09/16 22:42:43]

Reverting the change in assembler (SetupPP), using LoadPoolPointer.

+  *entry_patch_offset = assembler->CodeSize();
+
   const intptr_t kObjectTypeArgumentsOffset = 1 * kWordSize;
   // The generated code is different if the class is parameterized.
   const bool is_cls_parameterized = cls.NumTypeArguments() > 0;
@@ -1180,10 +1188,9 @@
   // Restore the frame pointer.
   __ LeaveStubFrame();
   __ ret();
-  uword patch_code_pc_offset = assembler->CodeSize();
+  *patch_code_pc_offset = assembler->CodeSize();
   StubCode* stub_code = Isolate::Current()->stub_code();
-  __ JmpPatchable(&stub_code->FixAllocationStubTargetLabel(), R13);
-  return patch_code_pc_offset;
+  __ JmpPatchable(&stub_code->FixAllocationStubTargetLabel(), new_pp);
 }