Patchwork [doc] Fix example of multiple baseboards, PR 54711

login
register
mail settings
Submitter Steve Ellcey
Date Sept. 26, 2012, 8:28 p.m.
Message ID <c01a4012-04f7-4c79-8c4a-e6bc949fd879@EXCHHUB01.MIPS.com>
Download mbox | patch
Permalink /patch/187196/
State New
Headers show

Comments

Steve Ellcey - Sept. 26, 2012, 8:28 p.m.
While doing some testing and trying to run multiple versions of the GCC
testsuite (big-endian, little-endian, etc) I looked at the example at
http://gcc.gnu.org/install/test.html and thought I could run multiple
versions by using multiple --target_board flags in RUNTESTFLAGS.  It turns
out this does not work and what you want to do is use one target_board
flag with a space seperated list of boardnames/options.

This patch changes the example to make that more clear.

OK to checkin?

Steve Ellcey
sellcey@mips.com


2012-09-26  Steve Ellcey  <sellcey@mips.com>

	PR web/54711
	* doc/install.texi: Fix example.
Mike Stump - Sept. 26, 2012, 10:34 p.m.
On Sep 26, 2012, at 1:28 PM, Steve Ellcey <sellcey@mips.com> wrote:
> While doing some testing and trying to run multiple versions of the GCC
> testsuite (big-endian, little-endian, etc) I looked at the example at
> http://gcc.gnu.org/install/test.html and thought I could run multiple
> versions by using multiple --target_board flags in RUNTESTFLAGS.  It turns
> out this does not work and what you want to do is use one target_board
> flag with a space seperated list of boardnames/options.
> 
> This patch changes the example to make that more clear.
> 
> OK to checkin?

Ok.  This sort of patch makes be nervous, I wonder if someone broke a feature along the way…

Make sure you review in a web browser to ensure it looks the way you want.

Patch

diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
index d4384e8..bc692d4 100644
--- a/gcc/doc/install.texi
+++ b/gcc/doc/install.texi
@@ -2567,14 +2567,14 @@  The following will run each testsuite eight times using the @samp{arm-sim}
 target, as if you had specified all possible combinations yourself:
 
 @smallexample
---target_board=arm-sim/-mhard-float/-O1
---target_board=arm-sim/-mhard-float/-O2
---target_board=arm-sim/-mhard-float/-O3
---target_board=arm-sim/-mhard-float
---target_board=arm-sim/-msoft-float/-O1
---target_board=arm-sim/-msoft-float/-O2
---target_board=arm-sim/-msoft-float/-O3
---target_board=arm-sim/-msoft-float
+--target_board='arm-sim/-mhard-float/-O1 \
+                arm-sim/-mhard-float/-O2 \
+                arm-sim/-mhard-float/-O3 \
+                arm-sim/-mhard-float \
+                arm-sim/-msoft-float/-O1 \
+                arm-sim/-msoft-float/-O2 \
+                arm-sim/-msoft-float/-O3 \
+                arm-sim/-msoft-float'
 @end smallexample
 
 They can be combined as many times as you wish, in arbitrary ways.  This