IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with e4x and --enable-more-deterministic

VERIFIED WONTFIX

Status

()

--
critical
VERIFIED WONTFIX
6 years ago
6 years ago

People

(Reporter: gkw, Unassigned)

Tracking

(Blocks: 2 bugs, {assertion, regression, testcase})

Trunk
x86_64
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
Created attachment 693229 [details]
stack

function g() {
    y = newGlobal('')
    return function(c) {
        evalcx(c, y)
    }
}
f = g()
f("\
    x = this;\
    Object.seal(this);\
    options('allow_xml');\
");
f("x.XML");

asserts js debug shell on m-c changeset 2e70b718903a with --ion-eager at Assertion failure: !cx->compartment->activeAnalysis,

autoBisecting..
(Reporter)

Comment 1

6 years ago
This seems to only occur with --enable-more-deterministic builds.
Summary: IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," → IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with --enable-more-determinstic
Whiteboard: [jsbugmon:update]
(Reporter)

Comment 2

6 years ago
autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   109238:a85c0f30cdfa
user:        Jan de Mooij
date:        Thu Oct 04 13:26:16 2012 +0200
summary:     Bug 797185 - Disable DVG stack search when building with --enable-more-deterministic. r=decoder

jandem, is bug 797185 as possible regressor?
Blocks: 797185
Flags: needinfo?(jdemooij)
(Reporter)

Comment 3

6 years ago
I'm going to assume this only occurs with e4x, moreover we do not ship --enable-more-deterministic builds by default, so clearing NEEDINFO flag.

e4x will be obsolete soon, see bug 788293.
Flags: needinfo?(jdemooij)
Summary: IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with --enable-more-determinstic → IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with e4x and --enable-more-determinstic
(Reporter)

Updated

6 years ago
Summary: IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with e4x and --enable-more-determinstic → IonMonkey: "Assertion failure: !cx->compartment->activeAnalysis," with e4x and --enable-more-deterministic
(Reporter)

Comment 4

6 years ago
E4X is now history thanks to bug 788293. -> WONTFIX
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WONTFIX
(Reporter)

Comment 5

6 years ago
Running the testcase through the js shell now throws an error:

822540.js:4:39 Error: unknown option name 'allow_xml'. The valid names are atline, methodjit, methodjit_always, strict, typeinfer, werror and strict_mode.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.