[Bug fortran/26025] New: Optionally use BLAS for matmul

[Bug fortran/26025] New: Optionally use BLAS for matmul

[tkoenig at gcc dot gnu dot org]

There should be a flag for replacing our home-grown
matmul and dot_product with calls to BLAS.  If the user has
installed an optimized BLAS version, this could be a huge win.

There was some discussion of this on the mailing list, but no PR
that I could find.

Something like -fblas-intrinsics (and -fblas-f2c, for complex)
would be good.


-- 
           Summary: Optionally use BLAS for matmul
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Keywords: missed-optimization
          Severity: enhancement
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: tkoenig at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-01-30 12:36 -------
Confirmed


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-01-30 12:36:04
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[jb at gcc dot gnu dot org]

------- Comment #2 from jb at gcc dot gnu dot org  2006-02-21 20:55 -------
Here's a message on the mailing list that provides some benchmarks as well as
one possible interface for how one could use it:

http://gcc.gnu.org/ml/fortran/2005-11/msg00474.html

Actually, I'm feeling a bit tired of banging at the IO library for the moment,
so I'll assign this bug to myself.


-- 

jb at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |jb at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-01-30 12:36:04         |2006-02-21 20:55:12
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[fxcoudert at gcc dot gnu dot org]

------- Comment #3 from fxcoudert at gcc dot gnu dot org  2006-09-25 08:58 -------
Patch proposed here: http://gcc.gnu.org/ml/fortran/2006-09/msg00325.html

Janne, as I know you were interested in that feature, could you look at my
patch if you have some time?


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fxcoudert at gcc dot gnu dot
                   |                            |org
                URL|                            |http://gcc.gnu.org/ml/fortra
                   |                            |n/2006-09/msg00325.html


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[steven at gcc dot gnu dot org]

------- Comment #4 from steven at gcc dot gnu dot org  2006-10-07 10:02 -------
New patch.  New link.  One-oh-one.


-- 

steven at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://gcc.gnu.org/ml/fortra|http://gcc.gnu.org/ml/gcc-
                   |n/2006-09/msg00325.html     |patches/2006-
                   |                            |10/msg00343.html


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[fxcoudert at gcc dot gnu dot org]

------- Comment #5 from fxcoudert at gcc dot gnu dot org  2006-10-22 07:42 -------
Subject: Bug 26025

Author: fxcoudert
Date: Sun Oct 22 07:41:48 2006
New Revision: 117948

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117948
Log:
        PR fortran/26025

        * lang.opt: Add -fexternal-blas and -fblas-matmul-limit options.
        * options.c (gfc_init_options): Initialize new flags.
        (gfc_handle_option): Handle new flags.
        * gfortran.h (gfc_option): Add flag_external_blas and
        blas_matmul_limit flags.
        * trans-expr.c (gfc_conv_function_call): Use new argument
        append_args, appending it at the end of the argument list
        built for a function call.
        * trans-stmt.c (gfc_trans_call): Use NULL_TREE for the new
        append_args argument to gfc_trans_call.
        * trans.h (gfc_conv_function_call): Update prototype.
        * trans-decl.c (gfc_build_intrinsic_function_decls): Add
        prototypes for BLAS ?gemm routines.
        * trans-intrinsic.c (gfc_conv_intrinsic_funcall): Generate the
        extra arguments given to the library matmul function, and give
        them to gfc_conv_function_call.
        * invoke.texi: Add documentation for -fexternal-blas and
        -fblas-matmul-limit.

        * m4/matmul.m4: Add possible call to gemm routine.
        * generated/matmul_r8.c: Regenerate.
        * generated/matmul_r16.c: Regenerate.
        * generated/matmul_c8.c: Regenerate.
        * generated/matmul_i8.c: Regenerate.
        * generated/matmul_c16.c: Regenerate.
        * generated/matmul_r10.c: Regenerate.
        * generated/matmul_r4.c: Regenerate.
        * generated/matmul_c10.c: Regenerate.
        * generated/matmul_c4.c: Regenerate.
        * generated/matmul_i4.c: Regenerate.
        * generated/matmul_i16.c: Regenerate.

Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/invoke.texi
    trunk/gcc/fortran/lang.opt
    trunk/gcc/fortran/options.c
    trunk/gcc/fortran/trans-decl.c
    trunk/gcc/fortran/trans-expr.c
    trunk/gcc/fortran/trans-intrinsic.c
    trunk/gcc/fortran/trans-stmt.c
    trunk/gcc/fortran/trans.h
    trunk/libgfortran/ChangeLog
    trunk/libgfortran/generated/matmul_c10.c
    trunk/libgfortran/generated/matmul_c16.c
    trunk/libgfortran/generated/matmul_c4.c
    trunk/libgfortran/generated/matmul_c8.c
    trunk/libgfortran/generated/matmul_i16.c
    trunk/libgfortran/generated/matmul_i4.c
    trunk/libgfortran/generated/matmul_i8.c
    trunk/libgfortran/generated/matmul_r10.c
    trunk/libgfortran/generated/matmul_r16.c
    trunk/libgfortran/generated/matmul_r4.c
    trunk/libgfortran/generated/matmul_r8.c
    trunk/libgfortran/m4/matmul.m4


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025

[Bug fortran/26025] Optionally use BLAS for matmul

[fxcoudert at gcc dot gnu dot org]

------- Comment #6 from fxcoudert at gcc dot gnu dot org  2006-10-22 07:43 -------
Fixed on mainline.


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26025