bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

IonMonkey: Assertion failure: numSuccessorsWithPhis == 1, at IonAnalysis.cpp:494

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: adrake, Assigned: adrake)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 538650 [details] [diff] [review]
Patch v0

Test case:

function loop() {
 do { } while (0);
}
loop();

Yields assert in summary. The patch changes the assert to be <= 1, as some blocks may not have any phis.
(Assignee)

Updated

7 years ago
Attachment #538650 - Flags: review?(dvander)
(Assignee)

Updated

7 years ago
Assignee: general → adrake
Comment on attachment 538650 [details] [diff] [review]
Patch v0

Phew, I thought our invariant was broken!
Attachment #538650 - Flags: review?(dvander) → review+
(Assignee)

Comment 2

7 years ago
http://hg.mozilla.org/users/danderson_mozilla.com/ionmonkey/rev/92d0bcca27d2
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.