svn commit: r240827 - head/lib/msun/src

[Steve Kargl]

Author: kargl
Date: Sat Sep 22 15:19:11 2012
New Revision: 240827
URL: http://svn.freebsd.org/changeset/base/240827

Log:
  * Make STRICT_ALIGN() work for doubles as well as for floats.  This
    only affects i386.  The double case was intentionally left broken
    as an optimization, but we are getting closer to supporting
    applications and/or kernels that change the (FreeBSD i386) default
    rounding precision from FP_PD to FP_PE and never change it back,
    and this requires the STRICT_ALIGN()s that were added to support
    FP_PE to actually work in all precisions.
  
  * Remove an extraneous semicolon at the end of a macro that was
    supposed to be function-like.
  
  Submitted by:	bde
  Approved by:	das (mentor)

Modified:
  head/lib/msun/src/math_private.h

Modified: head/lib/msun/src/math_private.h
==============================================================================
--- head/lib/msun/src/math_private.h	Sat Sep 22 14:39:20 2012	(r240826)
+++ head/lib/msun/src/math_private.h	Sat Sep 22 15:19:11 2012	(r240827)
@@ -224,7 +224,7 @@ do {								\
 #define	STRICT_ASSIGN(type, lval, rval) do {	\
 	volatile type __lval;			\
 						\
-	if (sizeof(type) >= sizeof(double))	\
+	if (sizeof(type) >= sizeof(long double))	\
 		(lval) = (rval);		\
 	else {					\
 		__lval = (rval);		\
@@ -241,7 +241,7 @@ do {								\
 	fp_prec_t __oprec;			\
 						\
 	if ((__oprec = fpgetprec()) != FP_PE)	\
-		fpsetprec(FP_PE);
+		fpsetprec(FP_PE)
 #define	RETURNI(x) do {				\
 	__retval = (x);				\
 	if (__oprec != FP_PE)			\