Assertion failure: bce->sc->funIsHeavyweight(), at frontend/BytecodeEmitter.cpp:886

RESOLVED FIXED in mozilla16

Status

()

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

People

(Reporter: decoder, Assigned: luke)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla16
x86
Linux
assertion, testcase
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [js:t][jsbugmon:update])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The following test asserts on mozilla-central revision 19bfe36cace8 (options -m -n -a):


function loop( p = 1, actual = this)  {
  for (var i = 0; 
        (function() { 
        actual += this.assertEq + ' toString, '; 
        }); 
  ++i) { }
}
(Assignee)

Comment 1

5 years ago
Created attachment 634475 [details] [diff] [review]
rm assert and add test

Oops, the assert is overly conservative: the heavyweight flag can be set later in rare cases, but the only important thing is that it gets set.

On a side note, the way the heavyweight flag gets set (and closed vars get noted) is pretty awkward.
Assignee: general → luke
Status: NEW → ASSIGNED
Attachment #634475 - Flags: review?(jorendorff)
Whiteboard: js-triage-needed [jsbugmon:update] → [js:t][jsbugmon:update]
Attachment #634475 - Flags: review?(jorendorff) → review+
(Assignee)

Comment 2

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6c5677e58de
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/e6c5677e58de
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Reporter)

Comment 4

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