1633405 - Remove dynamic GC options that are enabled everywhere

Status: RESOLVED FIXED

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

Description

[Jon Coppeard (:jonco)]

There are a couple of GC options - dynamic heap growth and dynamic mark slice - that are enabled everywhere (although there are prefs to disable them).

We should probably just remove these and simplify the code.

One reason for disable the dynamic heap growth options is to make the GC more deterministic, but we can handle that by never going into high frequency GC mode if JS_MORE_DETERMINISTIC is defined.

Comment 1

[Jon Coppeard (:jonco)]

Attachment: Bug 1633405 - Remove dynamic GC options that are enabled everywhere r?sfink

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1fc50aa5c6bc
Remove dynamic GC options that are enabled everywhere r=sfink

Comment 3

[Noemi Erli[:noemi_erli]]

Backed out changeset 1fc50aa5c6bc (Bug 1633405) for causing bustages in bigint.js CLOSED TREE

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedTaskRun=dG9u7NpZSPaplGQ1LYHY1w-0&resultStatus=testfailed%2Cbusted%2Cexception&revision=1fc50aa5c6bc442e19c9feae8d8c3bf60af427d3

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=299749848&repo=autoland&lineNumber=46097

Backout: https://hg.mozilla.org/integration/autoland/rev/15f0dc9f73e62d8e359356d7ff3522c22e4293b9

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c39d9152c793
Remove dynamic GC options that are enabled everywhere r=sfink

Comment 5

[Noemi Erli[:noemi_erli]]

This was relanded since the issues seems to be cause by Bug 1608771
Reland: https://hg.mozilla.org/integration/autoland/rev/c39d9152c79368aff7b59c1bdc61f8b6b73277ff

Comment 6

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/c39d9152c793