public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/28947] New: Double MATMUL() uses wrong array elements
@ 2006-09-04 12:51 anlauf at gmx dot de
2006-09-04 12:51 ` [Bug fortran/28947] " anlauf at gmx dot de
` (6 more replies)
0 siblings, 7 replies; 8+ messages in thread
From: anlauf at gmx dot de @ 2006-09-04 12:51 UTC (permalink / raw)
To: gcc-bugs
Hi *,
the attached code exhibits bad code generation for a
double invocation of MATMUL ().
Running the code with any recent gfortran gives:
h(3,:) = -3.4028235E+38 -3.4028235E+38 -3.4028235E+38
Bummer!
The result should be:
h(3,:) = 2.000000 2.000000 3.000000
Cheers,
-ha
--
Summary: Double MATMUL() uses wrong array elements
Product: gcc
Version: 4.2.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: anlauf at gmx dot de
GCC host triplet: i686-pc-linux-gnu
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
@ 2006-09-04 12:51 ` anlauf at gmx dot de
2006-09-04 16:07 ` jvdelisle at gcc dot gnu dot org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: anlauf at gmx dot de @ 2006-09-04 12:51 UTC (permalink / raw)
To: gcc-bugs
------- Comment #1 from anlauf at gmx dot de 2006-09-04 12:51 -------
Created an attachment (id=12184)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=12184&action=view)
Demo code for MATMUL bug
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
2006-09-04 12:51 ` [Bug fortran/28947] " anlauf at gmx dot de
@ 2006-09-04 16:07 ` jvdelisle at gcc dot gnu dot org
2006-09-09 14:17 ` pault at gcc dot gnu dot org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: jvdelisle at gcc dot gnu dot org @ 2006-09-04 16:07 UTC (permalink / raw)
To: gcc-bugs
--
jvdelisle at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Severity|normal |major
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
2006-09-04 12:51 ` [Bug fortran/28947] " anlauf at gmx dot de
2006-09-04 16:07 ` jvdelisle at gcc dot gnu dot org
@ 2006-09-09 14:17 ` pault at gcc dot gnu dot org
2006-09-10 17:22 ` pault at gcc dot gnu dot org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-09 14:17 UTC (permalink / raw)
To: gcc-bugs
------- Comment #2 from pault at gcc dot gnu dot org 2006-09-09 14:17 -------
I have just posted a patch to the list.
Paul
--
pault at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
AssignedTo|unassigned at gcc dot gnu |pault at gcc dot gnu dot org
|dot org |
Status|UNCONFIRMED |ASSIGNED
Ever Confirmed|0 |1
Last reconfirmed|0000-00-00 00:00:00 |2006-09-09 14:17:31
date| |
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
` (2 preceding siblings ...)
2006-09-09 14:17 ` pault at gcc dot gnu dot org
@ 2006-09-10 17:22 ` pault at gcc dot gnu dot org
2006-09-10 17:27 ` pault at gcc dot gnu dot org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-10 17:22 UTC (permalink / raw)
To: gcc-bugs
------- Comment #3 from pault at gcc dot gnu dot org 2006-09-10 17:21 -------
Subject: Bug 28947
Author: pault
Date: Sun Sep 10 17:21:44 2006
New Revision: 116817
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=116817
Log:
2006-09-10 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28947
* m4/matmul.m4: For the case where the second input argument is
transposed, ensure that the case with rank (a) == 1 is
correctly calculated.
* generated/matmul_r4.c: Regenerate.
* generated/matmul_r8.c: Regenerate.
* generated/matmul_r10.c: Regenerate.
* generated/matmul_r16.c: Regenerate.
* generated/matmul_c4.c: Regenerate.
* generated/matmul_c8.c: Regenerate.
* generated/matmul_c10.c: Regenerate.
* generated/matmul_c16.c: Regenerate.
* generated/matmul_i4.c: Regenerate.
* generated/matmul_i8.c: Regenerate.
* generated/matmul_i16.c: Regenerate.
2006-09-10 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28947
gfortran.dg/matmul_4.f90: New test.
Added:
trunk/gcc/testsuite/gfortran.dg/matmul_4.f90
Modified:
trunk/gcc/testsuite/ChangeLog
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=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
` (3 preceding siblings ...)
2006-09-10 17:22 ` pault at gcc dot gnu dot org
@ 2006-09-10 17:27 ` pault at gcc dot gnu dot org
2006-09-10 17:32 ` pault at gcc dot gnu dot org
2006-09-10 19:45 ` pault at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-10 17:27 UTC (permalink / raw)
To: gcc-bugs
------- Comment #4 from pault at gcc dot gnu dot org 2006-09-10 17:27 -------
Subject: Bug 28947
Author: pault
Date: Sun Sep 10 17:26:54 2006
New Revision: 116818
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=116818
Log:
2006-09-10 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28947
* m4/matmul.m4: For the case where the second input argument is
transposed, ensure that the case with rank (a) == 1 is
correctly calculated.
* generated/matmul_r4.c: Regenerate.
* generated/matmul_r8.c: Regenerate.
* generated/matmul_r10.c: Regenerate.
* generated/matmul_r16.c: Regenerate.
* generated/matmul_c4.c: Regenerate.
* generated/matmul_c8.c: Regenerate.
* generated/matmul_c10.c: Regenerate.
* generated/matmul_c16.c: Regenerate.
* generated/matmul_i4.c: Regenerate.
* generated/matmul_i8.c: Regenerate.
* generated/matmul_i16.c: Regenerate.
2006-09-10 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28947
gfortran.dg/matmul_4.f90: New test.
Modified:
branches/gcc-4_1-branch/gcc/fortran/ChangeLog
branches/gcc-4_1-branch/gcc/fortran/expr.c
branches/gcc-4_1-branch/gcc/fortran/trans-types.c
branches/gcc-4_1-branch/gcc/testsuite/ChangeLog
branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/array_initializer_2.f90
branches/gcc-4_1-branch/libgfortran/ChangeLog
branches/gcc-4_1-branch/libgfortran/generated/matmul_c10.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_c16.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_c4.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_c8.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_i16.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_i4.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_i8.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_r10.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_r16.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_r4.c
branches/gcc-4_1-branch/libgfortran/generated/matmul_r8.c
branches/gcc-4_1-branch/libgfortran/m4/matmul.m4
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
` (4 preceding siblings ...)
2006-09-10 17:27 ` pault at gcc dot gnu dot org
@ 2006-09-10 17:32 ` pault at gcc dot gnu dot org
2006-09-10 19:45 ` pault at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-10 17:32 UTC (permalink / raw)
To: gcc-bugs
------- Comment #5 from pault at gcc dot gnu dot org 2006-09-10 17:32 -------
Subject: Bug 28947
Author: pault
Date: Sun Sep 10 17:32:22 2006
New Revision: 116819
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=116819
Log:
2006-09-09 Paul Thomas <pault@gcc.gnu.org>
PR fortran/28923
expr.c (find_array_section): Only use the array lower and upper
bounds for the start and end of the sections, where the expr is
NULL.
PR fortran/28959
trans-types.c (gfc_get_derived_type): Use the parent namespace of
the procedure if the type's own namespace does not have a parent.
2006-09-09 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28947
* m4/matmul.m4: For the case where the second input argument is
transposed, ensure that the case with rank (a) == 1 is
correctly calculated.
* generated/matmul_r4.c: Regenerate.
* generated/matmul_r8.c: Regenerate.
* generated/matmul_r10.c: Regenerate.
* generated/matmul_r16.c: Regenerate.
* generated/matmul_c4.c: Regenerate.
* generated/matmul_c8.c: Regenerate.
* generated/matmul_c10.c: Regenerate.
* generated/matmul_c16.c: Regenerate.
* generated/matmul_i4.c: Regenerate.
* generated/matmul_i8.c: Regenerate.
* generated/matmul_i16.c: Regenerate.
2006-09-09 Paul Thomas <pault@gcc.gnu.org>
PR libfortran/28923
gfortran.dg/array_initializer_2.f90: Fill in missing index start value.
gfortran.dg/array_initializer_3.f90: New test.
PR libfortran/28947
gfortran.dg/matmul_4.f90: New test.
PR fortran/28959
gfortran.dg/used_types_10: New test.
Added:
branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/array_initializer_3.f90
branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/matmul_4.f90
branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/used_types_10.f90
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug fortran/28947] Double MATMUL() uses wrong array elements
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
` (5 preceding siblings ...)
2006-09-10 17:32 ` pault at gcc dot gnu dot org
@ 2006-09-10 19:45 ` pault at gcc dot gnu dot org
6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-10 19:45 UTC (permalink / raw)
To: gcc-bugs
------- Comment #6 from pault at gcc dot gnu dot org 2006-09-10 19:44 -------
Fixed on trunk and 4.1
Paul
--
pault at gcc dot gnu dot org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28947
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2006-09-10 19:45 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-09-04 12:51 [Bug fortran/28947] New: Double MATMUL() uses wrong array elements anlauf at gmx dot de
2006-09-04 12:51 ` [Bug fortran/28947] " anlauf at gmx dot de
2006-09-04 16:07 ` jvdelisle at gcc dot gnu dot org
2006-09-09 14:17 ` pault at gcc dot gnu dot org
2006-09-10 17:22 ` pault at gcc dot gnu dot org
2006-09-10 17:27 ` pault at gcc dot gnu dot org
2006-09-10 17:32 ` pault at gcc dot gnu dot org
2006-09-10 19:45 ` pault 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).