[Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments

public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed

* [Bug fortran/43111]  New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments
@ 2010-02-18  5:33 pault at gcc dot gnu dot org
  2010-02-18  9:25 ` [Bug fortran/43111] [4.5 Regression] " burnus at gcc dot gnu dot org
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: pault at gcc dot gnu dot org @ 2010-02-18  5:33 UTC (permalink / raw)
  To: gcc-bugs

As reported by Joost VandeVondele, this does not compile correctly since my fix
for PR41113/7

Paul

SUBROUTINE S2(I)
 INTEGER :: I(4)
 write(6,*) I
 IF (ANY(I.NE.(/3,5,7,9/))) CALL ABORT()
END SUBROUTINE S2

MODULE M1
 TYPE T1
  INTEGER, POINTER, DIMENSION(:) :: data
 END TYPE T1
CONTAINS
 SUBROUTINE S1()
   TYPE(T1) :: d
   INTEGER, TARGET, DIMENSION(10) :: scratch=(/(i,i=1,10)/)
   INTEGER :: i=2
   d%data=>scratch(1:9:2)
   write(6,*) d%data(i:)
   CALL S2(d%data(i:))
 END SUBROUTINE S1
END MODULE M1

USE M1
CALL S1
END


-- 
           Summary: [4.4, 4.5 regression] No temporary 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 org
        ReportedBy: pault at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug fortran/43111] [4.5 Regression] No temporary produced for array pointer actual arguments
  2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
@ 2010-02-18  9:25 ` burnus at gcc dot gnu dot org
  2010-02-18 10:26 ` pault at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: burnus at gcc dot gnu dot org @ 2010-02-18  9:25 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from burnus at gcc dot gnu dot org  2010-02-18 09:25 -------
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 committed to the trunk. I just checked and the test case of comment
#0 still works with a 6h old gcc-4_4-branch. -> Thus I mark this as 4.5 only.

 * * *

By the way, the Fortran 2008 standard lists in

   5.3.7 CONTIGUOUS attribute
   6.5.4 Simply contiguous array designators

some things which are/aren't contiguous in Fortran; maybe one should browse
that list just to get some more ideas (both for optimization and for avoiding
over-optimization). See: ftp://ftp.nag.co.uk/sc22wg5/N1751-N1800/N1791.pdf


-- 

burnus at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Priority|P3                          |P4
   Last reconfirmed|0000-00-00 00:00:00         |2010-02-18 09:25:19
               date|                            |
            Summary|[4.4, 4.5 regression] No    |[4.5 Regression] No
                   |temporary produced for array|temporary produced for array
                   |pointer actual arguments    |pointer actual arguments
   Target Milestone|---                         |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug fortran/43111] [4.5 Regression] No temporary produced for array pointer actual arguments
  2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
  2010-02-18  9:25 ` [Bug fortran/43111] [4.5 Regression] " burnus at gcc dot gnu dot org
@ 2010-02-18 10:26 ` pault at gcc dot gnu dot org
  2010-02-20 12:47 ` pault at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: pault at gcc dot gnu dot org @ 2010-02-18 10:26 UTC (permalink / raw)
  To: gcc-bugs



------- 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 committed to the trunk. I just checked and the test case of comment
> #0 still works with a 6h old gcc-4_4-branch. -> Thus I mark this as 4.5 only.

My mistake - OK!

Paul


-- 

pault at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2010-02-18 09:25:19         |2010-02-18 10:26:03
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug fortran/43111] [4.5 Regression] No temporary produced for array pointer actual arguments
  2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
  2010-02-18  9:25 ` [Bug fortran/43111] [4.5 Regression] " burnus at gcc dot gnu dot org
  2010-02-18 10:26 ` pault at gcc dot gnu dot org
