public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/26025]  New: Optionally use BLAS for matmul
@ 2006-01-30  6:57 tkoenig at gcc dot gnu dot org
  2006-01-30 12:36 ` [Bug fortran/26025] " pinskia at gcc dot gnu dot org
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: tkoenig at gcc dot gnu dot org @ 2006-01-30  6:57 UTC (permalink / raw)
  To: gcc-bugs

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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
@ 2006-01-30 12:36 ` pinskia at gcc dot gnu dot org
  2006-02-21 20:55 ` jb at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-01-30 12:36 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
  2006-01-30 12:36 ` [Bug fortran/26025] " pinskia at gcc dot gnu dot org
@ 2006-02-21 20:55 ` jb at gcc dot gnu dot org
  2006-09-25  8:58 ` fxcoudert at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: jb at gcc dot gnu dot org @ 2006-02-21 20:55 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
  2006-01-30 12:36 ` [Bug fortran/26025] " pinskia at gcc dot gnu dot org
  2006-02-21 20:55 ` jb at gcc dot gnu dot org
@ 2006-09-25  8:58 ` fxcoudert at gcc dot gnu dot org
  2006-10-07 10:02 ` steven at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2006-09-25  8:58 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
                   ` (2 preceding siblings ...)
  2006-09-25  8:58 ` fxcoudert at gcc dot gnu dot org
@ 2006-10-07 10:02 ` steven at gcc dot gnu dot org
  2006-10-22  7:42 ` fxcoudert at gcc dot gnu dot org
  2006-10-22  7:43 ` fxcoudert at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: steven at gcc dot gnu dot org @ 2006-10-07 10:02 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
                   ` (3 preceding siblings ...)
  2006-10-07 10:02 ` steven at gcc dot gnu dot org
@ 2006-10-22  7:42 ` fxcoudert at gcc dot gnu dot org
  2006-10-22  7:43 ` fxcoudert at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2006-10-22  7:42 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [Bug fortran/26025] Optionally use BLAS for matmul
  2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
                   ` (4 preceding siblings ...)
  2006-10-22  7:42 ` fxcoudert at gcc dot gnu dot org
@ 2006-10-22  7:43 ` fxcoudert at gcc dot gnu dot org
  5 siblings, 0 replies; 7+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2006-10-22  7:43 UTC (permalink / raw)
  To: gcc-bugs



------- 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


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2006-10-22  7:43 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-01-30  6:57 [Bug fortran/26025] New: Optionally use BLAS for matmul tkoenig at gcc dot gnu dot org
2006-01-30 12:36 ` [Bug fortran/26025] " pinskia at gcc dot gnu dot org
2006-02-21 20:55 ` jb at gcc dot gnu dot org
2006-09-25  8:58 ` fxcoudert at gcc dot gnu dot org
2006-10-07 10:02 ` steven at gcc dot gnu dot org
2006-10-22  7:42 ` fxcoudert at gcc dot gnu dot org
2006-10-22  7:43 ` fxcoudert at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).