TM: knownNative for num_toString

RESOLVED FIXED in mozilla1.9.1

Status

()

Core
JavaScript Engine
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: brendan, Assigned: brendan)

Tracking

Trunk
mozilla1.9.1
Points:
---
Bug Flags:
in-testsuite -
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

10 years ago
Created attachment 335651 [details] [diff] [review]
fix

Patch here. Easy for zero-args (radix 10) case thanks to pre-existing FASTCALL js_NumberToString. Helps fannkuch already in my testing.

/be
Attachment #335651 - Flags: review?(danderson)
(Assignee)

Comment 1

10 years ago
Comment on attachment 335651 [details] [diff] [review]
fix

Argh, I didn't qdiff. I wanted to get diff -U 8 effects, so used diff -r qparent. Hg still annoys. Right patch in a sec.

/be
Attachment #335651 - Attachment is obsolete: true
Attachment #335651 - Flags: review?(danderson)
(Assignee)

Comment 2

10 years ago
Created attachment 335652 [details] [diff] [review]
fix

This does not include the somewhat bigger patch for bug 452140. :-)

/be
Attachment #335652 - Flags: review?(danderson)
Comment on attachment 335652 [details] [diff] [review]
fix

Pushed as changeset 8dc9089541b0.
Attachment #335652 - Flags: review?(danderson) → review+
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

10 years ago
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.