http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
Dominique d'Humieres dominiq at lps dot ens.fr changed:
What|Removed |Added
Status|NEW |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
Mikael Morin mikael at gcc dot gnu.org changed:
What|Removed |Added
CC||mikael at gcc dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43665
--- Comment #27 from Thomas Koenig tkoenig at gcc dot gnu.org 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=gccview=revrev=165559
Log:
2010-10-16 Thomas
--- 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=gccview=revrev=164348
Log:
2010-09-16 Tobias Burnus bur...@net-b.de
PR
--- 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);
as
--- 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 #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
--
--- 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=21765action=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 #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=21765action=view) [edit]
Updated patch
Note: I forgot to include the test case from attachment 21265
--- 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=gccview=revrev=163960
Log:
2010-09-07 Martin Jambor mjam...@suse.cz
PR
--- 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?
--
--- 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=21714action=view)
Patch to address IPA-CP parameter removal issues
This patch makes IPA-CP to refrain from modifying a function
--- 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 options.
--- 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 fortran
--- 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 #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 #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=21265action=view)
Draft patch for external/user procs with INTENT (mostly OK, but breaks existing
test)
The attached patch mostly
--- 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
--- 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 #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 #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
--- 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
