Am 30.04.2011 17:31, schrieb Jerry DeLisle:
On 04/29/2011 02:45 PM, Tobias Burnus wrote:
Nearly obvious patch. Build and regtested on x86-64-linux.
OK for the trunk?

Thats a oneliner.  OK

Thanks for the review! Unfortunately, I had submitted/committed the wrong test case. (Both test cases are extremely similar, but test something quite different.)

I have now corrected the test suite using the attached patch. (Rev. 173221).

Tobias
Index: gcc/testsuite/gfortran.dg/interface_37.f90
===================================================================
--- gcc/testsuite/gfortran.dg/interface_37.f90	(Revision 173218)
+++ gcc/testsuite/gfortran.dg/interface_37.f90	(Arbeitskopie)
@@ -1,31 +0,0 @@
---- /dev/null
-+++ gcc/testsuite/gfortran.dg/interface_36.f90	2011-04-29 19:10:43.000000000 +0200
-@@ -0,0 +1,28 @@
-+! { dg-do compile }
-+!
-+! PR fortran/48800
-+!
-+! Contributed by Daniel Carrera
-+!
-+     pure function runge_kutta_step(t, r_, dr, h) result(res)
-+         real, intent(in) :: t, r_(:), h
-+         real, dimension(:), allocatable :: k1, k2, k3, k4, res
-+         integer :: N
-+
-+         interface
-+             pure function dr(t, r_)  ! { dg-error "cannot have a deferred shape" }
-+                 real, intent(in) :: t, r_(:)
-+                 real :: dr(:)
-+             end function
-+         end interface
-+
-+         N = size(r_)
-+         allocate(k1(N),k2(N),k3(N),k4(N),res(N))
-+
-+         k1 = dr(t, r_)
-+         k2 = dr(t + h/2, r_ + k1*h/2)
-+         k3 = dr(t + h/2, r_ + k2*h/2)
-+         k4 = dr(t + h  , r_ + k3*h)
-+
-+         res = r_ + (k1 + 2*k2 + 2*k3 + k4) * h/6
-+     end function
Index: gcc/testsuite/gfortran.dg/import9.f90
===================================================================
--- gcc/testsuite/gfortran.dg/import9.f90	(Revision 0)
+++ gcc/testsuite/gfortran.dg/import9.f90	(Revision 0)
@@ -0,0 +1,30 @@
+! { dg-do compile }
+!
+! PR fortran/48821
+!
+! Contributed by Daniel Carrera
+!
+
+contains
+     pure subroutine rk4_vec(t, Y, dY, h)
+         real, intent(inout) :: t, Y(:)
+         real, intent(in) :: h
+         real, dimension(size(Y)) :: k1, k2, k3, k4
+
+         interface
+             pure function dY(t0, y0)
+                 import :: Y
+                 real, intent(in) :: t0, y0(size(Y))
+                 real :: dY(size(y0))
+             end function
+         end interface
+
+         k1 = dY(t, Y)
+         k2 = dY(t + h/2, Y + k1*h/2)
+         k3 = dY(t + h/2, Y + k2*h/2)
+         k4 = dY(t + h  , Y + k3*h)
+
+         Y = Y + (k1 + 2*k2 + 2*k3 + k4) * h/6
+         t = t + h
+     end subroutine
+end
Index: gcc/testsuite/ChangeLog
===================================================================
--- gcc/testsuite/ChangeLog	(Revision 173219)
+++ gcc/testsuite/ChangeLog	(Arbeitskopie)
@@ -1,6 +1,13 @@
 2011-04-30  Tobias Burnus  <bur...@net-b.de>
 
-	PR fortran/48800
+	PR fortran/48821
+	* gfortran.dg/import9.f90: New, proper test.
+	* gfortran.dg/interface_37.f90: Remove bogus
+	test (bogus copy of interface_36.f90).
+
+2011-04-30  Tobias Burnus  <bur...@net-b.de>
+
+	PR fortran/48821
 	* gfortran.dg/interface_37.f90: New.
 
 2011-04-30  Paul Thomas  <pa...@gcc.gnu.org>
Index: gcc/fortran/ChangeLog
===================================================================
--- gcc/fortran/ChangeLog	(Revision 173219)
+++ gcc/fortran/ChangeLog	(Arbeitskopie)
@@ -1,6 +1,6 @@
 2011-04-30  Tobias Burnus  <bur...@net-b.de>
 
-	PR fortran/48800
+	PR fortran/48821
 	* decl.c (gfc_match_import): Don't try to find the
 	symbol if already found.
 

Reply via email to