Closed
Bug 449999
Opened 16 years ago
Closed 16 years ago
TM: Intern globals per context, not per tree.
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: gal, Assigned: gal)
References
Details
Attachments
(3 obsolete files)
Currently we lazily intern globals into the global frame per-tree. This makes it difficult to switch between trees since we have to unbox different globals onto the global frame. Instead, track which global object is currently associated with the current context, and lazily add globals that any tree uses to a per-context list. All trees we activate unbox all globals in the list, allowing us to easily transition between trees without having to re-box or unbox any globals during said transition.
Assignee | ||
Comment 1•16 years ago
|
||
Assignee | ||
Comment 3•16 years ago
|
||
This is a fix for 450535, but it doesn't solve the whole problem (449999) yet. http://hg.mozilla.org/index.cgi/tracemonkey/rev/cad6ca9a1e79
Assignee | ||
Comment 4•16 years ago
|
||
Confirmed to fix yahoo.
Assignee | ||
Comment 5•16 years ago
|
||
Two additional fixes: http://hg.mozilla.org/index.cgi/tracemonkey/rev/6c6266b8311c http://hg.mozilla.org/index.cgi/tracemonkey/rev/735b68bb8168 Globals are still per-tree, but all corner cases are intercepted and we should be sound (albeit not optimal wrt speed). I will keep working on the patch for the globals, but this is no longer an urgent blocker.
Status: NEW → ASSIGNED
Assignee | ||
Updated•16 years ago
|
Assignee: general → gal
Status: ASSIGNED → NEW
Assignee | ||
Updated•16 years ago
|
Attachment #333144 -
Attachment is obsolete: true
Assignee | ||
Comment 6•16 years ago
|
||
Current state. Passes trace-tests but fails math-partial.
Assignee | ||
Comment 7•16 years ago
|
||
Attachment #334492 -
Attachment is obsolete: true
Assignee | ||
Comment 8•16 years ago
|
||
Pushed. Reviews + testing welcome. http://hg.mozilla.org/index.cgi/tracemonkey/rev/fca2a565f6c1 http://hg.mozilla.org/index.cgi/tracemonkey/rev/ab683b57667d
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Attachment #334527 -
Attachment is obsolete: true
Updated•15 years ago
|
Flags: in-testsuite-
Flags: in-litmus-
You need to log in
before you can comment on or make changes to this bug.
Description
•