If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Combine MarkAndSweep/MarkAndSweepCompartment

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: gwagner, Assigned: gwagner)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment, 3 obsolete attachments)

(Assignee)

Description

7 years ago
These functions are mostly redundant and it happened to me more than once that I only changed one function...
(Assignee)

Comment 1

7 years ago
Created attachment 521701 [details] [diff] [review]
patch
Assignee: general → anygregor
Great! Let's get this in soon.

One thing I noticed: when moving shapes to the GC heap, I forgot to take out the TIMESTAMP code for shape sweeping. It doesn't really time anything anymore. This might be a good opportunity to take it out.
(Assignee)

Comment 3

7 years ago
(In reply to comment #2)
> Great! Let's get this in soon.
> 
> One thing I noticed: when moving shapes to the GC heap, I forgot to take out
> the TIMESTAMP code for shape sweeping. It doesn't really time anything anymore.
> This might be a good opportunity to take it out.

We could still time the shape finalization if we move it out of finalizeObjectArenaLists.


Brendan, is there some magic behind DumpLoopStats and DumpScopeDepthMeter? I can't find an implementation for them. Is it safe to remove?
(Assignee)

Comment 4

7 years ago
Created attachment 521728 [details] [diff] [review]
patch
Attachment #521701 - Attachment is obsolete: true
(Assignee)

Comment 5

7 years ago
Created attachment 521868 [details] [diff] [review]
patch

I restored the shape-finalization timing again. 
Otherwise there shouldn't be any change to the previous code.
Attachment #521728 - Attachment is obsolete: true
Attachment #521868 - Flags: review?(wmccloskey)
(Assignee)

Comment 6

7 years ago
Created attachment 521882 [details] [diff] [review]
patch

Now for real...
Attachment #521868 - Attachment is obsolete: true
Attachment #521868 - Flags: review?(wmccloskey)
Attachment #521882 - Flags: review?(wmccloskey)
Comment on attachment 521882 [details] [diff] [review]
patch

Could you also take out the sentence about how "We finalize iterators before other objects..."? As far as I can tell, that's just not true any more.
Attachment #521882 - Flags: review?(wmccloskey) → review+
(Assignee)

Comment 8

7 years ago
http://hg.mozilla.org/tracemonkey/rev/5c7a2ced20f8
Whiteboard: fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/5c7a2ced20f8
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.