The default bug view has changed. See this FAQ.

IonMonkey: Assertion failure: opd->type() == phi->type(), at ion/Lowering.cpp:1302

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
major
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: decoder, Assigned: dvander)

Tracking

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

Other Branch
x86_64
Linux
assertion, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The following testcase asserts on ionmonkey revision c34398f961e7 (run with --ion -n -m), tested on 64 bit:


function testInt(n, result) {
    var x = 0;
    for (var i = 0; i < 15; i++) {
        if (x % 2 == 0)
            x = 10;
        else
            x %=  0;
    }
    for (var i = 0; i < 15; i++) {    }
}
testInt(2147483647, 2147483647);
testInt(-2147483648, -2147483648);
(Assignee)

Comment 1

5 years ago
Created attachment 595608 [details] [diff] [review]
fix

Don't fold MMod when doing so could change its result type.
Assignee: general → dvander
Status: NEW → ASSIGNED
Attachment #595608 - Flags: review?(jdemooij)

Updated

5 years ago
Attachment #595608 - Flags: review?(jdemooij) → review+
(Assignee)

Comment 2

5 years ago
http://hg.mozilla.org/projects/ionmonkey/rev/fa22733dd172
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 3

4 years ago
A testcase for this bug was automatically identified at js/src/jit-test/tests/ion/bug725061.js.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.