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)
Tamarin Graveyard
Virtual Machine
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: edwsmith, Unassigned)
Details
Attachments
(1 file)
16.96 KB,
patch
|
Details | Diff | Splinter Review |
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.
Reporter | ||
Updated•12 years ago
|
Attachment #605511 -
Attachment is patch: true
Comment 1•6 years ago
|
||
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.
Description
•