TM: Assertion failed: "Should not move data from GPR to XMM"

RESOLVED FIXED

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: dvander)

Tracking

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

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
./js -j
js> for (var j = 0; j < 5; ++j) { if (''[-1]) { } }

Assertion failed: "Should not move data from GPR to XMM": false (nanojit/Nativei386.cpp:1139)
Created attachment 336098 [details] [diff] [review]
aborts recording on invalid string indexes

For invalid string indexes, the interpreter pushes a non-string on the stack whereas the recorder expects a string.  Patch attempt at fixing this attached.
Assignee: general → danderson
Status: NEW → ASSIGNED
Attachment #336098 - Flags: review?(brendan)

Updated

10 years ago
Attachment #336098 - Flags: review?(brendan) → review+
Pushed fix as changeset 268cce1707b6.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Summary: Assertion failed: "Should not move data from GPR to XMM" → TM: Assertion failed: "Should not move data from GPR to XMM"

Comment 3

10 years ago
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-452713.js,v  <--  regress-452713.js
initial revision: 1.1

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