Closed Bug 1856295 Opened 1 year ago Closed 1 year ago

Assertion failure: !linearStr->isPermanentAtom() when adding more permanent atoms

Categories

(Core :: JavaScript Engine, task, P2)

Product:
Core
Component:
JavaScript Engine
Type:
task

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1856739

People

(Reporter: arai, Assigned: arai)

References

(Blocks 1 open bug)

Details

(Whiteboard: [sp3])

In bug 1848278 and related bugs, I'm going to add more permanent atoms, for all built-ins and also embedding-provided strings,
which causes the following assertion failure:

https://searchfox.org/mozilla-central/rev/95787423bc1c7ba895ef9c6918feb054866d9f41/js/src/gc/TraceMethods-inl.h#92-93,111

template <uint32_t opts>
void js::GCMarker::eagerlyMarkChildren(JSLinearString* linearStr) {
...
    MOZ_ASSERT(!linearStr->isPermanentAtom());

https://treeherder.mozilla.org/jobs?repo=try&group_state=expanded&revision=eea1c1200bbcc1d27840e5950ff3358f5f6e4b48&selectedTaskRun=FgHl_BLHSmmRRfpw9854Bw.0

it's not clear why the assertion is there.
I'm going to investigate why the assertion doesn't hit in the current code.

Severity: -- → S3
Priority: -- → P2
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Duplicate of bug: 1856739
Resolution: --- → DUPLICATE
Whiteboard: [sp3]
You need to log in before you can comment on or make changes to this bug.