@ 2010-02-20 12:47 ` pault at gcc dot gnu dot org
  2010-02-20 12:48 ` pault at gcc dot gnu dot org
  2010-02-23 17:06 ` hjl at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: pault at gcc dot gnu dot org @ 2010-02-20 12:47 UTC (permalink / raw)
  To: gcc-bugs



------- 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  <pault@gcc.gnu.org>

        PR fortran/36932
        PR fortran/36933
        PR fortran/43072
        PR fortran/43111
        * dependency.c (gfc_check_argument_var_dependency): Use enum
        value instead of arithmetic vaue for 'elemental'.
        (check_data_pointer_types): New function.
        (gfc_check_dependency): Call check_data_pointer_types.
        * trans-array.h : Change fourth argument of
        gfc_conv_array_parameter to boolean.
        * trans-array.c (gfc_conv_array_parameter): A contiguous array
        can be a dummy but it must not be assumed shape or deferred.
        Change fourth argument to boolean. Array constructor exprs will
        always be contiguous and do not need packing and unpacking.
        * trans-expr.c (gfc_conv_procedure_call): Clean up some white
        space and change fourth argument of gfc_conv_array_parameter
        to boolean.
        (gfc_trans_arrayfunc_assign): Change fourth argument of
        gfc_conv_array_parameter to boolean.
        * trans-io.c (gfc_convert_array_to_string): The same.
        * trans-intrinsic.c (gfc_conv_intrinsic_loc): The same.

2010-02-20  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/36932
        PR fortran/36933
        * gfortran.dg/dependency_26.f90: New test.

        PR fortran/43072
        * gfortran.dg/internal_pack_7.f90: New test.

        PR fortran/43111
        * gfortran.dg/internal_pack_8.f90: New test.


Added:
    trunk/gcc/testsuite/gfortran.dg/dependency_26.f90
    trunk/gcc/testsuite/gfortran.dg/internal_pack_7.f90
    trunk/gcc/testsuite/gfortran.dg/internal_pack_8.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/dependency.c
    trunk/gcc/fortran/trans-array.c
    trunk/gcc/fortran/trans-array.h
    trunk/gcc/fortran/trans-expr.c
    trunk/gcc/fortran/trans-intrinsic.c
    trunk/gcc/fortran/trans-io.c
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug fortran/43111] [4.5 Regression] No temporary produced for array pointer actual arguments
  2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
                   ` (2 preceding siblings ...)
  2010-02-20 12:47 ` pault at gcc dot gnu dot org
@ 2010-02-20 12:48 ` pault at gcc dot gnu dot org
  2010-02-23 17:06 ` hjl at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: pault at gcc dot gnu dot org @ 2010-02-20 12:48 UTC (permalink / raw)
  To: gcc-bugs



------- 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|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug fortran/43111] [4.5 Regression] No temporary produced for array pointer actual arguments
  2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
                   ` (3 preceding siblings ...)
  2010-02-20 12:48 ` pault at gcc dot gnu dot org
@ 2010-02-23 17:06 ` hjl at gcc dot gnu dot org
  4 siblings, 0 replies; 6+ messages in thread
From: hjl at gcc dot gnu dot org @ 2010-02-23 17:06 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from hjl at gcc dot gnu dot org  2010-02-23 17:04 -------
Subject: Bug 43111

Author: hjl
Date: Tue Feb 23 17:02:26 2010
New Revision: 157010

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=157010
Log:
Backport testcases from mainline.

2010-02-23  H.J. Lu  <hongjiu.lu@intel.com>

        Backport from mainline:
        2010-02-22  Richard Guenther  <rguenther@suse.de>

        PR tree-optimization/42749
        * gcc.c-torture/compile/pr42749.c: New testcase.

        2010-02-21  Dodji Seketeli  <dodji@redhat.com>

        PR c++/42824
        * g++.dg/template/memclass4.C: New test.

        2010-02-20  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/43111
        * gfortran.dg/internal_pack_8.f90: New test.

        2010-02-18  Jason Merrill  <jason@redhat.com>

        PR c++/43109
        * g++.dg/parse/namespace12.C: New.

        2010-02-18  Martin Jambor  <mjambor@suse.cz>

        PR tree-optimization/43066
        * gcc.c-torture/compile/pr43066.c: New test.

        2010-02-17  Jason Merrill  <jason@redhat.com>

        PR c++/43069
        * g++.dg/parse/namespace11.C: New.

        PR c++/43093
        * g++.dg/ext/attrib37.C: New.

        PR c++/43079
        * g++.dg/template/ptrmem20.C: New.

        2010-02-16  Jason Merrill  <jason@redhat.com>

        PR c++/43031
        * g++.dg/ext/attrib36.C: New.

        2010-02-15  Richard Guenther  <rguenther@suse.de>

        PR middle-end/43068
        * g++.dg/torture/pr43068.C: New testcase.

        2010-02-11  Richard Guenther  <rguenther@suse.de>

        PR tree-optimization/42998
        * gcc.c-torture/compile/pr42998.c: New testcase.

        2010-02-10  Richard Guenther  <rguenther@suse.de>

        PR tree-optimization/43017
        * gcc.dg/torture/pr43017.c: New testcase.

        2010-02-10  Richard Guenther  <rguenther@suse.de>

        PR c/43007
        * gcc.c-torture/execute/20100209-1.c: New testcase.
        * gcc.dg/fold-div-3.c: Likewise.

        2010-02-09  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

        PR fortran/42999
        * gfortran.dg/array_constructor_35.f90: New test.

        2010-02-09  Richard Guenther  <rguenther@suse.de>

        PR tree-optimization/43008
        * gcc.c-torture/execute/pr43008.c: New testcase.

        2010-02-09  Richard Guenther  <rguenther@suse.de>

        PR tree-optimization/43000
        * gcc.dg/torture/pr43000.c: New testcase.
        * gcc.dg/torture/pr43002.c: Likewise.

        2010-02-06  Jerry DeLisle  <jvdelisle@gcc.gnu.org>

        PR libfortran/42742
        * gfortran.dg/fmt_cache_2.f: New test.

        2010-02-03  Jason Merrill  <jason@redhat.com>

        PR c++/42870
        * g++.dg/ext/dllexport3.C: New.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/ext/attrib36.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/ext/attrib36.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/ext/attrib37.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/ext/attrib37.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/ext/dllexport3.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/ext/dllexport3.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/parse/namespace11.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/parse/namespace11.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/parse/namespace12.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/parse/namespace12.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/template/memclass4.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/template/memclass4.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/template/ptrmem20.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/template/ptrmem20.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/torture/pr43068.C
      - copied unchanged from r157009,
trunk/gcc/testsuite/g++.dg/torture/pr43068.C
    branches/gcc-4_4-branch/gcc/testsuite/gcc.c-torture/compile/pr42749.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.c-torture/compile/pr42749.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.c-torture/compile/pr42998.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.c-torture/compile/pr42998.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.c-torture/compile/pr43066.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.c-torture/compile/pr43066.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.c-torture/execute/20100209-1.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.c-torture/execute/20100209-1.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.c-torture/execute/pr43008.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.c-torture/execute/pr43008.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/fold-div-3.c
      - copied unchanged from r157009, trunk/gcc/testsuite/gcc.dg/fold-div-3.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/torture/pr43000.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.dg/torture/pr43000.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/torture/pr43002.c
      - copied unchanged from r157009,
trunk/gcc/testsuite/gcc.dg/torture/pr43002.c
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/array_constructor_35.f90
      - copied unchanged from r157009,
trunk/gcc/testsuite/gfortran.dg/array_constructor_35.f90
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/fmt_cache_2.f
      - copied unchanged from r157009,
trunk/gcc/testsuite/gfortran.dg/fmt_cache_2.f
    branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/internal_pack_8.f90
      - copied unchanged from r157009,
trunk/gcc/testsuite/gfortran.dg/internal_pack_8.f90
Modified:
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43111


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2010-02-23 17:06 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-02-18  5:33 [Bug fortran/43111] New: [4.4, 4.5 regression] No temporary produced for array pointer actual arguments pault at gcc dot gnu dot org
2010-02-18  9:25 ` [Bug fortran/43111] [4.5 Regression] " burnus at gcc dot gnu dot org
2010-02-18 10:26 ` pault at gcc dot gnu dot org
2010-02-20 12:47 ` pault at gcc dot gnu dot org
2010-02-20 12:48 ` pault at gcc dot gnu dot org
2010-02-23 17:06 ` hjl at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).