[vsipl++] [patch] Quickstart updates for ref-impl

[Jules Bergmann]

Stefan, Thanks for the feedback. -- Jules


> This paragraph appears to contain an extra 'to'.

Fixed.

> 
>>      <section>
>> @@ -1031,6 +1061,50 @@
>>       </varlistentry>
>>  
>>       <varlistentry>
>> +      <term><option>--enable-cvsip</option></term>
>> +      <listitem>
>> +       <para>
>> +        Enable Sourcery VSIPL++ to search for an appropriate
>> +	C-VSIP implementation on the platform.  If found, it
>> +	will be used to perform linear algebra (matrix-vector
>> +	products and solvers) and signal processing (FFT, convolution,
>> +	correlation, and FIR).
>> +       </para>
> 
> I believe this is not entirely correct. In order for it to be used for FFT,
> an additional mention in the FFT backend list ('--enable-fft=cvsip') is needed.

Thanks.  How about

"Enable Sourcery VSIPL++ to search for an appropriate C-VSIP 
implementation on the platform.  It found, it will be used to perform 
linear algebra (matrix-vector products and solvers) and some signal 
processing (convolution, correlation, and FIR).  If the 
<option>--enable-fft=cvsip</option> option is also given, C-VSIP will be 
used to perform FFTs."

I'll also update the '--enable-fft=xxx' documentation.

> 
> As noted above, --enable-fft=cvsip needs to be provided, too.
> (Technically, nothing would prevent us from letting --enable-ref-impl
> imply the two options --enable-cvsip and --enable-fft=cvsip.)

Is --enable-fft=cvsip necessary with --enable-ref-impl?  I thought that 
--enable-ref-impl skipped dispatch and went directly to the C-VSIP FFT.

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705