Closed Bug 1755534 Opened 2 years ago Closed 2 years ago

Remove permanent atom checks from string marking code

Categories

(Core :: JavaScript: GC, task, P3)

Product:
Core
Component:
JavaScript: GC
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
99 Branch
Tracking Flags:
Tracking Status
firefox99 --- fixed

People

(Reporter: jonco, Assigned: jonco)

References

Details

Attachments

(1 file)

Bug 1755534 - Remove remaining permanent atom checks from the marking code r?sfink
48 bytes, text/x-phabricator-request
Details | Review

I noticed that we still check whether strings are permanent atoms in a few places in the marking code. This is no longer necessary since bug 1734801 ensured that permanent atoms are marked all the time and so are no longer traced through.

Pushed by jcoppeard@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/537589c3f80d
Remove remaining permanent atom checks from the marking code r=sfink

https://hg.mozilla.org/mozilla-central/rev/537589c3f80d

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox99: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Regressions: 1856739
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: