This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch, Fortran] Fix dependency checking regression for 64-bit


Tobias Burnus wrote:
> The actual patch looks fine, however, the test case looks incomplete:
> No 
> reference to the PR - and it is completely unclear what you are
> testing 
> for - and ICE? Or for dependencies - if the latter, then you miss a 
> check for that! OK if you fix the testcase.

Thanks for the quick review, and for spotting the faulty test case!

Here's the test case as I committed it:

! { do-do compile }
! { dg-options "-Warray-temporaries" }
! PR 45159 - make sure no temporary is created for this.
subroutine foo(a,b,i,j,k,n)
  implicit none
  integer, intent(in) :: i, j, k, n
  real, dimension(n) :: a,b
  a(k:i-1) = a(i:j)
end subroutine foo

	Thomas

Sende          fortran/ChangeLog
Sende          fortran/dependency.c
Sende          testsuite/ChangeLog
Hinzufügen     testsuite/gfortran.dg/dependency_30.f90
Übertrage Daten ....
Revision 162848 übertragen.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]