On Sat, Feb 25, 2017 at 02:29:12PM +0100, Thomas Koenig wrote:
>
> there still was one piece missing for the new matmul library
> version. To make sure that users (usually) benefit, we need
> to call the library by default up from a certain limit.
> The attached patch does that, with a limit of 30, which seems
> to be reasonable given a few benchmarks.
>
> Some test cases had to be changed to scan the optimized tree
> instead of the original because the version still had some
> if (0) statement in them.
>
> Regeression-tested. OK for trunk?
Looks ok to me with one comment below.
> Index: fortran/invoke.texi
> ===================================================================
> --- fortran/invoke.texi (Revision 245564)
> +++ fortran/invoke.texi (Arbeitskopie)
> @@ -1630,7 +1630,7 @@ square, the size comparison is performed using the
> the dimensions of the argument and result matrices.
>
> The default value for @var{n} is the value specified for
> -@code{-fblas-matmul-limit} if this option is specified, or unlimitited
> +@code{-fblas-matmul-limit} if this option is specified, or 30
> otherwise.
This description looks a little muddled. I think something like
The default value for N is 30. The @code{-fblas-matmul-limit}
can be used to change this value.
--
Steve
