Closed
Bug 1587034
Opened 5 years ago
Closed 3 years ago
Audit use of UnsafeTraceManuallyBarrieredEdge and TraceManuallyBarrieredEdge
Categories
(Core :: JavaScript: GC, task, P2)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
People
(Reporter: jonco, Assigned: jonco)
References
Details
Attachments
(1 file, 1 obsolete file)
Following on from bug 1585921, we should make sure that heap edges that are not in containers have the required barriers.
|
Assignee | |
Comment 1•5 years ago
|
||
This removes a few uses but there are still a lot around.
|
|
Assignee | |
Updated•5 years ago
|
Keywords: leave-open
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
||
Updated•5 years ago
|
Attachment #9101337 -
Attachment is obsolete: true
Pushed by jcoppeard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/df7e0599db69 Remove some uses of TraceManuallyBarrieredEdge r=sfink
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
|
||
Comment 5•4 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:jonco, maybe it's time to close this bug?
Flags: needinfo?(jcoppeard)
|
|
Assignee | |
Comment 6•4 years ago
|
||
We still want to do this.
Ted expressed a desire to add more abstractions to handle some of these cases.
Flags: needinfo?(jcoppeard)
|
|
||
Comment 7•4 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:jonco, maybe it's time to close this bug?
Flags: needinfo?(jcoppeard)
|
|
Assignee | |
Comment 8•3 years ago
|
||
Let's close this bug and open another one later to continue this work.
Status: NEW → RESOLVED
Closed: 3 years ago
Flags: needinfo?(jcoppeard)
Keywords: leave-open
Resolution: --- → FIXED
|
|
||
Updated•3 years ago
|
Assignee: nobody → jcoppeard
You need to log in
before you can comment on or make changes to this bug.
Description
•