[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread thomasr at nikhef dot nl 


--- Comment #1 from thomasr at nikhef dot nl  2010-05-14 09:24 ---
Created an attachment (id=20660)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20660action=view)
Test program that reproduces the bug


-- 


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #2 from kargl at gcc dot gnu dot org  2010-05-14 16:23 ---
The problem seems to be triggered by the upper bound value of 0.
If I change it to 1 or -1 in both the main program and the 
subroutine, then the warning is not triggered.


-- 

kargl at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever Confirmed|0   |1
  Known to fail||4.3.5 4.4.3 4.5.1 4.6.0
   Last reconfirmed|-00-00 00:00:00 |2010-05-14 16:23:36
   date||


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #3 from kargl at gcc dot gnu dot org  2010-05-14 16:34 ---
I have a patch that I'm testing.


-- 

kargl at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |kargl at gcc dot gnu dot org
   |dot org |
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-05-14 16:23:36 |2010-05-14 16:34:06
   date||


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #4 from kargl at gcc dot gnu dot org  2010-05-14 18:29 ---
Patch submitted here:

http://gcc.gnu.org/ml/gcc-patches/2010-05/msg01080.html


-- 


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #5 from kargl at gcc dot gnu dot org  2010-05-14 21:02 ---
Subject: Bug 44135

Author: kargl
Date: Fri May 14 21:02:26 2010
New Revision: 159415

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=159415
Log:
2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* gfortran.dg/actual_array_interface_2.f90: New test.

2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* fortran/interface.c (get_sym_storage_size): Use signed instead of
unsigned mpz_get_?i routines.

Added:
trunk/gcc/testsuite/gfortran.dg/actual_array_interface_2.f90
Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/interface.c
trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #6 from kargl at gcc dot gnu dot org  2010-05-14 21:17 ---
Subject: Bug 44135

Author: kargl
Date: Fri May 14 21:17:42 2010
New Revision: 159417

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=159417
Log:
2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* gfortran.dg/actual_array_interface_2.f90: New test.

2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* fortran/interface.c (get_sym_storage_size): Use signed instead of
unsigned mpz_get_?i routines.

Added:
   
branches/gcc-4_5-branch/gcc/testsuite/gfortran.dg/actual_array_interface_2.f90
Modified:
branches/gcc-4_5-branch/gcc/fortran/ChangeLog
branches/gcc-4_5-branch/gcc/fortran/interface.c
branches/gcc-4_5-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #7 from kargl at gcc dot gnu dot org  2010-05-14 22:40 ---
Subject: Bug 44135

Author: kargl
Date: Fri May 14 22:40:01 2010
New Revision: 159419

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=159419
Log:
2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* gfortran.dg/actual_array_interface_2.f90: New test.

2010-05-14  Steven G. Kargl  ka...@gcc.gnu.org

PR fortran/44135
* fortran/interface.c (get_sym_storage_size): Use signed instead of
unsigned mpz_get_?i routines.

Added:
   
branches/gcc-4_4-branch/gcc/testsuite/gfortran.dg/actual_array_interface_2.f90
Modified:
branches/gcc-4_4-branch/gcc/fortran/ChangeLog
branches/gcc-4_4-branch/gcc/fortran/interface.c
branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


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

[Bug fortran/44135] Negative array bounds lead to spurious warning

2010-05-14 Thread kargl at gcc dot gnu dot org 


--- Comment #8 from kargl at gcc dot gnu dot org  2010-05-14 22:49 ---
Fixed on 4.4, 4.5, and trunk.

Thomas, thanks for the bug report.


-- 

kargl at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


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