[patch] Re: [vsipl++] Compiling VSIPL++ on Cell PPE

[Don McCoy <don@xxxxxxxxxxxxxxxx>]

The attached patch fixes the quickstart guide.



Jules Bergmann wrote:
> Don McCoy wrote:
> > Hahn Kim wrote:
> > > Can you tell me why it's trying to configure ATLAS when I try to 
> > > disable it?  Any information is appreciated.  Thanks!
> > > > ./configure 
> > > --prefix=<dir-containing-vsipl++>/sourceryvsipl++-1.3-mpich2-1.0.6-sal 
> > > --with-mpi-prefix=<dir-containing-mpich2>/mpich2-1.0.6 CXXFLAGS="-O2 
> > > -DMPICH_IGNORE_CXX_SEEK" 
> > > --with-sal-include=/opt/MultiCorePlus/include 
> > > --with-sal-lib=/opt/MultiCorePlus/lib --enable-fft=sal 
> > > --disable-builtin-atlas
> > I find this somewhat confusing myself and usually have to look it up 
> > each time.  I looked in a current tree, but I believe this is true 
> > for 1.3 also:  use --with-lapack=no or --with-lapack=simple-builtin 
> > for a lapack that doesn't need atlas.
> FWIW, this looks like a quickstart documentation bug.  It mentions 
> --disable-builtin-atlas, but does not mention --with-lapack=no.
> configure --help appears to get it right.
>
> Don, can you fix the quickstart?
>
>                 thanks,
>                 -- Jules

--
Don McCoy
don (at) CodeSourcery
(888) 776-0262 / (650) 331-3385, x712

2007-10-09  Don McCoy  <don@xxxxxxxxxxxxxxxx>

	* doc/quickstart/quickstart.xml: Corrected description of 
	  --with-lapack to include 'simple-builtin' and 'no' options.  
	  Removed deprecated --disable-builtin-atlas option.

Index: doc/quickstart/quickstart.xml
===================================================================
--- doc/quickstart/quickstart.xml	(revision 184409)
+++ doc/quickstart/quickstart.xml	(working copy)
@@ -941,8 +941,9 @@
         (matrix-vector products and solvers).  Valid choices for
         <replaceable>lib</replaceable> include <option>mkl</option>,
 	<option>acml</option>, <option>atlas</option>, 
-        <option>generic</option>, <option>builtin</option>, and
-	<option>fortran-builtin</option>.
+        <option>generic</option>, <option>builtin</option>,
+	<option>fortran-builtin</option>, <option>simple-builtin</option>, and
+        <option>no</option>.
        </para>
 
        <para>
@@ -982,17 +983,14 @@
         as well as the g2c library.  Use the <option>--with-g2c-path=</option> 
         option if this library is not installed in a standard location.
        </para>
-      </listitem>
-     </varlistentry>
-
-     <varlistentry>
-      <term><option>--disable-builtin-atlas</option></term>
-      <listitem>
        <para>
-        Disables the consideration of Sourcery VSIPL++'s builtin
-	ATLAS for performing linear algebra.  This option is useful
-	if building on a platform that is not supported by ATLAS.
+        <option>simple-builtin</option> selects a version of LAPACK 
+        that doesn't require ATLAS.
        </para>
+       <para>
+        <option>no</option> is used to disable searching for a LAPACK
+        library.
+       </para>
       </listitem>
      </varlistentry>