Closed Bug 474450 Opened 16 years ago Closed 6 years ago

LIR needs to mark dxns live in epilogue

Categories

(Tamarin Graveyard :: Baseline JIT (CodegenLIR), defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX
Future

People

(Reporter: stejohns, Unassigned)

Details

(Whiteboard: needs-test-case, loose-end)

Attachments

(1 file)

Attached patch PatchSplinter Review
dxns wasn't being kept live, so in some situations it was being recycled for other temp storage, causing havok in code that used E4X defaultXmlNamespace.
Attachment #357816 - Flags: review?(edwsmith)
Attachment #357816 - Flags: review?(edwsmith) → review+
test case needed before closing the bug
agreed -- haven't seen it in avmshell, bug was found in Flash integration
Flags: in-testsuite?
pushed to redux as changeset:   1318:828ce1fdff63
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
Steven, can this be closed?
Edwin suggested we keep it open until we can come up with a testcase  that actually tests it in avmshell (as opposed to Flash)
Ooohhhh.... the new assembler to the rescue?
Flags: flashplayer-qrb? → flashplayer-qrb+
Priority: -- → P3
Target Milestone: --- → flash10.x
Priority: P3 → --
Target Milestone: flash10.1 → Future
Whiteboard: Needs TC
Component: Virtual Machine → JIT Compiler (NanoJIT)
Whiteboard: Needs TC → needs-test-case
How to add the requested test case? Can the assembler help with this?
Assignee: stejohns → nobody
Flags: flashplayer-needsbackport-
Whiteboard: needs-test-case → needs-test-case, loose-end
If the offending function in a SWF can be found, then it can be isolated and compiled with ABC.  Or, a selftest-style test case can be written that analyzes the LIR via brute force.
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: