This is the mail archive of the glibc-bugs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug math/17748] fe* function calls yield link namespace conform failures on dbl-64 hosts


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

--- Comment #6 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  8116321f657549e80d0167b8a790a7aa54d6379d (commit)
      from  882dfe184a81a8924bc8acb79e96fbf7556fa60b (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=8116321f657549e80d0167b8a790a7aa54d6379d

commit 8116321f657549e80d0167b8a790a7aa54d6379d
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Wed Jan 7 19:01:20 2015 +0000

    Fix libm feupdateenv namespace (bug 17748).

    Concluding the fixes for C90 libm functions calling C99 fe* functions,
    this patch fixes the case of feupdateenv by making it a weak alias for
    __feupdateenv and making the affected code call __feupdateenv.

    Tested for x86_64 (testsuite, and that installed stripped shared
    libraries are unchanged by the patch).  Also tested for ARM
    (soft-float) that the math.h linknamespace tests now pass.

        [BZ #17748]
        * include/fenv.h (__feupdateenv): Use libm_hidden_proto.
        * math/feupdateenv.c (__feupdateenv): Use libm_hidden_def.
        * sysdeps/aarch64/fpu/feupdateenv.c (feupdateenv): Rename to
        __feupdateenv and define as weak alias of __feupdateenv.  Use
        libm_hidden_weak.
        * sysdeps/alpha/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/arm/feupdateenv.c (feupdateenv): Rename to __feupdateenv
        and define as weak alias of __feupdateenv.  Use libm_hidden_weak.
        * sysdeps/hppa/fpu/feupdateenv.c (feupdateenv): Likewise.
        * sysdeps/i386/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/ia64/fpu/feupdateenv.c (feupdateenv): Rename to
        __feupdateenv and define as weak alias of __feupdateenv.  Use
        libm_hidden_weak.
        * sysdeps/m68k/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/mips/fpu/feupdateenv.c (feupdateenv): Rename to
        __feupdateenv and define as weak alias of __feupdateenv.  Use
        libm_hidden_weak.
        * sysdeps/powerpc/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/powerpc/nofpu/feupdateenv.c (__feupdateenv): Likewise.
        * sysdeps/powerpc/powerpc32/e500/nofpu/feupdateenv.c
        (__feupdateenv): Likewise.
        * sysdeps/s390/fpu/feupdateenv.c (feupdateenv): Rename to
        __feupdateenv and define as weak alias of __feupdateenv.  Use
        libm_hidden_weak.
        * sysdeps/sh/sh4/fpu/feupdateenv.c (feupdateenv): Likewise.
        * sysdeps/sparc/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/tile/math_private.h (__feupdateenv): New inline
        function.
        * sysdeps/x86_64/fpu/feupdateenv.c (__feupdateenv): Use
        libm_hidden_def.
        * sysdeps/generic/math_private.h (default_libc_feupdateenv): Call
        __feupdateenv instead of feupdateenv.
        (default_libc_feupdateenv_test): Likewise.
        (libc_feresetround_ctx): Likewise.

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

Summary of changes:
 ChangeLog                                          |   43 ++++++++++++++++++++
 NEWS                                               |    4 +-
 include/fenv.h                                     |    1 +
 math/feupdateenv.c                                 |    1 +
 sysdeps/aarch64/fpu/feupdateenv.c                  |    6 ++-
 sysdeps/alpha/fpu/feupdateenv.c                    |    1 +
 sysdeps/arm/feupdateenv.c                          |    6 ++-
 sysdeps/generic/math_private.h                     |    6 +-
 sysdeps/hppa/fpu/feupdateenv.c                     |    6 ++-
 sysdeps/i386/fpu/feupdateenv.c                     |    1 +
 sysdeps/ia64/fpu/feupdateenv.c                     |    6 ++-
 sysdeps/m68k/fpu/feupdateenv.c                     |    1 +
 sysdeps/mips/fpu/feupdateenv.c                     |    6 ++-
 sysdeps/powerpc/fpu/feupdateenv.c                  |    1 +
 sysdeps/powerpc/nofpu/feupdateenv.c                |    1 +
 sysdeps/powerpc/powerpc32/e500/nofpu/feupdateenv.c |    1 +
 sysdeps/s390/fpu/feupdateenv.c                     |    6 ++-
 sysdeps/sh/sh4/fpu/feupdateenv.c                   |    6 ++-
 sysdeps/sparc/fpu/feupdateenv.c                    |    1 +
 sysdeps/tile/math_private.h                        |    1 +
 sysdeps/x86_64/fpu/feupdateenv.c                   |    1 +
 21 files changed, 87 insertions(+), 19 deletions(-)

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]