--- Comment #2 from pault at gcc dot gnu dot org 2009-02-12 09:19 ---
(In reply to comment #1)
Interestingly, if the WRITE statement is cut, the compilation now fails with:
pr36703.f90: In function 'gp':
pr36703.f90:46: internal compiler error: in expand_expr_real_1, at expr.c:7282
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=35476
--- Comment #1 from pault at gcc dot gnu dot org 2009-02-09 08:15 ---
Coo! I did't know anything about PR323. I now don't want to know anything
about it:-)
Confirmed
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from pault at gcc dot gnu dot org 2009-02-09 08:42 ---
Does not
if (ns-save_all || !gfc_option.flag_automatic)
gfc_save_all (ns);
in resolve_types not fix the problem? (I have not had a chance to try this
yet.)
Confirmed
Paul
--
pault at gcc dot gnu dot org
--- Comment #5 from pault at gcc dot gnu dot org 2009-02-06 13:44 ---
I guess that since Richard says that it's a problem, we had better confirm
it:-)
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from pault at gcc dot gnu dot org 2009-02-03 08:55 ---
(In reply to comment #4)
I would have said that the value of the integer component after the first
assignment is, at best, ill-defined. If L_TO_T assigns a value to it, gfortran
gives the same result as any other
--- Comment #6 from pault at gcc dot gnu dot org 2009-02-03 19:59 ---
I have just realised that this is a case of complete overlap that we miss
completely in dependency analysis:
If one of the lhs or rhs is a full array, the stride is unity and one of lbound
== start or ubound == end
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-28 17:31 ---
*** Bug 39006 has been marked as a duplicate of this bug. ***
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from pault at gcc dot gnu dot org 2009-01-28 17:31 ---
This is the same as PR38852 and the fix for that one works for this too.
Paul
*** This bug has been marked as a duplicate of 38852 ***
--
pault at gcc dot gnu dot org changed:
What|Removed
--- Comment #10 from pault at gcc dot gnu dot org 2009-01-28 17:33 ---
Marked up the severity to that of PR39006.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from pault at gcc dot gnu dot org 2009-01-28 20:52 ---
Created an attachment (id=17204)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17204action=view)
Updated patch
Following discussion on fortran list and #gfortran, a more restrictive test is
needed.
--
pault
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-28 21:49 ---
Subject: Bug 39006
Author: pault
Date: Wed Jan 28 21:48:53 2009
New Revision: 143743
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143743
Log:
2009-01-28 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #12 from pault at gcc dot gnu dot org 2009-01-28 21:49 ---
Subject: Bug 38852
Author: pault
Date: Wed Jan 28 21:48:53 2009
New Revision: 143743
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143743
Log:
2009-01-28 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #13 from pault at gcc dot gnu dot org 2009-01-28 21:52 ---
Fixed on trunk.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-27 12:16 ---
I have done most of the work on the corner cases and the fix below looks good.
I am marking it as pending for temporary book-keeping purposes.
Paul
--
pault at gcc dot gnu dot org changed:
What
--- Comment #3 from pault at gcc dot gnu dot org 2009-01-27 20:53 ---
See http://gcc.gnu.org/ml/fortran/2009-01/msg00217.html and subsequent
postings.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pault at gcc dot gnu dot org 2009-01-28 06:21 ---
See http://gcc.gnu.org/ml/fortran/2009-01/msg00331.html
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from pault at gcc dot gnu dot org 2009-01-25 18:21 ---
(In reply to comment #5)
(In reply to comment #4)
(In reply to comment #3)
In the latter case, it is non-empty if ubound lbound only. Comparing
ubound and lbound according to the stride to check for zero
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-25 21:08 ---
Created an attachment (id=17182)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17182action=view)
A provisional patch for the PR
It take back what I said previously:-)
The attached bootstraps and regtests OK.
I
--- Comment #17 from pault at gcc dot gnu dot org 2009-01-26 05:12 ---
Subject: Bug 38657
Author: pault
Date: Mon Jan 26 05:12:03 2009
New Revision: 143669
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143669
Log:
2009-01-26 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #18 from pault at gcc dot gnu dot org 2009-01-26 05:13 ---
Fixed on trunk and 4.3
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-26 05:43 ---
Subject: Bug 38859
Author: pault
Date: Mon Jan 26 05:43:44 2009
New Revision: 143670
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143670
Log:
2009-01-26 Mikael Morin mikael.mo...@tele2.fr
PR
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-26 05:44 ---
Closed on trunk and 4.3.
Once again, thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #10 from pault at gcc dot gnu dot org 2009-01-26 06:15 ---
Subject: Bug 38907
Author: pault
Date: Mon Jan 26 06:15:41 2009
New Revision: 143671
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143671
Log:
2009-01-26 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #11 from pault at gcc dot gnu dot org 2009-01-26 06:16 ---
Fixed on trunk and 4.3
Thanks for the report and thanks to Mikael for the fix.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pault at gcc dot gnu dot org 2009-01-24 12:53 ---
Created an attachment (id=17174)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17174action=view)
Fix for the PR -
OK'd http://gcc.gnu.org/ml/fortran/2009-01/msg00297.html
2009-01-22 Paul Thomas pa
--- Comment #3 from pault at gcc dot gnu dot org 2009-01-24 12:54 ---
See PR38918 for fix
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Summary
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-22 12:33 ---
This has a trivial fix:
expr.c: 2913 becomes
if (have_pointer lvalue-symtree-n.sym-attr.data)
Sorry about the lack of a diff - I will remedy this on submission:-)
paul
--
pault at gcc dot gnu dot org changed
--- Comment #5 from pault at gcc dot gnu dot org 2009-01-22 12:39 ---
(In reply to comment #4)
(In reply to comment #3)
In the latter case, it is non-empty if ubound lbound only. Comparing
ubound and lbound according to the stride to check for zero-sized arrays
doesn't make
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-22 15:00 ---
(In reply to comment #1)
Confirm. ICE with 4.1.x and 4.2.x and wrong code with 4.3.x and 4.4.
Thanks for the report.
The character length is being incorrectly set to 1 for the failing assignments;
ie. the last
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-22 20:16 ---
I am just about to post a fix.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-21 20:22 ---
(In reply to comment #1)
http://gcc.gnu.org/ml/gcc-patches/2009-01/msg00937.html
Richard,
I am not sure that any of the gfortran developers have the skills to deal with
this. I read the words about canonical
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-20 21:57 ---
Subject: Bug 38907
Author: pault
Date: Tue Jan 20 21:56:49 2009
New Revision: 143530
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143530
Log:
2009-01-20 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-20 21:59 ---
Fixed on trunk
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from pault at gcc dot gnu dot org 2009-01-19 23:08 ---
Created an attachment (id=17148)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17148action=view)
A patch for the PR
This regtests and bootstraps on FC9/x86_i64. I'll do ChangeLogs and so on
tomorrow.
Thanks
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-20 05:54 ---
Ticked off 4.4...:-)
Cheers
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #12 from pault at gcc dot gnu dot org 2009-01-17 08:39 ---
The fix has caused the regression reported in
http://gcc.gnu.org/ml/fortran/2009-01/msg00197.html so I am reverting on trunk
and 4.3.
Cheers
Paul
--
pault at gcc dot gnu dot org changed:
What
--- Comment #13 from pault at gcc dot gnu dot org 2009-01-17 09:29 ---
Subject: Bug 38657
Author: pault
Date: Sat Jan 17 09:28:50 2009
New Revision: 143458
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143458
Log:
2009-01-17 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #14 from pault at gcc dot gnu dot org 2009-01-17 09:44 ---
Subject: Bug 38657
Author: pault
Date: Sat Jan 17 09:43:57 2009
New Revision: 143459
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143459
Log:
2009-01-17 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #22 from pault at gcc dot gnu dot org 2009-01-17 11:32 ---
Subject: Bug 34955
Author: pault
Date: Sat Jan 17 11:32:02 2009
New Revision: 143462
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143462
Log:
2009-01-17 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #23 from pault at gcc dot gnu dot org 2009-01-17 11:34 ---
Fixed on trunk.
Thanks for the report and sorry that it took so long to fix.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #15 from pault at gcc dot gnu dot org 2009-01-17 11:58 ---
Subject: Bug 38657
Author: pault
Date: Sat Jan 17 11:58:48 2009
New Revision: 143463
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143463
Log:
2009-01-17 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-15 09:19 ---
(In reply to comment #1)
This would mean an ABI change, but it is worth thinking about.
The required fix is trivial, although I have not regtested yet; eliminate
1127-1131 in trans-common.c.
Cheers
Paul
--- Comment #20 from pault at gcc dot gnu dot org 2009-01-12 16:20 ---
Created an attachment (id=17079)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17079action=view)
A preliminary patch for the PR
A slightly earlier version of this regtested OK. Will do it to this version
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: pault at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38802
--- Comment #3 from pault at gcc dot gnu dot org 2009-01-11 13:42 ---
Subject: Bug 38765
Author: pault
Date: Sun Jan 11 13:42:32 2009
New Revision: 143271
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143271
Log:
2009-01-11 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-11 13:42 ---
Subject: Bug 38763
Author: pault
Date: Sun Jan 11 13:42:32 2009
New Revision: 143271
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143271
Log:
2009-01-11 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-12 06:21 ---
This does the job:
Index: gcc/fortran/trans-array.c
===
--- gcc/fortran/trans-array.c (revision 143242)
+++ gcc/fortran/trans-array.c (working copy
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-12 06:22 ---
Fixed on trunk and 4.3. Thanks for the reort.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #4 from pault at gcc dot gnu dot org 2009-01-12 06:23 ---
Fixed on trunk and 4.3. Thanks for the reort.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-10 21:01 ---
Subject: Bug 38665
Author: pault
Date: Sat Jan 10 21:01:14 2009
New Revision: 143248
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143248
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #10 from pault at gcc dot gnu dot org 2009-01-10 21:06 ---
Subject: Bug 38657
Author: pault
Date: Sat Jan 10 21:06:27 2009
New Revision: 143249
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143249
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #10 from pault at gcc dot gnu dot org 2009-01-10 21:25 ---
Subject: Bug 38665
Author: pault
Date: Sat Jan 10 21:24:54 2009
New Revision: 143250
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143250
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #11 from pault at gcc dot gnu dot org 2009-01-10 21:41 ---
Subject: Bug 38665
Author: pault
Date: Sat Jan 10 21:41:16 2009
New Revision: 143252
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143252
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #12 from pault at gcc dot gnu dot org 2009-01-10 21:44 ---
Fixed on trunk and latest 4.3
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38665
--- Comment #13 from pault at gcc dot gnu dot org 2009-01-10 21:44 ---
Fixed on trunk and latest 4.3
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from pault at gcc dot gnu dot org 2009-01-10 21:45 ---
Fixed on trunk and 4.3
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from pault at gcc dot gnu dot org 2009-01-10 00:11 ---
Subject: Bug 38765
Author: pault
Date: Sat Jan 10 00:11:18 2009
New Revision: 143237
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143237
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #6 from pault at gcc dot gnu dot org 2009-01-10 00:15 ---
Subject: Bug 38763
Author: pault
Date: Sat Jan 10 00:15:37 2009
New Revision: 143238
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143238
Log:
2009-01-10 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-10 00:21 ---
Fixed on trunk
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-10 00:21 ---
Fixed on trunk
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Version: 4.4.0
Status: UNCONFIRMED
Keywords: ice-on-valid-code
Severity: normal
Priority: P3
Component: fortran
AssignedTo: pault at gcc dot gnu dot org
ReportedBy: pault at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Ever Confirmed|0 |1
Last
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-08 16:42 ---
Steve,
gfc_target_encode_expr has no means to deal with EXPR_NULL. It's quite easy to
change that, so I have assigned myself.
Thanks for the report. Yet another don't be so hard! Making TRANSFER
work has been
--- Comment #5 from pault at gcc dot gnu dot org 2009-01-08 22:48 ---
(In reply to comment #3)
It was meant in jest. I'll change the summary to something
more pleasing.
I know:-)
BTW, James claims that some other failures occur, but I
can't reproduce those. See
http
--- Comment #2 from pault at gcc dot gnu dot org 2009-01-07 13:12 ---
(In reply to comment #0)
The lack of simplification is due to an omission in
target-memory.c(gfc_target_memory_size), in which the possibility of (1) a
constant cl-length and (ii) no defined character length, as here
--- Comment #6 from pault at gcc dot gnu dot org 2009-01-05 16:32 ---
(In reply to comment #1)
Assuming no local problems and a clean tree (looks like), the following commit
might have caused the regression:
r130257 | fxcoudert | 2007-11-17 14:46:53 +0100 (Sat, 17 Nov 2007) | 6
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-05 17:35 ---
I'm just about to post a patch.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-05 19:46 ---
Subject: Bug 38657
Author: pault
Date: Mon Jan 5 19:46:06 2009
New Revision: 143090
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143090
Log:
2009-01-05 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #9 from pault at gcc dot gnu dot org 2009-01-05 19:47 ---
Trunk done, 4.3 to come!
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-04 17:59 ---
(In reply to comment #6)
In the mean time, I know how to apply a fix along the lines of 1.
Ok, then; go for (1). I have nothing to propose.
I have just realised that the matching can be eliminated, although I
--- Comment #8 from pault at gcc dot gnu dot org 2009-01-04 23:17 ---
Subject: Bug 38665
Author: pault
Date: Sun Jan 4 23:17:37 2009
New Revision: 143064
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143064
Log:
2009-01-05 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #5 from pault at gcc dot gnu dot org 2009-01-03 14:15 ---
I have just posted a patch on the list.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from pault at gcc dot gnu dot org 2009-01-03 17:48 ---
Subject: Bug 38594
Author: pault
Date: Sat Jan 3 17:47:20 2009
New Revision: 143032
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=143032
Log:
2009-01-03 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #7 from pault at gcc dot gnu dot org 2009-01-03 22:58 ---
Fixed on trunk. Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from pault at gcc dot gnu dot org 2009-01-03 23:01 ---
Mikael,
Now the solutions:
(1) Add some conditions to the if before to prevent executing this.
(2) Remove the gfc_match_whatever that has nothing to do in resolve.c and
find
a better way (yes, I prefer (2
--- Comment #5 from pault at gcc dot gnu dot org 2008-12-31 10:47 ---
(In reply to comment #4)
Created an attachment (id=17016)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17016action=view) [edit]
fix
Does anyone know the use of the block variable I remove in this patch
Keywords: rejects-valid
Severity: normal
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: pault at gcc dot gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38657
--- Comment #15 from pault at gcc dot gnu dot org 2008-12-24 17:08 ---
(In reply to comment #13)
Dear Dominique,
print *, len(string)
print *, size(transfer(string,xy,len(string)))
yields a wrong code: the executable gives
8
4
ab
--- Comment #17 from pault at gcc dot gnu dot org 2008-12-23 09:26 ---
(In reply to comment #12)
(In reply to comment #11)
Patch committed, fixed on trunk, 4.4
I think it was the wrong PR number (35780 instead of PR34820).
Dang it! Sorry about that, I used the ChangeLogs from
--- Comment #8 from pault at gcc dot gnu dot org 2008-12-22 19:39 ---
Created an attachment (id=16969)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16969action=view)
patch for the PR
This is a patch that I cannot commit until January 3rd at the earliest. Please
could you do
--- Comment #9 from pault at gcc dot gnu dot org 2008-12-22 20:03 ---
Created an attachment (id=16970)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16970action=view)
The real patch for the PR without the extra bits:-)
Have eliminated the extra bits on the patch - sorry about
--- Comment #8 from pault at gcc dot gnu dot org 2008-12-22 21:55 ---
All routes to deal with this are way too invasive for this stage in the
proceedings. This PR must wait for the array descriptor upgrade, so I am
suspending it for now.
Paul
--
pault at gcc dot gnu dot org
--- Comment #12 from pault at gcc dot gnu dot org 2008-12-19 23:59 ---
Created an attachment (id=16949)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16949action=view)
a not quite patch for the PR
I'm down to one regression now - the last test of char_length_7.f90.
Pah
--- Comment #5 from pault at gcc dot gnu dot org 2008-12-18 09:13 ---
I am most of the way onto a solution for this so I'll assign myself.
The source of the problem is highlighted by two tests:
s/call foo(func(_//bar()))/call foo(func(bar()))/
works fine as does a modified testcase
--- Comment #6 from pault at gcc dot gnu dot org 2008-12-18 16:35 ---
Created an attachment (id=16928)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16928action=view)
Provisional patch for the PR
Testcase follows in next attachement.
--
http://gcc.gnu.org/bugzilla
--- Comment #7 from pault at gcc dot gnu dot org 2008-12-18 16:38 ---
Created an attachment (id=16929)
-- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=16929action=view)
testcase for the PR
This might have departed somewhat from the original but it does at least work
with the patch
--- Comment #19 from pault at gcc dot gnu dot org 2008-12-18 22:29 ---
I have noticed that the scalar TRANSFER does not respect the undefined
condition for the rest of the result if the SOURCE size is less than the MOLD
size. I have a patch for this, which is probably not complete
--- Comment #2 from pault at gcc dot gnu dot org 2008-12-17 12:20 ---
This is, of course, confirmed!
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #19 from pault at gcc dot gnu dot org 2008-12-14 16:01 ---
Subject: Bug 35937
Author: pault
Date: Sun Dec 14 16:00:25 2008
New Revision: 142750
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142750
Log:
2008-12-14 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #20 from pault at gcc dot gnu dot org 2008-12-14 16:09 ---
Subject: Bug 35937
Author: pault
Date: Sun Dec 14 16:07:46 2008
New Revision: 142751
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142751
Log:
2008-12-14 Paul Thomas pa...@gcc.gnu.org
PR fortran
--- Comment #21 from pault at gcc dot gnu dot org 2008-12-14 16:11 ---
Fixed on trunk and 4.3.
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #16 from pault at gcc dot gnu dot org 2008-12-11 09:38 ---
Hah! My VMware Fedora catches the bug and demonstrates that my simple fix of
comment #5 does the job; ie. unconditionally do the constant fold to a
gfc_charlen_type_node. This can never do any harm!
I will fix
--- Comment #5 from pault at gcc dot gnu dot org 2008-12-11 10:52 ---
Tobias,
It's a bit tough to call it a regression, since the earlier versions of
gfortran produced wrong code.
Is this worth fixing for 4.4? If we get the array descriptor business sorted
out, it should go away
--- Comment #6 from pault at gcc dot gnu dot org 2008-12-11 11:21 ---
This cures the ICE and allows correct code within the subroutine. 'span' is
not transferred in the call and so wrong code is produced if the pointer is
subsequently used in the caller.
I can apply it if you like
--- Comment #12 from pault at gcc dot gnu dot org 2008-12-03 05:52 ---
(In reply to comment #11)
Did you get the expected result also?
I couldn't tell - I did not have the 32 bit library installed. I'll try to fix
that.
Cheers
Paul
--
http://gcc.gnu.org/bugzilla
--- Comment #5 from pault at gcc dot gnu dot org 2008-11-30 08:04 ---
This is fixed for trunk and 4.3.
I have prepared a testcase that will be committed tonight.
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed
--- Comment #6 from pault at gcc dot gnu dot org 2008-11-30 08:50 ---
Subject: Bug 35824
Author: pault
Date: Sun Nov 30 08:48:51 2008
New Revision: 142292
URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=142292
Log:
2008-11-30 Paul Thomas [EMAIL PROTECTED]
PR fortran
--- Comment #10 from pault at gcc dot gnu dot org 2008-11-30 13:13 ---
(In reply to comment #9)
I might as well take it too:-)
Since I cannot reproduce the bug, even at -m32, I am unassigning myself.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed
--- Comment #6 from pault at gcc dot gnu dot org 2008-11-30 19:53 ---
Index: libgfortran/generated/reshape_r4.c
===
--- libgfortran/generated/reshape_r4.c (revision 142291)
+++ libgfortran/generated/reshape_r4.c (working
501 - 600 of 3028 matches
Mail list logo