59963 - fdlibm code does "illegal type punning"

Status: VERIFIED DUPLICATE of bug 24892

(Core :: JavaScript Engine, defect, P3)

Description

[Braden]

Refer to the Red Hat Bugzilla report (URL) for details.

Comment 1

[tenthumbs]

Isn't this bug 24892 again?

Comment 2

[Braden]

It looks related, but I'm not convinced it's exactly the same thing.

Comment 3

[Braden]

Bug 24892 is marked fixed, but the problem I describe here is exibited at least
as of JS 1.5rc2. Note the problem file in this case is e_hypot.c.

Comment 4

[tenthumbs]

mozilla/js/src/fdlibm/fdlibm.h has modified _HI and _LO macros which use unions.
Doesn't that fix it?

Comment 5

[Phil Schwartau]

Braden, the fix for bug 24892 went in during September, after the JS 1.5 RC2 
tarball was published. Do you have access to the Mozilla CVS repository 
to pull the most recent source code? Details on this are at

                    http://www.mozilla.org/js/js15.html
                    http://www.mozilla.org/cvs.html


If not, could you attach a reduced JavaScript testcase to this bug,
and I'll try it myself - thanks. 


cc'ing rginda -

Comment 6

[Braden]

Phil: I'll try the latest code tonight. Note that the problem manifests itself
as an ICE under gcc 2.96 when compiling the code; I don't know whether or not
the behavior of JavaScript code is affected under a successfully-compiled
SpiderMonkey.

Comment 7

[Braden]

The problem does seem to be fixed in the current code. Marking duplicate.

*** This bug has been marked as a duplicate of 24892 ***

Comment 8

[Phil Schwartau]

Thanks, Braden! Marking Verified -