--- Comment #1 from pault at gcc dot gnu dot org 2010-02-15 07:51 ---
Yes, indeed.
In fact, S2((/(real (i),i=1,2)/)) produces calls to pack and unpack in both S0
and S1.
I'll take a look at it.
Thanks
Paul
--
pault at gcc dot gnu dot org changed:
What|Re
--- Comment #7 from pault at gcc dot gnu dot org 2010-02-15 11:57 ---
Since Tobias and I already did the business on trunk, I suppose that one of us
should take ownership of it.
Do we backport to 4.4 or just close it? I would go for the backporting.
Paul
--
pault at gcc dot gnu
--- Comment #10 from pault at gcc dot gnu dot org 2010-02-16 09:04 ---
(In reply to comment #6)
Joost,
I held back from committing the patch this morning because I noticed that your
testcase below is also calling pack/unpack like it is going out of business.
I have to confess that I
--- Comment #10 from pault at gcc dot gnu dot org 2010-02-16 10:59 ---
(In reply to comment #9)
> Fixed.
>
Richard,
With one thing and another this slipped my mind.
Sorry
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42997
--- Comment #3 from pault at gcc dot gnu dot org 2010-02-16 15:46 ---
(In reply to comment #2)
I have just patched a tree with the fix for PR36932 on an i686 at work. None
of these temporaries appear!
Something is different between 32 and 64 bits..
Paul
--
http://gcc.gnu.org
ary produced for array
pointer actual arguments
Product: gcc
Version: 4.5.0
Status: UNCONFIRMED
Keywords: wrong-code
Severity: normal
Priority: P3
Component: fortran
AssignedTo: pault at gcc dot gnu dot
--- Comment #2 from pault at gcc dot gnu dot org 2010-02-18 10:26 ---
(In reply to comment #1)
> Why is this marked as 4.4 regression? According to my regression hunt it fails
> since the patch http://gcc.gnu.org/viewcvs?view=revision&revision=156749 which
> has only be com
--- Comment #3 from pault at gcc dot gnu dot org 2010-02-20 12:47 ---
Subject: Bug 43111
Author: pault
Date: Sat Feb 20 12:46:43 2010
New Revision: 156926
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156926
Log:
2010-02-20 Paul Thomas
PR fortran/36932
--- Comment #4 from pault at gcc dot gnu dot org 2010-02-20 12:47 ---
Subject: Bug 43072
Author: pault
Date: Sat Feb 20 12:46:43 2010
New Revision: 156926
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156926
Log:
2010-02-20 Paul Thomas
PR fortran/36932
--- Comment #12 from pault at gcc dot gnu dot org 2010-02-20 12:47 ---
Subject: Bug 36932
Author: pault
Date: Sat Feb 20 12:46:43 2010
New Revision: 156926
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156926
Log:
2010-02-20 Paul Thomas
PR fortran/36932
--- Comment #8 from pault at gcc dot gnu dot org 2010-02-20 12:47 ---
Subject: Bug 36933
Author: pault
Date: Sat Feb 20 12:46:43 2010
New Revision: 156926
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156926
Log:
2010-02-20 Paul Thomas
PR fortran/36932
--- Comment #4 from pault at gcc dot gnu dot org 2010-02-20 12:47 ---
Fixed.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #7 from pault at gcc dot gnu dot org 2010-02-22 05:44 ---
Subject: Bug 43072
Author: pault
Date: Mon Feb 22 05:43:57 2010
New Revision: 156949
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=156949
Log:
2010-02-22 Paul Thomas
PR fortr
--- Comment #25 from pault at gcc dot gnu dot org 2010-02-22 05:45 ---
Fixed on trunk. Thanks for reportimg the problems and all the help, Tobias and
Joost.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #5 from pault at gcc dot gnu dot org 2010-02-22 05:45 ---
Fixed on trunk. Thanks for reportimg the problems and all the help, Tobias and
Joost.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pault at gcc dot gnu dot org 2010-02-22 05:48 ---
Fixed on trunk. Thanks for reportimg the problem, Joost.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
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=43136
--- Comment #7 from pault at gcc dot gnu dot org 2010-02-24 06:46 ---
(In reply to comment #6)
> Just to make sure it won't get forgotten: REOPEN.
>
> (In reply to comment #4)
> > Paul, for the test case in comment 0 we still create a temporary:
> >
> >
--- Comment #2 from pault at gcc dot gnu dot org 2010-02-24 20:31 ---
I have been having trouble in recent times detecting these memory leaks.
Neither FC9 nor RHEL5.2 x86_64 seem to show them with valgrind. Since I know
that others still see them, I wonder are there any ideas what I
--- Comment #1 from pault at gcc dot gnu dot org 2010-02-25 12:42 ---
symbol.c:gfc_symbols_could_alias is the source of this extra temporary (called
by trans-array.c:gfc_could_be_alias and before that
gfc_conv_resolve_dependencies).
I am reading section 12.6 of Adams, Brainerd
--- Comment #1 from pault at gcc dot gnu dot org 2010-02-25 12:48 ---
This does not occur on i386. I will try on a x86_64 tonight.
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43173
--- Comment #5 from pault at gcc dot gnu dot org 2010-02-25 21:42 ---
I cannot persuade trunk to produce different output to 4.4.
I am not saying that you are wrong but I can only reiterate Joost's request for
a testcase in which the result changes for the lack of a temporary.
--- Comment #3 from pault at gcc dot gnu dot org 2010-02-26 05:28 ---
> The warning for -Warray-temporaries is shown here for both -m32 and -m64 --
> and
> I also see "_gfortran_internal_pack" in the -fdump-tree-original. Do you have
> by chance local change
--- Comment #1 from pault at gcc dot gnu dot org 2010-02-26 07:35 ---
This fixes it and regtests.
Index: gcc/fortran/check.c
===
*** gcc/fortran/check.c (revision 157061)
--- gcc/fortran/check.c (working copy
--- Comment #4 from pault at gcc dot gnu dot org 2010-02-26 16:06 ---
(In reply to comment #3)
> > by chance local changes which fix this issue?
> I will go back and confirm that the tree on my machine at work is clean.
No, it wasn't, so my comment was incorrect.
--- Comment #10 from pault at gcc dot gnu dot org 2010-02-28 13:46 ---
(In reply to comment #9)
> Reduced test case below. The problem is the call. On the trunk the call looks
> as follows:
> set_set_v (&ru, D.1578);
> which is complete nonesense. It should be:
>
--- Comment #11 from pault at gcc dot gnu dot org 2010-03-02 11:58 ---
Subject: Bug 43180
Author: pault
Date: Tue Mar 2 11:58:02 2010
New Revision: 157163
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157163
Log:
2010-03-02 Paul Thomas
PR fortr
--- Comment #10 from pault at gcc dot gnu dot org 2010-03-02 11:58 ---
Subject: Bug 43173
Author: pault
Date: Tue Mar 2 11:58:02 2010
New Revision: 157163
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157163
Log:
2010-03-02 Paul Thomas
PR fortr
--- Comment #11 from pault at gcc dot gnu dot org 2010-03-02 12:10 ---
Fixed on trunk, thanks for the report!
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #12 from pault at gcc dot gnu dot org 2010-03-02 12:11 ---
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 2010-03-03 17:50 ---
Subject: Bug 43243
Author: pault
Date: Wed Mar 3 17:49:53 2010
New Revision: 157199
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157199
Log:
2010-03-03 Paul Thomas
PR fortr
--- Comment #7 from pault at gcc dot gnu dot org 2010-03-03 17:53 ---
Fixed on trunk and yes, thanks for the report!
Cheers
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-04 14:59 ---
Created an attachment (id=20020)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20020&action=view)
Fixes this bug
This does all the right things, except that the array elements cannot yet be
referenced -
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-04 15:00 ---
Created an attachment (id=20021)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20021&action=view)
Fixes this bug
This does all the right things, except that the array elements cannot yet be
referenced -
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-09 11:33 ---
(In reply to comment #3)
>
> Paul, since you were the one who implemented this: Could you me remind me why
> this is needed at all? Shouldn't it be enough to resolve the call as is, i.e.
> just f
--- Comment #5 from pault at gcc dot gnu dot org 2010-03-09 13:09 ---
Created an attachment (id=20056)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20056&action=view)
Fix for the PR
This is just now regtesting. I believe that it is OK, since it works for
gfortran.dg/cla
--- Comment #6 from pault at gcc dot gnu dot org 2010-03-09 13:34 ---
> If all appears to be well with the regtest and I do not hear back from you, I
> will commit the patch to trunk tonight.
Confidence before the fall and all that.
The patch clobbers dynamic_dispatch
--- Comment #7 from pault at gcc dot gnu dot org 2010-03-09 13:46 ---
(In reply to comment #6)
> The patch clobbers dynamic_dispatch_4.f03. Remember it Salvatore?
I should have said that this is for -O1 and higher.
/svn/trunk/gcc/testsuite/gfortran.dg/dynamic_dispatch_4.f03:
--- Comment #10 from pault at gcc dot gnu dot org 2010-03-10 09:29 ---
In preparing a testcase, I foolishly decided to check the original for correct
execution.
The following gives the wrong result:
module m1
type :: t1
contains
procedure :: sizeof
end type
contains
--- Comment #13 from pault at gcc dot gnu dot org 2010-03-12 22:01 ---
Subject: Bug 43291
Author: pault
Date: Fri Mar 12 22:00:52 2010
New Revision: 157411
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157411
Log:
2010-03-12 Paul Thomas
PR fortran/43291
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-12 22:01 ---
Subject: Bug 43326
Author: pault
Date: Fri Mar 12 22:00:52 2010
New Revision: 157411
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157411
Log:
2010-03-12 Paul Thomas
PR fortran/43291
--- Comment #14 from pault at gcc dot gnu dot org 2010-03-12 22:05 ---
Fixed on trunk.
I´ll keep 43326 open for a bit, until I figure out what to do with
fortran-dev.
Thanks for the report.
Cheers
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed
--- Comment #10 from pault at gcc dot gnu dot org 2010-03-14 06:27 ---
Will check this out as soon as I am back at base.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #11 from pault at gcc dot gnu dot org 2010-03-15 17:23 ---
(In reply to comment #10)
> Will check this out as soon as I am back at base.
HJ,
4.4 does not deal with the original problem and so still produces the
unnecessary temporaries.
from the end of internal_pack12.
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-17 21:07 ---
As Daniel has indicated, this has nothing to do with gfortran.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-18 18:00 ---
The following fixes the PR. I have not regtested yet but anticipate that all
will be well.
Index: ../trunk/gcc/fortran/trans-expr.c
===
--- ../trunk
--- Comment #1 from pault at gcc dot gnu dot org 2010-03-18 18:17 ---
This is fixed by the following, which is not yet regtested:
Index: ../trunk/gcc/fortran/resolve.c
===
--- ../trunk/gcc/fortran/resolve.c (revision
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-18 19:09 ---
(In reply to comment #3)
> (In reply to comment #2)
> > The following fixes the PR. I have not regtested yet but anticipate that
> > all
> > will be well.
>
> Looks good. Does is al
--- Comment #5 from pault at gcc dot gnu dot org 2010-03-18 20:12 ---
(In reply to comment #3)
> (In reply to comment #2)
snip
> I suspect that it is similar but not identical. Will look after dinner :-)
>
> Paul
>
This surmise is correct. As soon as the other
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-18 21:24 ---
Subject: Bug 43039
Author: pault
Date: Thu Mar 18 21:23:35 2010
New Revision: 157552
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157552
Log:
2010-03-18 Paul Thomas
PR fortr
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-18 21:24 ---
Subject: Bug 43044
Author: pault
Date: Thu Mar 18 21:23:35 2010
New Revision: 157552
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157552
Log:
2010-03-18 Paul Thomas
PR fortr
--- Comment #6 from pault at gcc dot gnu dot org 2010-03-18 21:24 ---
Subject: Bug 43043
Author: pault
Date: Thu Mar 18 21:23:35 2010
New Revision: 157552
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157552
Log:
2010-03-18 Paul Thomas
PR fortr
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-18 21:28 ---
Fixed on trunk.
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 2010-03-18 21:29 ---
Fixed on trunk.
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-18 21:29 ---
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 2010-03-20 11:29 ---
Problem is slightly different on fortran-dev, as Dominique says in comment #5.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #1 from pault at gcc dot gnu dot org 2010-03-20 14:06 ---
Created an attachment (id=20146)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20146&action=view)
A fix for the PR
This bootstraps and regtests on FC9/x86_64
Paul
--
pault at gcc dot gnu dot org
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-20 16:01 ---
Subject: Bug 43450
Author: pault
Date: Sat Mar 20 16:01:17 2010
New Revision: 157595
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157595
Log:
2010-03-20 Paul Thomas
PR fortr
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-20 16:02 ---
Fixed on trunk
Thanks for the report.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
etween an SSA_NAME and
its symbol with -O3
Product: gcc
Version: 4.5.0
Status: UNCONFIRMED
Keywords: ice-on-valid-code
Severity: major
Priority: P3
Component: fortran
AssignedTo: unassigned at gcc
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-24 11:09 ---
(In reply to comment #1)
> Hm, but the error is about hts_1, not gridws.
>
Does that not merely reflect the order that they are checked in the assignment?
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-24 11:39 ---
(In reply to comment #3)
> No, the checking complains that TREE_TYPE (hts_1) (the type of the
> SSA name) is != TREE_TYPE (hts) (its symbol). That should never happen.
Ah, OK.
Interestingly, when HTS is de
--- Comment #5 from pault at gcc dot gnu dot org 2010-03-25 21:59 ---
I am a bit lost :-(
(gdb) print ssa_name->common
$6 = {base = {code = SSA_NAME, side_effects_flag = 0, constant_flag = 0,
addressable_flag = 0, volatile_flag = 0, readonly_flag = 0,
unsigned_flag
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-27 07:55 ---
This is a bit odd. SIZEOF seems to be properly registered in intrinsic.c:
add_sym_1 ("sizeof", GFC_ISYM_SIZEOF, NO_CLASS, ACTUAL_NO, BT_INTEGER, ii,
GFC_STD_GNU, gfc_check_sizeof,
--- Comment #1 from pault at gcc dot gnu dot org 2010-03-27 17:59 ---
With the function calls interchanged, the problem goes away. ie. with:
write (*,*) shape(this%myarray) /= shape(array) ! SEGFAULT
I can't see from the code yet why this is happening but can confirm the bug.
--- Comment #2 from pault at gcc dot gnu dot org 2010-03-27 18:45 ---
Replacing the gcc_assert with
if (!overriding || !overriding->n.tb)
return FAILURE;
Fixes the problem. This looks like one ICE too far :-)
It might be best, I suppose, to limit the above
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-27 18:47 ---
Confirmed
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
Status
--- Comment #16 from pault at gcc dot gnu dot org 2010-03-27 18:53 ---
What is the status of this one Tobias?
I'll confirm it whilst I'm here :-)
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed
--- Comment #7 from pault at gcc dot gnu dot org 2010-03-27 18:55 ---
Tobias,
Can this be closed now?
Certainly, it can be confirmed!
Cheers
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #13 from pault at gcc dot gnu dot org 2010-03-27 19:11 ---
Can we close this one, guys?
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43146
--- Comment #3 from pault at gcc dot gnu dot org 2010-03-28 15:05 ---
Created an attachment (id=20231)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20231&action=view)
Fix for the PR
Bootstraps and regtests on FC9/x86_64
Paul
--
http://gcc.gnu.org/bugzilla/show_bug
--- Comment #1 from pault at gcc dot gnu dot org 2010-03-28 20:39 ---
(In reply to comment #0)
> My understanding is that thus the following program is invalid:
>
> pure subroutine foo(a, c, d)
> integer, volatile, intent(inout) :: a, c ! OK
> integer, vol
--- Comment #1 from pault at gcc dot gnu dot org 2010-03-28 20:50 ---
Dear Dominiq,
Unfortunately, what you report here is a consequence of the way in which
-fwhole-file has to be implemented. The order changes because of the need to
resolve called procedures and I can well believe
--- Comment #4 from pault at gcc dot gnu dot org 2010-03-31 18:03 ---
(In reply to comment #3)
> Assigning to myself. This is probably due to ftell not taking into account
> fbuf.
>
Janne,
I hate to say it but you are going to have to move pretty fast to beat Richi&#x
--- Comment #25 from pault at gcc dot gnu dot org 2010-04-16 05:59 ---
Created an attachment (id=20391)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20391&action=view)
A provisional fix for this PR
This bootstraps and regtests. I understand why it works but I want to
und
--- Comment #27 from pault at gcc dot gnu dot org 2010-04-18 13:50 ---
(In reply to comment #26)
> (In reply to comment #25)
> > A provisional fix for this PR
>
> Actually, what's the issue here? At rev. 158483 the fortran-dev testsuite runs
> cleanly on x86_64
--- Comment #29 from pault at gcc dot gnu dot org 2010-04-18 17:24 ---
Subject: Bug 42353
Author: pault
Date: Sun Apr 18 17:24:35 2010
New Revision: 158492
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158492
Log:
2010-04-18 Paul Thomas
PR fortr
--- Comment #30 from pault at gcc dot gnu dot org 2010-04-18 17:25 ---
Subject: Bug 42353
Author: pault
Date: Sun Apr 18 17:25:41 2010
New Revision: 158493
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158493
Log:
2010-04-18 Paul Thomas
PR fortr
--- Comment #31 from pault at gcc dot gnu dot org 2010-04-18 17:29 ---
Fixed on fortran-dev.
Thanks for the report, Harald.
Paul
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=42353
--- Comment #32 from pault at gcc dot gnu dot org 2010-04-18 17:29 ---
Hmmm better mark it as fixed
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #7 from pault at gcc dot gnu dot org 2010-04-18 17:33 ---
Created an attachment (id=20410)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20410&action=view)
Fix for the problem
This needs to be regtested but I believe it to be bombproof.
However, I should att
--- Comment #8 from pault at gcc dot gnu dot org 2010-04-18 18:30 ---
(In reply to comment #6)
> (In reply to comment #5)
> > What about pr42274? Is it a duplicate or not?
>
> I don't think so.
>
My patch fixes pr42274 comment #9 but not the main part of it
--- Comment #9 from pault at gcc dot gnu dot org 2010-04-19 12:12 ---
I decided to take a look at this during lunchtime today. The source that I had
to hand is the 20091203 4.5.0 snapshot. To my astonishment, this does not show
the problem. I have had a quick look at the intervening
--- Comment #18 from pault at gcc dot gnu dot org 2010-04-19 18:48 ---
(In reply to comment #16)
I sort of doubt it. The problem arises because mio_symbol crashes in writing
the character length of the procedure symbol:
Breakpoint 1, mio_symbol (sym=0x9d02370)
at ../../fortran-dev
--- Comment #20 from pault at gcc dot gnu dot org 2010-04-19 21:16 ---
Created an attachment (id=20429)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20429&action=view)
A provisional fix for the PR
This needs cleaning up and FAILUREs of the gfc_resolve_expr's need
--- Comment #22 from pault at gcc dot gnu dot org 2010-04-20 05:00 ---
(In reply to comment #21)
>
> Could you explain what the other stuff is needed for? I currently fail to see
> that.
>
Ignore the first bit in resolve.c,
The change to trans-decl.c fixes the second se
--- Comment #23 from pault at gcc dot gnu dot org 2010-04-20 06:19 ---
Created an attachment (id=20433)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20433&action=view)
fix for this PR and PR43266
The attached is what I intend to submit tonight, unless somebody approves it
--- Comment #6 from pault at gcc dot gnu dot org 2010-04-20 18:57 ---
(In reply to comment #5)
> (In reply to comment #4)
> > Technically this PR, fixed on trunk but not on fortran-dev, is now a
> > [fortran-dev Regression]. Could it be marked that way?
>
> Yes.
&g
--- Comment #5 from pault at gcc dot gnu dot org 2010-04-20 19:07 ---
Subject: Bug 43266
Author: pault
Date: Tue Apr 20 19:07:14 2010
New Revision: 158570
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158570
Log:
2010-04-20 Paul Thomas
PR fortr
--- Comment #26 from pault at gcc dot gnu dot org 2010-04-20 19:07 ---
Subject: Bug 43227
Author: pault
Date: Tue Apr 20 19:07:14 2010
New Revision: 158570
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158570
Log:
2010-04-20 Paul Thomas
PR fortr
--- Comment #27 from pault at gcc dot gnu dot org 2010-04-21 04:27 ---
Fixed on trunk - will do 4.5 next week.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from pault at gcc dot gnu dot org 2010-04-21 04:53 ---
I'll do this one next - assigning to self.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |
--- Comment #6 from pault at gcc dot gnu dot org 2010-04-21 05:35 ---
Subject: Bug 43492
Author: pault
Date: Wed Apr 21 05:35:04 2010
New Revision: 158585
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158585
Log:
2010-04-21 Janus Weil
PR fortr
--- Comment #7 from pault at gcc dot gnu dot org 2010-04-21 05:37 ---
Not only does it regtest but I had a few minutes to commit it in your name, as
obvious! Thanks for the fix, Janus, and thanks for the report, Salvatore.
Paul
--
pault at gcc dot gnu dot org changed
--- Comment #9 from pault at gcc dot gnu dot org 2010-04-21 14:29 ---
Created an attachment (id=20453)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20453&action=view)
Version of fix for fortran-dev
This hasn been fully bootstrapped but runs gfortran.dg/dynamic*, proc* an
--- Comment #10 from pault at gcc dot gnu dot org 2010-04-21 16:50 ---
Subject: Bug 43326
Author: pault
Date: Wed Apr 21 16:49:28 2010
New Revision: 158613
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158613
Log:
2010-04-21 Paul Thomas
PR fortr
--- Comment #11 from pault at gcc dot gnu dot org 2010-04-21 16:51 ---
Fixed on fortran-dev.
Thanks, as usual, for the report, Janus, and thanks for the reminder, Dominique
:-)
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #6 from pault at gcc dot gnu dot org 2010-04-22 20:01 ---
(In reply to comment #5)
> struct polar_t * D.1551;
> D.1551 = &b[0];
>
> S.5 = 1;
> while (1)
> {
> if (S.5 > 3) goto L.3;
> b[S.5
--- Comment #7 from pault at gcc dot gnu dot org 2010-04-23 08:12 ---
I posted a fix this morning.
Paul
--
pault at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #8 from pault at gcc dot gnu dot org 2010-04-24 07:44 ---
(In reply to comment #7)
> I posted a fix this morning.
...which gives,
{
struct polar_t D.1625;
D.1625 = b[0];
{
integer(kind=8) S.18;
S.18 = 1;
while (1)
{
if
2001 - 2100 of 3045 matches
Mail list logo
