Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

[Toon Moene <toon@moene.org>]

On 01/17/2016 01:44 PM, Thomas Koenig wrote:

> So... comments?  Toon, would this help you?  Could yo maybe give this
> a spin?

Thanks, the nightly test at my home computer will build with your patch.

> 2016-01-17  Thomas Koenig  <tkoenig@gcc.gnu.org>
>
>          PR fortran/66094
>          * frontend-passes.c (enum matrix_case):  Add case A2B2T for
>          MATMUL(A,TRANSPoSE(B)) where A and B are rank 2.
>          (inline_limit_check):  Also add A2B2T.
>          (matmul_lhs_realloc):  Handle A2B2T.
>          (check_conjg_variable):  Rename to
>          (check_conjg_transpose_variable):  and also count TRANSPOSE.
>          (inline_matmul_assign):  Handle A2B2T.

It will also perform the following tests (minus the 
"inline_matmul_13.f90" one, which wasn't included in the attachements :-)

> 2016-01-17  Thomas Koenig  <tkoenig@gcc.gnu.org>
>
>          PR fortran/66094
>          * gfortran.dg/inline_matmul_13.f90:  New test.
>          * gfortran.dg/matmul_bounds_8.f90:  New test.
>          * gfortran.dg/matmul_bounds_9.f90:  New test.
>          * gfortran.dg/matmul_bounds_10.f90:  New test.

Unfortunately, running the whole of our weather forecasting system with 
gcc-6 will be *a lot of work*, because I have to build all kinds of 
support libraries (for which I now depend on Debian Testing) by hand.

But I hope just testing your examples will at least give you an idea (on 
-march=haswell).

Thanks, and kind regards,

-- 
Toon Moene - e-mail: toon@moene.org - phone: +31 346 214290
Saturnushof 14, 3738 XG  Maartensdijk, The Netherlands
At home: http://moene.org/~toon/; weather: http://moene.org/~hirlam/
Progress of GNU Fortran: http://gcc.gnu.org/wiki/GFortran#news