Closed Bug 459085 Opened 18 years ago Closed 18 years ago

TM: Number.toString traceable native is broken

Categories

(Core :: JavaScript Engine, defect)

Other Branch
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: jorendorff, Assigned: jorendorff)

References

Details

Attachments

(1 file)

The knownNative entry has "TC" but the native expects a jsdouble argument. So m = new Number(3) function foo() { for (i=0; i<20;i++) m.toString(); } foo() Assertion failed: "Should not move data from GPR to XMM": false (nanojit/Nativei386.cpp:1185) Trace/BPT trap Patch later today if nobody beats me to it.
Attached patch fix — — Splinter Review
Assignee: general → jorendorff
Attachment #342305 - Flags: review?
Attachment #342305 - Flags: review? → review?(gal)
Attachment #342305 - Flags: review?(gal) → review+
This patch applies on top of the one in bug 458735.
Depends on: 458735
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-459085.js,v <-- regress-459085.js initial revision: 1.1 http://hg.mozilla.org/mozilla-central/rev/b04c04268a94
Flags: in-testsuite+
Flags: in-litmus-
v 1.9.1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: