[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-07-25 Thread tkoenig at gcc dot gnu dot org
--- Comment #11 from tkoenig at gcc dot gnu dot org 2009-07-25 12:27 --- Fixed on trunk, backporting to 4.5 would be non-trivial because trunk uses the array access macros, and 4.4 doesn't. Closing. -- tkoenig at gcc dot gnu dot org changed: What|Removed

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-07-19 Thread tkoenig at gcc dot gnu dot org
--- Comment #10 from tkoenig at gcc dot gnu dot org 2009-07-19 15:07 --- Subject: Bug 36874 Author: tkoenig Date: Sun Jul 19 15:07:21 2009 New Revision: 149792 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=149792 Log: 2009-07-19 Thomas Koenig PR libfortran/34670

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-06-07 Thread tkoenig at gcc dot gnu dot org
--- Comment #9 from tkoenig at gcc dot gnu dot org 2009-06-07 14:41 --- If nobody minds, I'll do this (it's part of PR 34670 anyway). -- tkoenig at gcc dot gnu dot org changed: What|Removed |Added ---

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-06-07 Thread burnus at gcc dot gnu dot org
--- Comment #8 from burnus at gcc dot gnu dot org 2009-06-07 13:59 --- Compile time check is now there (4.5 trunk). Missing is a run-time check. Example: real, dimension(1,0) :: a, b, c integer :: sp(3), i a = 4.0 sp = 1 i = 1 b = eoshift (a,sp(1:i)) ! Invalid end NAG f95

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-06-07 Thread burnus at gcc dot gnu dot org
--- Comment #7 from burnus at gcc dot gnu dot org 2009-06-07 13:46 --- Subject: Bug 36874 Author: burnus Date: Sun Jun 7 13:45:47 2009 New Revision: 148247 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=148247 Log: 2009-06-07 Daniel Franke * check.c (dim_rank_check)

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-04-05 Thread dfranke at gcc dot gnu dot org
--- Comment #6 from dfranke at gcc dot gnu dot org 2009-04-05 20:10 --- Subject: Bug 36874 Author: dfranke Date: Sun Apr 5 20:09:58 2009 New Revision: 145570 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=145570 Log: gcc/fortran/: 2009-04-05 Daniel Franke PR fortran

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-01-24 Thread burnus at gcc dot gnu dot org
--- Comment #5 from burnus at gcc dot gnu dot org 2009-01-24 19:41 --- (In reply to comment #4) > After adding the the missing checks to gfc_check_cshift() and > gfc_check_eoshift() respectively, I get a couple of errors for > gfortran.dg/zero_sized_1.f90 [...] Thanks! The errors look O

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-01-24 Thread dfranke at gcc dot gnu dot org
--- Comment #4 from dfranke at gcc dot gnu dot org 2009-01-24 18:39 --- After adding the the missing checks to gfc_check_cshift() and gfc_check_eoshift() respectively, I get a couple of errors for gfortran.dg/zero_sized_1.f90: $> gfortran-svn -g -Wall -W zero_sized_1.f90 zero_sized_1.f9

[Bug fortran/36874] Add shape checks to cshift/eoshift

2009-01-19 Thread dfranke at gcc dot gnu dot org
-- dfranke at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |dfranke at gcc dot gnu dot |dot org

[Bug fortran/36874] Add shape checks to cshift/eoshift

2008-07-28 Thread burnus at gcc dot gnu dot org
--- Comment #3 from burnus at gcc dot gnu dot org 2008-07-28 11:21 --- > Why a run-time check? Because the shape might not be known at compile time? And a compile-time check because compile-time checks are nicer ;-) > AFAIK, we can always determine the rank at runtime Yes, the RANK shou

[Bug fortran/36874] Add shape checks to cshift/eoshift

2008-07-28 Thread tkoenig at gcc dot gnu dot org
--- Comment #2 from tkoenig at gcc dot gnu dot org 2008-07-28 10:48 --- Why a run-time check? AFAIK, we can always determine the rank at runtime, so unless somebody messes with our array descriptors big time, we should be safe. -- tkoenig at gcc dot gnu dot org changed:

[Bug fortran/36874] Add shape checks to cshift/eoshift

2008-07-19 Thread burnus at gcc dot gnu dot org
--- Comment #1 from burnus at gcc dot gnu dot org 2008-07-19 13:48 --- Note: One should have a run-time and a compile-time check. Example: real, dimension(1,0) :: a, b, c b = eoshift (a,(/1/)) Since dim=1, the shape of the 2nd dimension of A needs to be the same as the first dimens