[Bug math/17261] New: [powerpc-nofpu] FE_NOMASK_ENV and FE_NONIEEE_ENV incorrect

[Bug math/17261] New: [powerpc-nofpu] FE_NOMASK_ENV and FE_NONIEEE_ENV incorrect

[jsm28 at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17261

            Bug ID: 17261
           Summary: [powerpc-nofpu] FE_NOMASK_ENV and FE_NONIEEE_ENV
                    incorrect
           Product: glibc
           Version: 2.19
            Status: NEW
          Severity: normal
          Priority: P2
         Component: math
          Assignee: unassigned at sourceware dot org
          Reporter: jsm28 at gcc dot gnu.org
              Host: powerpc*-*-*

sysdeps/powerpc/nofpu/fenv_const.c has values of __fe_enabled_env and
__fe_nonieee_env identical to those for hard-float, although the way fenv_t
values are interpreted for soft-float is completely different.  This means
those environments have exceptions already raised, causing
math/test-fenv-return to fail.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug math/17261] [powerpc-nofpu] FE_NOMASK_ENV and FE_NONIEEE_ENV incorrect

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17261

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  898c62f488e9c403bbb1a96d06c7a257b047fcbc (commit)
      from  9a8a57209aeab41109bec84f2eeee75a7c1fdc60 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=898c62f488e9c403bbb1a96d06c7a257b047fcbc

commit 898c62f488e9c403bbb1a96d06c7a257b047fcbc
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Tue Aug 12 20:31:54 2014 +0000

    Fix powerpc-nofpu __fe_enabled_env and __fe_nonieee_env (bug 17261).

    On powerpc, floating-point environment macros are defined as pointers
    to constants in the library that contain the bit-patterns of the
    desired environment, instead of being magic constants cast to pointer
    type.

    For soft-float, the bit-patterns used for fenv_t are not laid out the
    same as for hard-float.  (e500 has a third layout used; that's not an
    ABI issue because these values are only meaningful within a single
    process, all of whose glibc libraries must come from the same build of
    glibc.)  While the __fe_dfl_env value for soft-float was appropriate
    for the soft-float fenv_t representation, the other two constants had
    the same bit-patterns as for hard-float.  Those bit patterns had the
    effect of having exceptions already raised, causing
    math/test-fenv-return to fail; this patch fixes the patterns used.
    (__fe_nonieee_env also had exceptions unmasked, though they should be
    masked to match hard-float semantics.  Since there is no separate
    non-IEEE mode for soft-float, it's most appropriate for
    __fe_nonieee_env to be the same as __fe_dfl_env; this patch makes it
    an alias.)

    Tested for powerpc-nofpu.

        [BZ #17261]
        * sysdeps/powerpc/nofpu/fenv_const.c (__fe_enabled_env): Change
        value to 0.
        (__fe_nonieee_env): Define as an alias for __fe_dfl_env.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                          |    7 +++++++
 NEWS                               |   28 ++++++++++++++--------------
 sysdeps/powerpc/nofpu/fenv_const.c |    8 ++++----
 3 files changed, 25 insertions(+), 18 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug math/17261] [powerpc-nofpu] FE_NOMASK_ENV and FE_NONIEEE_ENV incorrect

[jsm28 at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17261

Joseph Myers <jsm28 at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Joseph Myers <jsm28 at gcc dot gnu.org> ---
Fixed for 2.20.

-- 
You are receiving this mail because:
You are on the CC list for the bug.