--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Known to fail||4.3.0
Known to work||4.2.1 4.1.3
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-14 21:33
---
Confirmed on x86_64-linux with -m32:
$ cat a.f90
real x(1)
call foo(nint(x))
end
$ gfortran a.f90 -m32
a.f90: In function MAIN__:
a.f90:1: internal compiler error: in gfc_add_modify, at fortran/trans.c
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-14 21:14
---
(In reply to comment #4)
> Your patch fixed my test cases.
>
> I do not have testresults on my HP workstation. I will try to find a copy of
> gfortran.dg/logical_3.f90 and test it.
OK, so I'
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|fxcoudert at gcc dot gnu dot|unassigned at gcc dot gnu
|org
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-14 15:26
---
(In reply to comment #7)
> I suspect that '' is labels.0 (though it *has* a name!)
Well, that was wrong. If you look at the gimple tree (004t.gimple), it contains
the following:
print_sub (l, la
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-14 14:28
---
I'll try to deal with that one, it looks nasty (although target-specific).
Thanks for the bug-report.
1. Can you report what HP-PA architecture this is happening on? And the OS?
(I have access to HP test
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
--- Comment #7 from fxcoudert at gcc dot gnu dot org 2007-08-14 14:10
---
Reduced testcase:
subroutine print_sub(l, labels)
logical :: l
character (len=*), optional :: labels(1)
if (l) call foo
if (present(labels)) then
print *, labels(1)
end if
end
The logical and
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32955
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32962
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32977
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32987
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33038
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33039
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33054
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-14 13:42
---
(In reply to comment #3)
> One needs therefore to replace the current code by something which calls the
> library.
This was done, and the current error message was corrected.
--
fxcoudert at gcc dot g
--- Comment #9 from fxcoudert at gcc dot gnu dot org 2007-08-14 12:47
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-14 12:44
---
Subject: Bug 32594
Author: fxcoudert
Date: Tue Aug 14 12:44:19 2007
New Revision: 127478
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127478
Log:
PR fortran/32594
* tran
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-14 09:04
---
write(6,*) (/(s1(),i=1,3,-1)/)
!write(6,*) shape((/(s1(),i=1,3,-1)/))
contains
function s1()
character(len=1) :: s1
s1=" "
end function s1
end
Further reduced: no need for the subst
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-13 23:48
---
A complete patch (for both original problem and a few other ones discovered
while preparing the testcase) was submitted here:
http://gcc.gnu.org/ml/gcc-patches/2007-08/msg00834.html
--
fxcoudert at gcc dot
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-13 10:25
---
As the error message you quote says:
"the most common reason for that is that you might have linked it to shared GMP
and/or MPFR libraries, and not set LD_LIBRARY_PATH accordingly. If you suspect
any
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-12 21:26
---
Updated summary to reflect current state: only MINLOC, MAXLOC and SHAPE needs
to be done. I'm not working on it any more.
--
fxcoudert at gcc dot gnu dot org changed:
What|Re
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-12 21:21
---
Subject: Bug 29600
Author: fxcoudert
Date: Sun Aug 12 21:21:08 2007
New Revision: 127385
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127385
Log:
PR fortran/29600
* int
--- Comment #12 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:46
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:46
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:45
---
Subject: Bug 33054
Author: fxcoudert
Date: Sun Aug 12 20:45:29 2007
New Revision: 127383
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127383
Log:
PR fortran/30964
PR fortr
--- Comment #11 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:45
---
Subject: Bug 30964
Author: fxcoudert
Date: Sun Aug 12 20:45:29 2007
New Revision: 127383
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127383
Log:
PR fortran/30964
PR fortr
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:42
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:39
---
Subject: Bug 32860
Author: fxcoudert
Date: Sun Aug 12 20:39:18 2007
New Revision: 127382
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127382
Log:
PR fortran/32860
*
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:20
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #7 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:20
---
Subject: Bug 31629
Author: fxcoudert
Date: Sun Aug 12 20:19:59 2007
New Revision: 127381
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127381
Log:
PR fortran/31629
* lang.opt (
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:16
---
(In reply to comment #5)
> Note that a test case would be welcome.
There is one in the patch I submitted (and which was approved).
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31629
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:15
---
(In reply to comment #2)
> Why not create a new i for the inner loop instead of saving it off?
This is indeed what we should do; however, I can't find a way to work it out.
Unassigning myself.
--
f
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|fxcoudert at gcc dot gnu dot|unassigned at gcc dot gnu
|org
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-12 20:13
---
Fixed, thanks Tobias for the first version of the patch.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-12 19:57
---
Subject: Bug 29600
Author: fxcoudert
Date: Sun Aug 12 19:57:01 2007
New Revision: 127380
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127380
Log:
PR fortran/29600
* int
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-12 19:50
---
(In reply to comment #7)
> It might even be better to do bit-wise shuffling to get
> the most out of the millisecond-field in date_and_time.
I thought about that at the beginning, but then it really gets
--- Comment #10 from fxcoudert at gcc dot gnu dot org 2007-08-12 16:41
---
Complete patch submitted:
http://gcc.gnu.org/ml/gcc-patches/2007-08/msg00782.html
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-12 16:40
---
(In reply to comment #1)
> The arguments to random_seed are restricted to default
> integer, at least up to F 2003, so we should just reject anything else.
> We'd then need two versions of random_s
--- Comment #9 from fxcoudert at gcc dot gnu dot org 2007-08-12 10:27
---
Fixed by Paul.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Severity|enhancement |normal
Status|UNCONFIRMED |NEW
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
Keywords: wrong-code
Severity: normal
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: fxcoudert at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33054
--- Comment #9 from fxcoudert at gcc dot gnu dot org 2007-08-12 00:16
---
Haven't yet had time to regtest this patch, but it should fix the bug:
Index: trans-expr.c
===
--- trans-expr.c(revision 127363)
+++
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-11 23:33
---
(In reply to comment #7)
> The reason for that is in gfc_trans_call
This should be: gfc_conv_function_call
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30964
--- Comment #7 from fxcoudert at gcc dot gnu dot org 2007-08-11 23:22
---
(In reply to comment #6)
> For the first one, gfc_conv_missing_dummy() is called, which leads to correct
> code. For the second one, gfc_conv_missing_dummy() is not called, leading to
> wrong-code.
T
--- Comment #7 from fxcoudert at gcc dot gnu dot org 2007-08-11 21:53
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2007-08-11 21:52
---
Subject: Bug 31189
Author: fxcoudert
Date: Sat Aug 11 21:52:22 2007
New Revision: 127364
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127364
Log:
PR fortran/31189
* runtime/bac
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-11 21:31
---
Subject: Bug 32937
Author: fxcoudert
Date: Sat Aug 11 21:31:35 2007
New Revision: 127363
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127363
Log:
PR fortran/32937
* trans
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2007-08-11 11:17
---
Hurray! It's been my wildest gdb session since a very long time (two sessions
of two hours each), but I've made it at last! My analysis in the previous
comment was right, we are generating a
--- Comment #5 from fxcoudert at gcc dot gnu dot org 2007-08-10 22:12
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-10 22:12
---
Subject: Bug 31270
Author: fxcoudert
Date: Fri Aug 10 22:12:04 2007
New Revision: 127352
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127352
Log:
PR fortran/31270
*
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-10 21:28
---
(In reply to comment #5)
> Your attempt to randomize the seeds really isn't all that good because
> there is such a limited amount of entropy in year, month, day, and hour.
I do agree. There's n
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-10 19:23
---
(In reply to comment #3)
> UGH.
>
> I'd rather document gfortran's behavior than add additional hacks
> into the compiler.
Hum, I was of the same opinion at first. What somehow convinc
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-10 19:04
---
I guess we have to scramble the seed given by the user, something like this:
Index: intrinsics/random.c
===
--- intrinsics/random.c (revision
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-10 15:31
---
The problem with integer kind is from the string length:
character(2_8) :: c(2)
integer :: i
c = "aa"
print *, c .eq. "aa"
call foo ((/(c(i), i = 1,2)/))
print *, c .eq. "a
--- Comment #11 from fxcoudert at gcc dot gnu dot org 2007-08-10 13:25
---
Fixed.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #10 from fxcoudert at gcc dot gnu dot org 2007-08-10 13:21
---
Subject: Bug 32933
Author: fxcoudert
Date: Fri Aug 10 13:20:46 2007
New Revision: 127334
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127334
Log:
PR fortran/32933
* tran
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-10 12:34
---
The further I can reduce this is:
$ cat u.f90
character(2) :: c(2)
integer(kind=4) :: i
c = "aa"
call foo ((/(c(i), i = 1_4,2_4)/))
print *, c .eq. "aa"
contains
subroutine f
--- Comment #9 from fxcoudert at gcc dot gnu dot org 2007-08-10 10:31
---
(In reply to comment #2)
> But there is, internally with __builtin_isnan.
And that's where I got it wrong. In the Fortran front-end, boolean_type_node is
an integer of the default logical kind, while I
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-09 23:06
---
How on earth did that PR get assigned to me?
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
OtherBugsDependingO||32630
nThis
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-09 22:20
---
Patch submitted for review.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-09 21:58
---
The patch in the previous comment wasn't even building, here is a one that
regtests and that I submitted to review.
--
fxcoudert at gcc dot gnu dot org changed:
What|Re
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-09 21:56
---
Still can't reproduce this on hppa2.0w-hp-hpux11.31. I'm starting to think that
printf might be broken there, after all. Since it's a corner case on a platform
that is not the current one, I'd
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-09 21:54
---
(In reply to comment #1)
> I have looked into this bug and I think it is an HP math lib bug. The 64 bit
> powf function is using %fr12R but not saving/restoring it the way it is
> supposed to. I have a
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-09 21:51
---
(In reply to comment #3)
> It looks like lroundl there is an implementation using roundl. There
> is an implementation of roundl using ceill, but ceill isn't available.
This has been fixed since, so
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-09 09:35
---
What about the following:
Index: gfortran.h
===
--- gfortran.h (revision 127224)
+++ gfortran.h (working copy)
@@ -1865,6 +1865,7 @@ typedef
strap fails on gcc/tree.c
Product: gcc
Version: 4.3.0
Status: UNCONFIRMED
Keywords: build
Severity: blocker
Priority: P3
Component: bootstrap
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: fxcoudert at gcc d
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-08 23:38
---
Fixed, with mention of C_SIZE_T.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from fxcoudert at gcc dot gnu dot org 2007-08-08 23:36
---
Subject: Bug 32902
Author: fxcoudert
Date: Wed Aug 8 23:36:24 2007
New Revision: 127307
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=127307
Log:
PR fortran/32902
* intrin
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:37
---
The following are easy to fix (I have a patch that just needs writing doc, I'll
submit later today): COUNT, IACHAR, ICHAR, INDEX, LBOUND, LEN, LEN_TRIM, SCAN,
SIZE, UBOUND, VERIFY.
This will leave: ACHAR (n
--- Comment #4 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:32
---
4.1 and 4.2 only means closing as WONTFIX, sorry :(
Please reopen if this bug happens with the latest gfortran.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--- Comment #8 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:27
---
See PR32943 for other examples of testcases that exhibit the same ICE. When
this one is fixed, we should make sure all these have gone.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32933
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:26
---
*** This bug has been marked as a duplicate of 32933 ***
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:26
---
*** Bug 32943 has been marked as a duplicate of this bug. ***
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:22
---
I think it is indeed valid.
type :: my_type
integer :: data
end type my_type
interface
subroutine func1(param)
import :: my_type
type(my_type) :: param
end subroutine func1
end
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |fxcoudert at gcc dot gnu dot
|dot org
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:08
---
Why not?
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-08 17:05
---
Target specific, I can't reproduce it on x86_64-linux, with both -m32 and -m64,
even when using valgrind.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |
--- Comment #1 from fxcoudert at gcc dot gnu dot org 2007-08-08 16:50
---
What do you think about the following doc patch? I'm CCing Thomas Koenig
because he's the one who implemented SIZEOF, IIRC.
Index: intr
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
CC||fxcoudert at gcc dot gnu dot
--- Comment #2 from fxcoudert at gcc dot gnu dot org 2007-08-08 15:26
---
Confirmed on x86_64-linux with -m32.
--
fxcoudert at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from fxcoudert at gcc dot gnu dot org 2007-08-08 15:24
---
I reduced bounds_check_5.f90 to the following:
$ cat o2.f90
INTEGER, POINTER :: x(:)
logical :: l
l = associated(x,x)
end
$ gfortran -fdefault-integer-8 -m32 o2.f90
o2.f90: In function MAIN__:
o2.f90:3
1101 - 1200 of 3201 matches
Mail list logo