If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Decimal support should be disabled or removed

VERIFIED FIXED

Status

Tamarin
Virtual Machine
VERIFIED FIXED
10 years ago
8 years ago

People

(Reporter: Dan Schaffer, Assigned: Erik Tierney)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
the Decimal checkin shows performance degradation (10-70%) in sunspider tests.  We should disable Decimal to get back to the pre-Decimal performance level in tamarin-central.
(Assignee)

Comment 1

10 years ago
Created attachment 314092 [details] [diff] [review]
Patch that removes decimal support from tamarin

This patch pretty much reverses the original decimal checkin.  I'm also going to remove the decimal test files, but didn't put those in the diff as it makes it really large and unreadable.
Assignee: nobody → tierney
Status: NEW → ASSIGNED
Attachment #314092 - Flags: review?(stejohns)

Updated

10 years ago
Attachment #314092 - Flags: review?(stejohns) → review+
(Assignee)

Comment 2

10 years ago
Pushed patch in changeset:
http://hg.mozilla.org/tamarin-central/index.cgi/rev/0446de105c63

Performance should be back to pre-Decimal levels.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED

Updated

8 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.