Ticket #181 (closed todo: fixed)

Opened 6 years ago

Last modified 6 years ago

Update VTABLE_morph to use a class PMC

Reported by: whiteknight Owned by: Whiteknight
Priority: normal Milestone: 0.9.1
Component: core Version:
Severity: medium Keywords: vtable morph
Cc: Language:
Patch status: Platform: all

Description

This is a semi-continuation of RT#41825:  http://rt.perl.org/rt3/Ticket/Display.html?id=41825

VTABLE_morph currently uses INTVAL PMC type numbers. I've set up the 'morph' vtable override from PIR to take a string of a class name instead (and am actually going to update that to take a Class PMC instead). This creates a disconnect between what's present in PIR and what's used internally in C.

Been talking to pmichaud, and he says that VTABLE_morph should use a class PMC also. Ideally, the morph opcode should take the same kinds of parameters as the get_class opcode, although that is currently in limbo. See #8 for details about that work.

I was doing some work on this in the morph_pmc_type branch, but I'm going to temporarily abandon that to focus my energy on the get_class issue first.

Change History

Changed 6 years ago by whiteknight

  • status changed from new to closed
  • resolution set to fixed

Should have been fixed with the merge of the vtable_morph_change branch last night. Infinoid++ for his help with the messy merger. Closing ticket

Note: See TracTickets for help on using tickets.