public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
* [newlib-cygwin] Don't call double rint from float powf.
@ 2017-12-13 10:04 Corinna Vinschen
  0 siblings, 0 replies; only message in thread
From: Corinna Vinschen @ 2017-12-13 10:04 UTC (permalink / raw)
  To: newlib-cvs

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=c338bc22554fcddf64f12148785d07c1a4021160

commit c338bc22554fcddf64f12148785d07c1a4021160
Author: Jim Wilson <jimw@sifive.com>
Date:   Tue Dec 12 11:38:01 2017 -0800

    Don't call double rint from float powf.
    
    Updated patch to use 0.0f in addition to calling rintf.
    
    Tested same way as before, with a testcase that triggers the code and
    make check.
    
    OK?
    
    	newlib/
    	* libm/math/wf_pow.c (powf): Call rintf instead of rint.  Use 0.0f
    	for compare.

Diff:
---
 newlib/libm/math/wf_pow.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/newlib/libm/math/wf_pow.c b/newlib/libm/math/wf_pow.c
index be45355..9a10254 100644
--- a/newlib/libm/math/wf_pow.c
+++ b/newlib/libm/math/wf_pow.c
@@ -127,11 +127,11 @@
 		    if (_LIB_VERSION == _SVID_) {
 		       exc.retval = HUGE;
 		       y *= 0.5;
-		       if(x<0.0&&rint(y)!=y) exc.retval = -HUGE;
+		       if(x<0.0f&&rintf(y)!=y) exc.retval = -HUGE;
 		    } else {
 		       exc.retval = HUGE_VAL;
                        y *= 0.5;
-		       if(x<0.0&&rint(y)!=y) exc.retval = -HUGE_VAL;
+		       if(x<0.0f&&rintf(y)!=y) exc.retval = -HUGE_VAL;
 		    }
 		    if (_LIB_VERSION == _POSIX_)
 		        errno = ERANGE;


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-12-13 10:04 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-12-13 10:04 [newlib-cygwin] Don't call double rint from float powf Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).