public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org>
To: newlib-cvs@sourceware.org
Subject: [newlib-cygwin] Change the return type of converttoint and document the semantics
Date: Fri, 06 Jul 2018 08:44:00 -0000	[thread overview]
Message-ID: <20180706084454.69741.qmail@sourceware.org> (raw)

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

commit 6a85e1a4e5a01dec3b423eb172b3ad9155487e75
Author: Szabolcs Nagy <szabolcs.nagy@arm.com>
Date:   Wed Jul 4 11:09:39 2018 +0100

    Change the return type of converttoint and document the semantics
    
    The roundtoint and converttoint internal functions are only called with small
    values, so 32 bit result is enough for converttoint and it is a signed int
    conversion so the natural return type is int32_t.
    
    The original idea was to help the compiler keeping the result in uint64_t,
    then it's clear that no sign extension is needed and there is no accidental
    undefined or implementation defined signed int arithmetics.
    
    But it turns out gcc does a good job with inlining so changing the type has
    no overhead and the semantics of the conversion is less surprising this way.
    Since we want to allow the asuint64 (x + 0x1.8p52) style conversion, the top
    bits were never usable and the existing code ensures that only the bottom
    32 bits of the conversion result are used.
    
    In newlib with default settings only aarch64 is affected and there is no
    significant code generation change with gcc after the patch.

Diff:
---
 newlib/libm/common/math_config.h | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/newlib/libm/common/math_config.h b/newlib/libm/common/math_config.h
index b46e44e..d5896bb 100644
--- a/newlib/libm/common/math_config.h
+++ b/newlib/libm/common/math_config.h
@@ -71,15 +71,23 @@
 #endif
 
 #if HAVE_FAST_ROUND
+/* When set, the roundtoint and converttoint functions are provided with
+   the semantics documented below.  */
 # define TOINT_INTRINSICS 1
 
+/* Round x to nearest int in all rounding modes, ties have to be rounded
+   consistently with converttoint so the results match.  If the result
+   would be outside of [-2^31, 2^31-1] then the semantics is unspecified.  */
 static inline double_t
 roundtoint (double_t x)
 {
   return round (x);
 }
 
-static inline uint64_t
+/* Convert x to nearest int in all rounding modes, ties have to be rounded
+   consistently with roundtoint.  If the result is not representible in an
+   int32_t then the semantics is unspecified.  */
+static inline int32_t
 converttoint (double_t x)
 {
 # if HAVE_FAST_LROUND


                 reply	other threads:[~2018-07-06  8:44 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180706084454.69741.qmail@sourceware.org \
    --to=corinna@sourceware.org \
    --cc=newlib-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).