IonMonkey: Assertion failure: enumerators == cx->enumerators, at jsinterp.cpp:464 or Crash [@ js::UnwindIteratorForUncatchableException]

RESOLVED FIXED

Status

()

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

People

(Reporter: decoder, Assigned: dvander)

Tracking

(Blocks: 2 bugs, {assertion, testcase})

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

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [jsbugmon:update])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The following testcase asserts on ionmonkey revision e8de64e7e9fe (run with --ion -n -m --ion-eager):


(function() {
  var a, b;
  for each (a in [{}, {__iterator__: function(){}}]) 
    for (b in a) { }
})();
(Assignee)

Updated

5 years ago
Assignee: general → dvander
Status: NEW → ASSIGNED
(Assignee)

Comment 1

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

Hoping this will fix some of the other GC bugs we've encountered. Two bugs:

 (1) PushRegsInMask stores in the wrong order.
 (2) MarkIonFrames didn't always decrement the spill position.
Attachment #623881 - Flags: review?(nicolas.b.pierron)
Comment on attachment 623881 [details] [diff] [review]
fix

Review of attachment 623881 [details] [diff] [review]:
-----------------------------------------------------------------

Good catches.
Attachment #623881 - Flags: review?(nicolas.b.pierron) → review+
(Assignee)

Updated

5 years ago
Duplicate of this bug: 754715
(Assignee)

Comment 4

5 years ago
checked in with a thinko fixed:  http://hg.mozilla.org/projects/ionmonkey/rev/d3593b87e0af
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Updated

5 years ago
Duplicate of this bug: 754717
(Assignee)

Updated

5 years ago
Duplicate of this bug: 746364
(Reporter)

Comment 7

5 years ago
A testcase for this bug was automatically identified at js/src/jit-test/tests/ion/bug754718.js.
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.