JM: "Assertion failure: otherReg != testReg,"

RESOLVED FIXED

Status

()

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

People

(Reporter: gkw, Assigned: sstangl)

Tracking

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

Trunk
x86
Mac OS X
assertion, regression, testcase
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
for (let x in []) {
  t(x !== x)
}

asserts js debug shell on JM changeset 33b05dd43cd4 at Assertion failure: otherReg != testReg, at ../methodjit/FastOps.cpp:1669
(Assignee)

Comment 1

7 years ago
Both 'other' and 'test' FrameEntries have the same backing FE, so tempRegForData(other) returns testReg. This is fixed by filling in the missing code in the haveSameBacking() case.
Assignee: general → sstangl
(Assignee)

Comment 2

7 years ago
Created attachment 469933 [details] [diff] [review]
Implement haveSameBacking() case for stricteq.
Attachment #469933 - Flags: review?(dvander)
Attachment #469933 - Flags: review?(dvander) → review+
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/projects/jaegermonkey/rev/088d22176324
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Automatically extracted testcase for this bug was committed:

https://hg.mozilla.org/mozilla-central/rev/efaf8960a929
Flags: in-testsuite+
You need to log in before you can comment on or make changes to this bug.