Closed Bug 735414 Opened 12 years ago Closed 6 years ago

Compile intrinsics.abs into avmshell instead of loading separately.

Categories

(Tamarin Graveyard :: Virtual Machine, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: edwsmith, Unassigned)

Details

Attachments

(1 file)

We now have intrinsics.abs in the test suite, containing a lot of wrappers for no-operand opcodes.  Before it landed, I attempted to have these functions compiled into avmshell, in the shell module.

The attempt failed miserably:

1. we have no easily-automatable tool for merging abcs, so i need to hack WAY too much C++ code, to add a second abc to the shell (separate from shell_toplevel.abc)

2. this in turn required several new .cpp files because of symbol conflicts caused by #including two generated .h or .cpp native thunk files in the same source file.

3. furthermore, nativegen.py and startup code in the vm break when there are 0 classes in an ABC.  bleh, I gave up.  Several corner cases will have to be fixed to make it work.
Attachment #605511 - Attachment is patch: true
Tamarin isn't maintained anymore. WONTFIX remaining bugs.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: