TM: "Assertion failure: !ti->stackTypeMap.matches(ti_other->stackTypeMap)" with array comprehension

VERIFIED FIXED

Status

()

defect
--
critical
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: jruderman, Assigned: dvander)

Tracking

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

Trunk
x86
macOS
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

[1 for each (x in {a:1, b:1, c:"", d:"", e:1, f:"", g:""}) if (0)];

Assertion failure: !ti->stackTypeMap.matches(ti_other->stackTypeMap), at jstracer.cpp:2738

This assertion is part of the "Check for duplicate entry type maps" block added as part of the patch in bug 450833.
edited wrong field... 3AM :(
Assignee: general → danderson
covered by js1_7/extensions/regress-455982-01.js, js1_8/extensions/regress-452476.js
Posted patch proposed fixSplinter Review
My logic for detecting demotions was wrong.  It should only demote if typemap=int and the value on the stack is a number and not promotable.
Attachment #345153 - Flags: review?(gal)
Attachment #345153 - Flags: review?(gal) → review+
Pushed changeset: http://hg.mozilla.org/tracemonkey/rev/6ab3538891c4
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 462142
This assertion no longer fires, but there is a new assertion in bug 462142. Not sure if it is really a different bug.
Status: RESOLVED → VERIFIED
http://hg.mozilla.org/mozilla-central/rev/37b3fdbb0f07
/cvsroot/mozilla/js/tests/js1_7/regress/regress-461945.js,v  <--  regress-461945.js
initial revision: 1.1
Flags: in-testsuite+
Flags: in-litmus-
Summary: "Assertion failure: !ti->stackTypeMap.matches(ti_other->stackTypeMap)" with array comprehension → TM: "Assertion failure: !ti->stackTypeMap.matches(ti_other->stackTypeMap)" with array comprehension
You need to log in before you can comment on or make changes to this bug.