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/17262] New: mathinline.h refines llrint on gcc-3.2 for x86_64


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

            Bug ID: 17262
           Summary: mathinline.h refines llrint on gcc-3.2 for x86_64
           Product: glibc
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: math
          Assignee: siddhesh at redhat dot com
          Reporter: siddhesh at redhat dot com

Since:

commit 409e00bd69b8d8dd74d7327085351d26769ea6fc
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Wed Jan 29 07:51:41 2014 -0800

    Disable x87 inline functions for SSE2 math

    When i386 and x86-64 mathinline.h was merged into a single mathinline.h,
    "gcc -m32" enables x87 inline functions on x86-64 even when -mfpmath=sse
    and SSE2 is enabled.  It is a regression on x86-64.  We should check
    __SSE2_MATH__ instead of __x86_64__ when disabling x87 inline functions.


gcc-3.2 is unable to correctly compile x86_64 routines for llrint since it gets
redefined.  This is because gcc 3.2 does not set __SSE2_MATH__ by default for
x86_64, thus exposing the duplicate definition.

The correct fix ought to be to check for both __SSE2_MATH__ and __x86_64__ and
enable those bits only when neither are defined.

-- 
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]