Closed
Bug 412619
Opened 18 years ago
Closed 17 years ago
Math failures in Tamarin Tracing on OSX 10.5 with intel hardware
Categories
(Tamarin Graveyard :: Tracing Virtual Machine, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: cpeyer, Assigned: stejohns)
Details
FAILURES:
ecma3/Math/e15_8_2_6.abc : Infinity/Math.ceil(-Number.MIN_VALUE) = Infinity FAILED! expected: -Infinity
ecma3/Math/e15_8_2_6.abc : Infinity/Math.ceil(-0.9) = Infinity FAILED! expected: -Infinity
These are failing in JIT and interp.
These pass on 10.5 on PPC
Transferred Comments:
Brent Baker - Thu Nov 15 10:42:36 CST 2007
These testcases are also failing in Tamarin.
Brent Baker - Fri Nov 30 09:11:01 CST 2007
Steven can you take a look at this mac failure. IT IS also happening in tamarin at this stage and it appears to be an issue ONLY when using 10.5 on intel hardware.
| Reporter | ||
Updated•18 years ago
|
Summary: Math failures in Quantum on OSX 10.5 with intel hardware → Math failures in Tamarin Tracing on OSX 10.5 with intel hardware
Comment 1•18 years ago
|
||
This a duplication of https://bugzilla.mozilla.org/show_bug.cgi?id=401095 The testcases fails in both TT and TC
| Assignee | ||
Comment 2•17 years ago
|
||
Err, the "wrong" values actually look correct to me! Consider:
Math.ceil(-0.9) == 0.0 (ditto for MIN_VALUE)
Infinity / 0.0 == Infinity, not -Infinity (as "expected")
FYI, executing the same snippet in SpiderMonkey (FF2) on my IntelMac on Leopard also gives me Infinity.
What am I missing here?
| Assignee | ||
Comment 3•17 years ago
|
||
Ah. I'm guessing that Math.ceil is expected to return -0 here (not 0). Hmm...
| Assignee | ||
Comment 4•17 years ago
|
||
It's easy to special-case a fix for this, but it appears to be broken in the ceil() function call in Leopard...
Assignee: nobody → stejohns
Component: Virtual Machine → Tracing Virtual Machine
| Assignee | ||
Comment 5•17 years ago
|
||
Fixed in changeset: 306:c811c024a458
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 6•17 years ago
|
||
Do not expect these testcases to fail anymore (pushed changeset: 307:c4383132e8b0)
You need to log in
before you can comment on or make changes to this bug.
Description
•