[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-02 Thread pault at gcc dot gnu dot org
--- 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 ---

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-02 Thread pault at gcc dot gnu dot org
--- 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 fortran/43180 * tr

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-02 Thread burnus at gcc dot gnu dot org
--- Comment #9 from burnus at gcc dot gnu dot org 2010-03-02 10:01 --- (In reply to comment #8) > Completely untested patch: Still untested, but I think it might lead to wrong code with either allocatable(1)%pointer(1:2) ! Potentially non-contiguous or pointer%allocatable(1:2) ! Ca

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-02 Thread burnus at gcc dot gnu dot org
--- Comment #8 from burnus at gcc dot gnu dot org 2010-03-02 09:24 --- Completely untested patch: --- trans-array.c (revision 157160) +++ trans-array.c (working copy) @@ -,5 +,5 @@ gfc_conv_array_parameter (gfc_se * se, g no_pack = ((sym && sym->as

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-02 Thread burnus at gcc dot gnu dot org
--- Comment #7 from burnus at gcc dot gnu dot org 2010-03-02 09:16 --- > I do not see the temporaries with [...] 4.5.0 20100214 > but I see this with [...]4.5.0 20100227 I think the "regression" is due to: http://gcc.gnu.org/viewcvs?view=revision&revision=156926 Namely

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-03-01 Thread rguenth at gcc dot gnu dot org
--- Comment #6 from rguenth at gcc dot gnu dot org 2010-03-01 11:54 --- Not a regression (we track regressions only against releases). I see both temporaries also on the 4.4 branch. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added --

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-02-26 Thread pault at gcc dot gnu dot org
--- 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. Cheers Paul --

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-02-25 Thread pault at gcc dot gnu dot org
--- 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 changes which fix this issue?

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-02-25 Thread burnus at gcc dot gnu dot org
--- Comment #2 from burnus at gcc dot gnu dot org 2010-02-25 12:56 --- (In reply to comment #1) > This does not occur on i386. I will try on a x86_64 tonight. The warning for -Warray-temporaries is shown here for both -m32 and -m64 -- and I also see "_gfortran_internal_pack" in the -fd

[Bug fortran/43173] Unnecessary array temporary: Passing contiguous array as actual argument

2010-02-25 Thread pault at gcc dot gnu dot org
--- 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