Closed Bug 1864090 Opened 11 months ago Closed 11 months ago

Expand js::debug::GetMarkInfo() to accept arbitrary pointers

Categories

(Core :: JavaScript: GC, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: sfink, Assigned: sfink)

References

Details

Attachments

(1 file)

I find it very convenient to call js::debug::GetMarkInfo on random Cell*s to see whether they're nursery or tenured and, if tenured, how they are marked.

With string buffers in the nursery, it would also be nice if it could accept an arbitrary pointer and say whether it was in the nursery or not. (If not, it is probably malloced, but could be static data or the stack or lots of other things.)

Assignee: nobody → sphink
Status: NEW → ASSIGNED
Assignee: sphink → nobody
Status: ASSIGNED → NEW
Summary: Expand JS::debug::markinfo() to accept arbitrary pointers → Expand js::debug::GetMarkInfo() to accept arbitrary pointers
Assignee: nobody → sphink
Status: NEW → ASSIGNED
Pushed by sfink@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/590a70641e5b Add MarkInfo::UNKNOWN to the (debug-only) output possibilities (instead of falling back to NURSERY most of the time) r=jonco

Backed out for causing bustage on Marking.cpp

Backout link

Push with failures

Failure log

Flags: needinfo?(sphink)
Pushed by sfink@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/1bf9a3824941 Add MarkInfo::UNKNOWN to the (debug-only) output possibilities (instead of falling back to NURSERY most of the time) r=jonco
Flags: needinfo?(sphink)
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: