http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
Dominique d'Humieres changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
Mikael Morin changed:
What|Removed |Added
CC||mikael at gcc dot gnu.org
--- Comment #28
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
--- Comment #27 from Thomas Koenig 2010-10-16
16:06:16 UTC ---
Author: tkoenig
Date: Sat Oct 16 16:06:07 2010
New Revision: 165559
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=165559
Log:
2010-10-16 Thomas Koenig
PR fortran/2016
--- Comment #26 from burnus at gcc dot gnu dot org 2010-09-16 21:30 ---
Subject: Bug 43665
Author: burnus
Date: Thu Sep 16 21:30:05 2010
New Revision: 164348
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=164348
Log:
2010-09-16 Tobias Burnus
PR fortran/43665
--- Comment #25 from dominiq at lps dot ens dot fr 2010-09-12 10:13 ---
> - create_fn_spec (sym, type);
> + type = create_fn_spec (sym, type);
>
> as in the original patch.
With this change the test succeeds.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
--- Comment #24 from burnus at gcc dot gnu dot org 2010-09-12 09:32 ---
(In reply to comment #23)
> I have applied the patch in comment #21 without regression, but the test case
> from attachment 21265 [edit] fails:
- create_fn_spec (sym, type);
+ type = create_fn_spec (sym, type);
a
--- Comment #23 from dominiq at lps dot ens dot fr 2010-09-11 15:12 ---
I have applied the patch in comment #21 without regression, but the test case
from attachment 21265 fails:
FAIL: gfortran.dg/intent_optimize_1.f90 -O scan-tree-dump-times optimized
"does_not_exist" 0
--
http:
--- Comment #22 from burnus at gcc dot gnu dot org 2010-09-10 12:12 ---
(In reply to comment #21)
> Created an attachment (id=21765)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21765&action=view) [edit]
> Updated patch
Note: I forgot to include the test case from attachment 2126
--- Comment #21 from burnus at gcc dot gnu dot org 2010-09-10 12:09 ---
Created an attachment (id=21765)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21765&action=view)
Updated patch
Updated patch to fix review issues and the Cray patch issue. I won't be able to
work on this (i.e
--- Comment #19 from burnus at gcc dot gnu dot org 2010-09-08 06:25 ---
Reviewed patch ("OK") available at
http://gcc.gnu.org/ml/fortran/2010-09/msg00198.html
however, it causes regressions as some of the intrinsics (in intrinsic.c) have
the wrong intents - which causes wrong code (too
--- Comment #18 from jamborm at gcc dot gnu dot org 2010-09-07 17:01
---
Subject: Bug 43665
Author: jamborm
Date: Tue Sep 7 17:00:44 2010
New Revision: 163960
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=163960
Log:
2010-09-07 Martin Jambor
PR fortran/43665
--- Comment #17 from burnus at gcc dot gnu dot org 2010-09-06 18:43 ---
(In reply to comment #16)
> This patch makes IPA-CP to refrain from modifying a function when it
> sees a any type attributes.
In a way that's unfortunate: Both "fn attr" and argument removal are
optimization option
--- Comment #16 from jamborm at gcc dot gnu dot org 2010-09-06 18:25
---
Created an attachment (id=21714)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21714&action=view)
Patch to address IPA-CP parameter removal issues
This patch makes IPA-CP to refrain from modifying a function
--- Comment #15 from jamborm at gcc dot gnu dot org 2010-09-06 14:12
---
I tried compiling the testcase from comment #8 and it did not fail for
me either on i686-linux ox x86_64-linux. Can you please check that it
still fails for you?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id
--- Comment #14 from burnus at gcc dot gnu dot org 2010-07-22 15:36 ---
(In reply to comment #13)
> (In reply to comment #12)
> IPA-CP can do that for quite some time please try with -fno-ipa-cp.
As expected: It works with -fno-ipa-cp.
> (I don't have a trunk built with enabled fortra
--- Comment #13 from jamborm at gcc dot gnu dot org 2010-07-21 08:27
---
(In reply to comment #12)
> So I wonder what code removes the arguments then.
>
IPA-CP can do that for quite some time please try with -fno-ipa-cp.
(I don't have a trunk built with enabled fortran at hand and I
--- Comment #12 from rguenther at suse dot de 2010-07-21 08:09 ---
Subject: Re: INTENT(IN) etc. optimization of calls:
function annotations for noclobber/noescape arguments
On Tue, 20 Jul 2010, burnus at gcc dot gnu dot org wrote:
> --- Comment #11 from burnus at gcc dot gnu dot
--- Comment #11 from burnus at gcc dot gnu dot org 2010-07-20 16:12 ---
(In reply to comment #10)
> No, this problem was present for ipa-sra, but I thought it was fixed.
> Does -fno-ipa-sra help?
No, it doesn't. But with that option, the number of arguments still reduces
from 4 to 2 (be
--- Comment #10 from rguenth at gcc dot gnu dot org 2010-07-20 16:01
---
No, this problem was present for ipa-sra, but I thought it was fixed.
Does -fno-ipa-sra help? Martin?
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #9 from burnus at gcc dot gnu dot org 2010-07-20 15:58 ---
The procedure 'checkoptional' gets '.rrrw' set as the fn spec.
With -O1, the if after the first "checkoptional" gets properly optimized away
(cf. comment 7 and original test case).
With -O2, the number of arguments
--- Comment #8 from burnus at gcc dot gnu dot org 2010-07-20 15:26 ---
Reduced test case (of allocatable_scalar_4.f90) - fails with "ERROR STOP 1":
program test
implicit none
integer, allocatable :: b
allocate(b)
b = 7482
call checkOptional(.false.,.true., 7482)
call checkOp
--- Comment #7 from burnus at gcc dot gnu dot org 2010-07-20 15:06 ---
Created an attachment (id=21265)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21265&action=view)
Draft patch for external/user procs with INTENT (mostly OK, but breaks existing
test)
The attached patch mostly
--- Comment #6 from burnus at gcc dot gnu dot org 2010-07-13 17:32 ---
Main library calls are done.
TODO:
- Intrinsic calls such as "call ctime()" which do not have a function
declaration
- Nonintrinsic functions with non-pointer INTENT(IN/OUT); handle also
(non)clobber [target/pointer
23 matches
Mail list logo