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/4997] lround(nexafter(0.5,-1)) has incorrect value


------- Additional Comments From rsa at us dot ibm dot com  2008-04-11 15:55 -------
Ulrich,

I just retried both versions of the patch (the one in the BZ and the one on the
mailing list) with both --with-cpu=power4 and --with-cpu=970 and both completed
building successfully and passed make check.

This patch didn't add or remove any files.  There should already be dummy
versions of s_llroundf.S in the necessary directories.

> find . -name s_llroundf.S
./powerpc/powerpc32/power4/fpu/s_llroundf.S
./powerpc/powerpc32/power5+/fpu/s_llroundf.S
./powerpc/powerpc32/power6/fpu/s_llroundf.S
./powerpc/powerpc64/fpu/s_llroundf.S

Did the patch apply properly?  I ask this because as an experiment I tried
removing sysdeps/powerpc/powerpc32/power4/fpu/s_llroundf.S and I _then_ got a
build warning about multiple definitions.

/home/ryanarn/build/glibc32_ppc970/math/libm_pic.a(s_llroundf.os): In function
`__llroundf':
/home/ryanarn/glibc/math/../sysdeps/powerpc/fpu/s_llroundf.c:30: multiple
definition of `__llroundf'
/home/ryanarn/build/glibc32_ppc970/math/libm_pic.a(s_llround.os):(.text+0x0):
first defined here



-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4997

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


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