Closed Bug 764792 Opened 12 years ago Closed 12 years ago

IonMonkey: Crash [@ js::ion::TypeInferenceOracle::binaryOp] or "Assertion failure: false (Bad SSA value),"

Categories

(Core :: JavaScript Engine, defect)

Other Branch
defect
Not set
critical

Tracking

()

RESOLVED FIXED

People

(Reporter: decoder, Assigned: dvander)

References

Details

(Keywords: assertion, regression, testcase, Whiteboard: [jsbugmon:update])

Crash Data

Attachments

(1 file)

The following testcase asserts on ionmonkey revision 7ab88528503e (run with --ion -n -m --ion-eager):


test();
function test() {
  for (var i=0; i<2; ++i) {};
  try {}  catch ([ q ]) {
      function g() {}
  }     
}
The assertion has changed to Assertion failure: false (Bad SSA value), and there is an opt crash at js::ion::TypeInferenceOracle::binaryOp when the testcase in comment 0 is run with --ion-eager on 10.6, tested on IonMonkey changeset 6688ede89a36.

autoBisect shows this is probably related to the following changeset:

The first bad revision is:
changeset:   97982:b2223a34e849
user:        David Anderson
date:        Thu Jun 14 01:21:31 2012 -0700
summary:     Add support for CallObjects and heavyweight functions (bug 761685, r=jandem,luke).
Blocks: 761685
Severity: major → critical
Crash Signature: [@ js::ion::TypeInferenceOracle::binaryOp]
Keywords: regression
OS: Linux → All
Hardware: x86 → All
Summary: IonMonkey: Assertion failure: Bad SSA value, at ../jsanalyze.h:1070 → IonMonkey: Crash [@ js::ion::TypeInferenceOracle::binaryOp] or "Assertion failure: false (Bad SSA value),"
Attached patch fixSplinter Review
Bug is that we're misusing the TI API a bit. If the local isn't tracked we can't asked for pushedTypes.
Assignee: general → dvander
Status: NEW → ASSIGNED
Attachment #639516 - Flags: review?(bhackett1024)
Attachment #639516 - Flags: review?(bhackett1024) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/13df3c40ef7a
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
A testcase for this bug was automatically identified at js/src/jit-test/tests/ion/bug764792.js.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.