1782095 - Try to remove JSContext::freeLists_

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P3)

Description

[Jan de Mooij [:jandem]]

I think post-Stencil we no longer need this to be on the context. This would make setZone a bit faster and simpler.

Comment 1

[Jan de Mooij [:jandem]]

Attachment: Bug 1782095 part 1 - Remove unused isAtomsZone argument from JSContext::setZone. r?jonco!

Comment 2

[Jan de Mooij [:jandem]]

Attachment: Bug 1782095 part 2 - Remove JSContext::freeLists_. r?jonco!

We needed this to support off-thread atom allocation, but we no longer do that
since Stencil.

Depends on D153143

Comment 3

[Jan de Mooij [:jandem]]

Attachment: Bug 1782095 part 3 - Remove JSContext::tenuredAllocsSinceMinorGC_. r?jonco!

It's now simpler to use the counter on the zone directly. It also no longer needs
to be an atomic because we only get/set it on the main thread.

Depends on D153144

Comment 4

[Jan de Mooij [:jandem]]

Attachment: Bug 1782095 part 4 - Remove freeLists argument from refillFreeListAndAllocate. r?jonco!

Depends on D153145

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b8f750002da
part 1 - Remove unused isAtomsZone argument from JSContext::setZone. r=jonco
https://hg.mozilla.org/integration/autoland/rev/92697fcf9823
part 2 - Remove JSContext::freeLists_. r=jonco
https://hg.mozilla.org/integration/autoland/rev/438076b73ef8
part 3 - Remove JSContext::tenuredAllocsSinceMinorGC_. r=jonco
https://hg.mozilla.org/integration/autoland/rev/ad0a95af8bf7
part 4 - Remove freeLists argument from refillFreeListAndAllocate. r=jonco

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/7b8f750002da
https://hg.mozilla.org/mozilla-central/rev/92697fcf9823
https://hg.mozilla.org/mozilla-central/rev/438076b73ef8
https://hg.mozilla.org/mozilla-central/rev/ad0a95af8bf7