[gnu-internal] Re: [vsipl++] [patch] Quickstart updates for ref-impl

[Mark Mitchell]

Stefan Seefeld wrote:

>> +     <section>
>> +      <title>C-VSIPL</title>
>> +      <para>
>> +       A C-VSIPL implementation can be used by Sourcery VSIPL++ to
>> +       to implement many functions, including linear algebra, solvers,
>> +       and signal processing objects (such as FFT).
>> +      </para>

Is "C-VSIPL" the accepted term?  I think the initial term is just
"VSIPL".  Maybe we should say "VSIPL Back End" in the <title>, which
will make clear that this is about a back end, but leave out the "C-",
here and elsewhere?

>> +    <para>
>> +     When configuring Sourcery VSIPL++ to be used as the reference
>> +     implementation under the BSD license, the following configuration
>> +     flags should be used:

"If you wish to use the BSD-licensed reference-implementation subset of
Sourcery VSIPL++, you must configure with the following options:"

>> +      <listitem>
>> +       <para><option>--enable-ref-impl</option></para>
>> +       <para>
>> +        Configures Sourcery VSIPL++ to be used as the reference
>> +	implementation.  This is necessary, otherwise Sourcery VSIPL++
>> +	requires non-BSD files to operate.

Saying "Configures" again here is redundant; these are configuration
options.  So, maybe:

"Build only the reference-implementation subset of Sourcery VSIPL++.  If
you do not use this option, the complete, optimized implementation of
Sourcery VSIPL++ will be built."

>> +        Configures Sourcery VSIPL++ to use a C-VSIP library for linear
>> +        algebra and signal processing.  This is necessary, otherwise
>> +        linear algebra and signal processing functionality will not be
>> +        available.

"Use a VSIPL library as the back end for VSIPL++.  You must provide this
option when configuring with <option>--enable-ref-impl</option> because
the reference-implementation subset of Sourcery VISPL++ does not support
alternative back ends."

> (Technically, nothing would prevent us from letting --enable-ref-impl
> imply the two options --enable-cvsip and --enable-fft=cvsip.)

That would probably be even better, if we could do it, but I don't want
to introduce feature-creep.

Thanks,

-- 
Mark Mitchell
CodeSourcery
mark at codesourcery.com
(650) 331-3385 x713