Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 28 Apr 2015 14:14:07 +0000 (UTC)
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r282146 - head/lib/msun/tests
Message-ID:  <201504281414.t3SEE7mx004975@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: imp
Date: Tue Apr 28 14:14:06 2015
New Revision: 282146
URL: https://svnweb.freebsd.org/changeset/base/282146

Log:
  The presence/absence of CPU features should be tested with
  MACHINE_CPUARCH or MACHINE_ARCH, not MACHINE. The latter is for kernel
  only things. Also, I think this should be unconditional since all our
  architectures have long double support, but I don't have time to test
  that thoroughly so just add a comment to that effect.

Modified:
  head/lib/msun/tests/Makefile

Modified: head/lib/msun/tests/Makefile
==============================================================================
--- head/lib/msun/tests/Makefile	Tue Apr 28 13:13:23 2015	(r282145)
+++ head/lib/msun/tests/Makefile	Tue Apr 28 14:14:06 2015	(r282146)
@@ -9,7 +9,9 @@ TESTSDIR=	${TESTSBASE}/lib/msun
 # All architectures on FreeBSD have fenv.h
 CFLAGS+=	-DHAVE_FENV_H
 
-.if ${MACHINE} == "amd64" || ${MACHINE} == "i386"
+# Not sure why this isn't defined for all architectures, since most
+# have long double.
+.if ${MACHINE_CPUARCH} == "amd64" || ${MACHINE_CPUARCH} == "i386"
 CFLAGS+=	-D__HAVE_LONG_DOUBLE
 .endif
 



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