Unwind the complex conditional around atoms marking

RESOLVED FIXED in Firefox 51

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

Trunk
mozilla51
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 affected, firefox51 fixed)

Details

Attachments

(1 attachment)

After everything gets evaluated, this gets run from traceRuntime and traceRuntimeForMajorGC(if tracing the atoms zone). We can now represent this fairly directly.
Attachment #8776150 - Flags: review?(jcoppeard)
Blocks: 1290603
Comment on attachment 8776150 [details] [diff] [review]
6_lift_atoms_tracing-v0.diff

Review of attachment 8776150 [details] [diff] [review]:
-----------------------------------------------------------------

Much nicer.
Attachment #8776150 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/9db0bc3c54ce
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.