Assertion failure: !types->unknown(), at ../ion/MIR.h:7396

RESOLVED FIXED in Firefox 24

Status

()

Product:
Core
Component:
JavaScript Engine
Type:
defect
Importance:
--
critical
Status:
RESOLVED FIXED
Opened:
6 years ago
Updated:
6 years ago

People

(Reporter: decoder, Assigned: h4writer)

Tracking

(Blocks 1 bug, {assertion, testcase})

Version:
Trunk
Target:
mozilla25
Platform:
x86_64
Linux
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox24 fixed, firefox25 fixed)

Details

(Whiteboard: [jsbugmon:update])

Attachments

(2 attachments)

[crash-signature] Machine-readable crash signature
728 bytes, text/plain
Details
Patch
1.57 KB, patch
bhackett
: review+
bajaj
: approval-mozilla-aurora+
Details | Diff | Splinter Review 
The following testcase asserts on mozilla-central revision 18467a85acf6 (run with --fuzzing-safe --ion-eager):


function f() {}
new EvalTest();
function EvalTest() {
  with (this) {
    f(EvalTest)
  }
}
evaluate("var obj = new f(1, 'x');");
Whiteboard: [jsbugmon:update,bisect]
Whiteboard: [jsbugmon:update,bisect] → [jsbugmon:update]
JSBugMon: Bisection requested, result:
autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   http://hg.mozilla.org/mozilla-central/rev/436c88ed1e5e
user:        Hannes Verschore
date:        Tue Jun 11 16:00:54 2013 +0200
summary:     Bug 879168 - IonMonkey: Improve the typeset of |this| when inlining a constructing call, r=bhackett

This iteration took 326.199 seconds to run.
Needinfo from Hannes based on comment 2 :)
Flags: needinfo?(hv1989)
Posted patch Patch — — Splinter Review
Assignee: general → hv1989
Attachment #775942 - Flags: review?(bhackett1024)
Flags: needinfo?(hv1989)
Attachment #775942 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/84e608715045
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
Comment on attachment 775942 [details] [diff] [review]
Patch

[Approval Request Comment]

Bug caused by (feature/regressing bug #): Bug 879168

User impact if declined: I think nothing too crazy. But there is a reason there is an assert there. Not sure if it is due to crashes or bailouts, though.

Testing completed (on m-c, etc.): m-c for the weekend

Risk to taking this patch (and alternatives if risky): I would say no risk. Very easy and understandable patch.

String or IDL/UUID changes made by this patch: /
Attachment #775942 - Flags: approval-mozilla-aurora?
Attachment #775942 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.