Closed Bug 895647 Opened 7 years ago Closed 7 years ago

Add LazyScript to the static exact rooting analysis

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: terrence, Assigned: sfink)

References

Details

Attachments

(1 file, 1 obsolete file)

It looks like it didn't get added with LazyScript. Since LazyScript doesn't derive JSScript, I think this is just missed by the analysis right now.
Note: this is totally untested.
Attachment #778091 - Flags: review?(bhackett1024)
Attachment #778091 - Flags: review?(bhackett1024) → review+
Attached patch new GC type: js::LazyScript (obsolete) — Splinter Review
(This is already deployed on the production analysis.)
Attachment #778098 - Flags: review?(terrence)
Assignee: terrence → sphink
Comment on attachment 778098 [details] [diff] [review]
new GC type: js::LazyScript

I was too slow!
Attachment #778098 - Attachment is obsolete: true
Attachment #778098 - Flags: review?(terrence)
https://hg.mozilla.org/mozilla-central/rev/4bbb18b109d4
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.