Closed Bug 816779 Opened 7 years ago Closed 7 years ago

GC: add type safety for JSScript

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

This should be simpler to implement than Shape because of the existing Return<T> work.
Depends on: 819118
Attached patch v0Splinter Review
This takes us from over 700 JSScript* down to under under 400.  This is focused mostly under ion/ because IonMonkey has seen the least exposure to the root analysis and uses JSScript* extensively throughout.
Attachment #689415 - Flags: review?(bhackett1024)
Comment on attachment 689415 [details] [diff] [review]
v0

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

Sorry for the delay.
Attachment #689415 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/eb53ab86f515
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.