Author: arichardson Date: Tue Nov 28 20:37:27 2017 New Revision: 326342 URL: https://svnweb.freebsd.org/changeset/base/326342
Log: Fix fabs() for MIPS when used on -0.0 It would previously return negative zero for -0.0 since -0.0 does not compare less than 0. The issue was discovered when running the libc++ test suite on softfloat MIPS64. I have verified that both clang and GCC generate sensible code for the builtin. For soft float they clear the sign bit using integer operations and in hard float mode they use abs.d. Reviewed by: #mips, jhb, brooks, imp, emaste Approved by: jhb (mentor) Differential Revision: https://reviews.freebsd.org/D13135 Modified: head/lib/libc/mips/gen/fabs.c Modified: head/lib/libc/mips/gen/fabs.c ============================================================================== --- head/lib/libc/mips/gen/fabs.c Tue Nov 28 19:57:16 2017 (r326341) +++ head/lib/libc/mips/gen/fabs.c Tue Nov 28 20:37:27 2017 (r326342) @@ -42,7 +42,6 @@ __FBSDID("$FreeBSD$"); double fabs(double x) { - if (x < 0) - x = -x; - return(x); + + return (__builtin_fabs(x)); } _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
