Closed Bug 766064 Opened 11 years ago Closed 11 years ago

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

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla16

People

(Reporter: decoder, Assigned: luke)

Details

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

Attachments

(1 file)

rm assert and add test
1.48 KB, patch
jorendorff
: review+
Details | Diff | Splinter Review 
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) { }
}
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+
https://hg.mozilla.org/mozilla-central/rev/e6c5677e58de
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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.