Note: There are a few cases of duplicates in user autocompletion which are being worked on.

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

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: decoder, Assigned: dvander)

Tracking

(Blocks: 3 bugs, {assertion, regression, testcase})

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

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [jsbugmon:update], crash signature)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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),"
Blocks: 349611
(Assignee)

Comment 2

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

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
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

5 years ago
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.