Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 Oct 2019 21:39:52 +0000 (UTC)
From:      Brooks Davis <brooks@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r353329 - head/lib/msun/src
Message-ID:  <201910082139.x98Ldqcr088825@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: brooks
Date: Tue Oct  8 21:39:51 2019
New Revision: 353329
URL: https://svnweb.freebsd.org/changeset/base/353329

Log:
  msun: Silence new harmless -Wimplicit-int-float-conversion warnings
  
  Clang from trunk recently added a warning for when implicit int-to-float
  conversions cause a loss of precision. The code in question is designed
  to be able to handle that, so add explicit casts to silence this.
  
  Submitted by:	James Clarke <jrtc27@jrtc27.com>
  Reviewed by:	dim
  Obtained from:	CheriBSD
  MFC after:	1 week
  Sponsored by:	DARPA, AFRL
  Differential Revision:	https://reviews.freebsd.org/D21913

Modified:
  head/lib/msun/src/s_lround.c

Modified: head/lib/msun/src/s_lround.c
==============================================================================
--- head/lib/msun/src/s_lround.c	Tue Oct  8 21:34:06 2019	(r353328)
+++ head/lib/msun/src/s_lround.c	Tue Oct  8 21:39:51 2019	(r353329)
@@ -49,9 +49,9 @@ __FBSDID("$FreeBSD$");
  * that everything is in range.  At compile time, INRANGE(x) should reduce to
  * two floating-point comparisons in the former case, or TRUE otherwise.
  */
-static const type dtype_min = DTYPE_MIN - 0.5;
-static const type dtype_max = DTYPE_MAX + 0.5;
-#define	INRANGE(x)	(dtype_max - DTYPE_MAX != 0.5 || \
+static const type dtype_min = (type)DTYPE_MIN - 0.5;
+static const type dtype_max = (type)DTYPE_MAX + 0.5;
+#define	INRANGE(x)	(dtype_max - (type)DTYPE_MAX != 0.5 || \
 			 ((x) > dtype_min && (x) < dtype_max))
 
 dtype



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201910082139.x98Ldqcr088825>