Closed Bug 1585718 Opened 10 months ago Closed 10 months ago

Do Bytecode Emission without allocating BigInt types

Categories

(Core :: JavaScript Engine, task, P1)

Product:
Core
Component:
JavaScript Engine
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla71
Tracking Flags:
Tracking Status
firefox71 --- fixed

People

(Reporter: mgaudet, Assigned: mgaudet)

References

Details

Attachments

(3 files)

Bug 1585718 - Defer BigInt allocation to JSScript construction, after bytecode emission r?tcampbell
47 bytes, text/x-phabricator-request
Details | Review
Bug 1585718 - Store variants inside of GCThings array to support deferred allocation of objects r?tcampbell
47 bytes, text/x-phabricator-request
Details | Review
Bug 1585718 - Minimize external consumption of BigIntBox r?tcampbell
47 bytes, text/x-phabricator-request
Details | Review

We should be able to avoid allocating BigInts until after bytecode emission, saving it to the final script assembly stage.

Attachment #9098318 - Attachment description: ToBecomeBug 1585718 - Defer BigInt allocation to JSScript construction, after bytecode emission rToBetcampbell → Bug 1585718 - Defer BigInt allocation to JSScript construction, after bytecode emission r?tcampbell
Assignee: nobody → mgaudet
Attachment #9098358 - Attachment description: Bug 1585718 - Make GCThings array an array of variants to allow powering deferral r?tcampbell → Bug 1585718 - Store variants inside of GCThings array to support deferred allocation of objects r?tcampbell
Priority: -- → P1
Pushed by mgaudet@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/473102ba0a6a
Store variants inside of GCThings array to support deferred allocation of objects r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/311c90cd217f
Minimize external consumption of BigIntBox r=tcampbell
https://hg.mozilla.org/integration/autoland/rev/770b56e591d5
Defer BigInt allocation to JSScript construction, after bytecode emission r=tcampbell

https://hg.mozilla.org/mozilla-central/rev/473102ba0a6a
https://hg.mozilla.org/mozilla-central/rev/311c90cd217f
https://hg.mozilla.org/mozilla-central/rev/770b56e591d5

Status: NEW → RESOLVED
Closed: 10 months ago
status-firefox71: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Regressions: 1605254
You need to log in before you can comment on or make changes to this bug.