--- Comment #13 from patchapp at dberlin dot org 2007-07-24 08:45 ---
Subject: Bug number PR32867
A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2007-07/msg01713.html
--
--- Comment #14 from dfranke at gcc dot gnu dot org 2007-07-24 16:57
---
Subject: Bug 32867
Author: dfranke
Date: Tue Jul 24 16:57:02 2007
New Revision: 126882
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=126882
Log:
gcc/fortran:
2007-07-24 Daniel Franke [EMAIL PROTECTED]
--- Comment #1 from hjl at lucon dot org 2007-07-23 15:02 ---
(gdb) p b-value.character
$10 = {length = 15180528, string = 0xe660fd __trim_1}
(gdb) p a-value.character
$11 = {length = 15170912, string = 0xe660fd __trim_1}
The length field doesn't look right.
--
--- Comment #2 from dfranke at gcc dot gnu dot org 2007-07-23 15:08 ---
Not yet.
http://gcc.gnu.org/ml/gcc-patches/2007-07/msg01652.html
is an update for the quoted patch, it might help to the extend that you get an
error, not an ice.
What does the corresponding Fortran code look
--- Comment #3 from hjl at lucon dot org 2007-07-23 16:09 ---
(In reply to comment #2)
Not yet.
http://gcc.gnu.org/ml/gcc-patches/2007-07/msg01652.html
is an update for the quoted patch, it might help to the extend that you get an
error, not an ice.
It makes no difference.
--- Comment #4 from hjl at lucon dot org 2007-07-23 16:23 ---
Here is a small testcase:
[EMAIL PROTECTED] 459.GemsFDTD]$ cat foo.f90
MODULE Readdata_mod
IMPLICIT NONE
Private
Public Parser
integer, parameter:: nkeywords = 2
character(80), PARAMETER,
--- Comment #5 from hjl at lucon dot org 2007-07-23 16:31 ---
A smaller testcase:
[EMAIL PROTECTED] 459.GemsFDTD]$ cat foo.f90
SUBROUTINE Parser(nx, ny, keyword)
character(80), PARAMETER, dimension(1:2) :: keywords =
(/'PROBLEMSIZE',
'NFTRANS_TD'/)
integer, intent(inout)
--- Comment #6 from dfranke at gcc dot gnu dot org 2007-07-23 16:36 ---
Confirmed. Will look into it.
As it works with scalars, I think I forgot to take arrays into account ...
--
dfranke at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from jv244 at cam dot ac dot uk 2007-07-23 18:46 ---
(In reply to comment #5)
character(80), PARAMETER, dimension(1:2) :: keywords =
(/'PROBLEMSIZE',
'NFTRANS_TD'/)
it is probably not related to the ICE, but the above is invalid F95.
Error: test.f90,
--- Comment #8 from dfranke at gcc dot gnu dot org 2007-07-23 18:50 ---
it is probably not related to the ICE, but the above is invalid F95.
Error: test.f90, line 2: Array constructor values have differing CHARACTER
lengths (11 and 10). gfortran should error out on this.
This is
--- Comment #9 from dfranke at gcc dot gnu dot org 2007-07-23 19:42 ---
Currently regtesting the patch below without problem so far.
H.J. could you please verify that it fixes your problem?
Index: expr.c
===
--- expr.c
--- Comment #10 from hjl at lucon dot org 2007-07-23 19:53 ---
(In reply to comment #9)
Currently regtesting the patch below without problem so far.
H.J. could you please verify that it fixes your problem?
Index: expr.c
--- Comment #11 from dfranke at gcc dot gnu dot org 2007-07-23 20:08
---
Patch in comment #9 passed regression testing on i686-pc-linux-gnu.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32867
--- Comment #12 from hjl at lucon dot org 2007-07-24 03:33 ---
(In reply to comment #9)
Currently regtesting the patch below without problem so far.
H.J. could you please verify that it fixes your problem?
Index: expr.c
14 matches
Mail list logo