From: Julian Seward <jseward@acm.org>
Date: Wed, 4 Jan 2006 03:31:07 +0000 (+0000)
Subject: Minor cleanups:
X-Git-Tag: svn/VALGRIND_3_2_0~404
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9f0a87221c18539cc4cb2cf9883a584ad6a0809a;p=thirdparty%2Fvalgrind.git

Minor cleanups:
- get rid of some unused _ALL defns
- clarify logic in setting up VG_<platform>
- clarify some comments


git-svn-id: svn://svn.valgrind.org/valgrind/trunk@5496
---

diff --git a/configure.in b/configure.in
index a157ac6e1e..6316ed156d 100644
--- a/configure.in
+++ b/configure.in
@@ -113,8 +113,7 @@ AC_CANONICAL_HOST
 
 AC_MSG_CHECKING([for a supported CPU])
 AC_SUBST(VG_ARCH)
-AC_SUBST(VG_ARCH_ALL)
-VG_ARCH_ALL="amd64 ppc32 ppc64 x86"
+
 AC_SUBST(VALT_LOAD_ADDRESS)
 
 case "${host_cpu}" in
@@ -170,8 +169,6 @@ fi
 
 AC_MSG_CHECKING([for a supported OS])
 AC_SUBST(VG_OS)
-AC_SUBST(VG_OS_ALL)
-VG_OS_ALL="linux"
 
 case "${host_os}" in
      *linux*)
@@ -213,20 +210,14 @@ case "${host_os}" in
 	;;
 esac
 
-# What's this used for?
-AC_SUBST(VG_PLATFORM_ALL)
-VG_PLATFORM_ALL="amd64-linux ppc32-linux ppc64-linux x86-linux"
 
 # Establish VG_PLATFORM_PRI.  This is the primary build target.  The
 # entire system, including regression and performance tests, will be
-# built for this target.  On 32-bit systems this is the only build
-# target.
+# built for this target.
 #
-# Now establish VG_PLATFORM_SEC, the secondary build target, if
+# Also establish VG_PLATFORM_SEC, the secondary build target, if
 # possible.  The system will also be built for this target, but not
-# the regression or performance tests.  Also decide what -m32/-m64
-# flags are needed for the primary and secondary build targets.
-# XXXXXXXXXXXXX is this needed?
+# the regression or performance tests.
 #
 AC_MSG_CHECKING([for a supported CPU/OS combination])
 
@@ -265,11 +256,13 @@ esac
 # Set up VG_<platform>.  Either one or two of these become defined.
 #
 AM_CONDITIONAL(VG_X86_LINUX,   
-               test x$VG_PLATFORM_PRI = xX86_LINUX -o x$VG_PLATFORM_PRI = xAMD64_LINUX)
+               test x$VG_PLATFORM_PRI = xX86_LINUX \
+                 -o x$VG_PLATFORM_SEC = xX86_LINUX)
 AM_CONDITIONAL(VG_AMD64_LINUX, 
                test x$VG_PLATFORM_PRI = xAMD64_LINUX)
 AM_CONDITIONAL(VG_PPC32_LINUX, 
-               test x$VG_PLATFORM_PRI = xPPC32_LINUX -o x$VG_PLATFORM_PRI = xPPC64_LINUX)
+               test x$VG_PLATFORM_PRI = xPPC32_LINUX \ 
+                 -o x$VG_PLATFORM_SEC = xPPC32_LINUX)
 AM_CONDITIONAL(VG_PPC64_LINUX, 
                test x$VG_PLATFORM_PRI = xPPC64_LINUX)
 
diff --git a/docs/internals/multiple-architectures.txt b/docs/internals/multiple-architectures.txt
index 52a45321d1..0c27a3cbb0 100644
--- a/docs/internals/multiple-architectures.txt
+++ b/docs/internals/multiple-architectures.txt
@@ -44,9 +44,6 @@ The configure script inspects the CPU.  It then sets
    VG_PLATFORM_PRI to be the primary target
    VG_PLATFORM_SEC to be the secondary target, if any
 
-(VG_PLATFORM_SEC is only used for printing an informational message at
-configure time).
-
 It also sets one (single-arch build) or two (dual-arch build) of
 the following: