1615016 - Store isAtomsZone and isSelfHostingZone flags on the Zone

Status: RESOLVED FIXED

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

Description

[Jon Coppeard (:jonco)]

At the moment calling isAtomsZone() on a Zone has to check with the runtime. We should just store this as a flag on the Zone (and similarly for isSelfHostingZone). This will remove some dependencies on JSRuntime.

Comment 1

[Jon Coppeard (:jonco)]

Attachment: Bug 1615016 - Store isAtomsZone and isSelfHostingZone flags on the Zone r?sfink

Remove some dependencies on JSRuntime by storing flags on the Zone to indicate whether a zone is the atoms zone or the self hosting zone.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c36521288f85
Store isAtomsZone and isSelfHostingZone flags on the Zone r=sfink

Comment 3

[Arthur Iakab [arthur_iakab]]

Backed out changeset c36521288f85 (bug 1615016) for causing spidermonkey bustages on GC-inl.h

Backout revision https://hg.mozilla.org/integration/autoland/rev/a1579bfa6b7b93b219942da5929c3e5b911fe18c

Failed push https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedJob=288704699&searchStr=linux%2Cx64%2Cdebug%2Cspidermonkey%2Cbuilds%2Cspidermonkey-sm-nonunified-linux64%2Fdebug%2Csm%28nu%29&revision=c36521288f851bf84592a072c28e0e2c6aacaafa

Failure logs https://treeherder.mozilla.org/logviewer.html#?job_id=288704699&repo=autoland

Jon can you please take a look?

Comment 4

[Jon Coppeard (:jonco)]

Sorry about that.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bcd88da6a9c6
Store isAtomsZone and isSelfHostingZone flags on the Zone r=sfink

Comment 6

[Cristian Brindusan [:cbrindusan]]

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