Closed Bug 663573 Opened 13 years ago Closed 13 years ago

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

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: adrake, Assigned: adrake)

Details

Attachments

(1 file)

Attached patch Patch v0Splinter Review
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.
Attachment #538650 - Flags: review?(dvander)
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+
http://hg.mozilla.org/users/danderson_mozilla.com/ionmonkey/rev/92d0bcca27d2
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.