public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/28817]  New: [gfortran] problems with -Wunused
@ 2006-08-23  8:28 martin at mpa-garching dot mpg dot de
  2006-09-14  9:09 ` [Bug fortran/28817] " paul dot richard dot thomas at cea dot fr
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: martin at mpa-garching dot mpg dot de @ 2006-08-23  8:28 UTC (permalink / raw)
  To: gcc-bugs

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 1237 bytes --]

The following code causes current mainline gfortran to produce warnings
about unused variables that are somewhat confused.

module test
contains

subroutine sub1 (arg1)
  integer arg1
  integer var1
end subroutine

subroutine sub2 (arg2)
  integer arg2
  integer var2
  call something(arg2)
end subroutine

end module test

martin@linux:~/tmp> gfortran -c -Wunused test.f90
test.f90: In function ‘sub2’:
test.f90:4: warning: unused variable ‘var2’
test.f90: In function ‘sub1’:
test.f90:12: warning: unused variable ‘arg1’
test.f90:12: warning: unused variable ‘var1’

The diagnostics appear in the wrong order, and the line numbers are confused.
This is most likely related to (or identical with) PR21918.


-- 
           Summary: [gfortran] problems with -Wunused
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: martin at mpa-garching dot mpg dot de
 GCC build triplet: x86_64-unknown-linux-gnu
  GCC host triplet: x86_64-unknown-linux-gnu
GCC target triplet: x86_64-unknown-linux-gnu


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
@ 2006-09-14  9:09 ` paul dot richard dot thomas at cea dot fr
  2006-09-16 19:58 ` pault at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: paul dot richard dot thomas at cea dot fr @ 2006-09-14  9:09 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from paul dot richard dot thomas at cea dot fr  2006-09-14 09:09 -------
> This is most likely related to (or identical with) PR21918.

Martin, that is spot on.  In fact, Tobi's patch works fine; albeit a bit more
verbosely.  Also, the locii need to be indirectly referenced, otherwise a
segfault occurs.

Paul


-- 


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
  2006-09-14  9:09 ` [Bug fortran/28817] " paul dot richard dot thomas at cea dot fr
@ 2006-09-16 19:58 ` pault at gcc dot gnu dot org
  2006-09-17 12:45 ` patchapp at dberlin dot org
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-16 19:58 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from pault at gcc dot gnu dot org  2006-09-16 19:58 -------
I propose to submit Tobi's patch

Paul


-- 

pault at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pault at gcc dot gnu dot org
                   |dot org                     |
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-09-16 19:58:03
               date|                            |


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
  2006-09-14  9:09 ` [Bug fortran/28817] " paul dot richard dot thomas at cea dot fr
  2006-09-16 19:58 ` pault at gcc dot gnu dot org
@ 2006-09-17 12:45 ` patchapp at dberlin dot org
  2006-09-18  6:25 ` pault at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: patchapp at dberlin dot org @ 2006-09-17 12:45 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from patchapp at dberlin dot org  2006-09-17 12:45 -------
Subject: Bug number PR28817

A patch for this bug has been added to the patch tracker.
The mailing list url for the patch is
http://gcc.gnu.org/ml/gcc-patches/2006-09/msg00674.html


-- 


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
                   ` (2 preceding siblings ...)
  2006-09-17 12:45 ` patchapp at dberlin dot org
@ 2006-09-18  6:25 ` pault at gcc dot gnu dot org
  2006-09-18  6:47 ` pault at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-18  6:25 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from pault at gcc dot gnu dot org  2006-09-18 06:25 -------
Subject: Bug 28817

Author: pault
Date: Mon Sep 18 06:24:54 2006
New Revision: 117014

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117014
Log:
2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * iresolve.c (resolve_spread): Build shape for result if the
        source shape is available and dim and ncopies are constants.

        PR fortran/28817
        PR fortran/21918
        * trans-decl.c (generate_local_decl): Change from 'warning' to
        'gfc_warning' to have line numbers correctly reported.

2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * gfortran.dg/spread_shape_1.f90: New test.

Added:
    trunk/gcc/testsuite/gfortran.dg/spread_shape_1.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/iresolve.c
    trunk/gcc/fortran/trans-decl.c
    trunk/gcc/testsuite/ChangeLog


-- 


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
                   ` (3 preceding siblings ...)
  2006-09-18  6:25 ` pault at gcc dot gnu dot org
@ 2006-09-18  6:47 ` pault at gcc dot gnu dot org
  2006-09-18  6:55 ` pault at gcc dot gnu dot org
  2006-09-18 22:02 ` pault at gcc dot gnu dot org
  6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-18  6:47 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from pault at gcc dot gnu dot org  2006-09-18 06:46 -------
Subject: Bug 28817

Author: pault
Date: Mon Sep 18 06:46:36 2006
New Revision: 117015

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117015
Log:
2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * iresolve.c (resolve_spread): Build shape for result if the
        source shape is available and dim and ncopies are constants.

        PR fortran/28817
        PR fortran/21918
        * trans-decl.c (generate_local_decl): Change from 'warning' to
        'gfc_warning' to have line numbers correctly reported.

2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * gfortran.dg/spread_shape_1.f90: New test.

Added:
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/spread_shape_1.f90
Modified:
    branches/gcc-4_1-branch/gcc/fortran/ChangeLog
    branches/gcc-4_1-branch/gcc/fortran/iresolve.c
    branches/gcc-4_1-branch/gcc/fortran/trans-decl.c
    branches/gcc-4_1-branch/gcc/testsuite/ChangeLog


-- 


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


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
                   ` (4 preceding siblings ...)
  2006-09-18  6:47 ` pault at gcc dot gnu dot org
@ 2006-09-18  6:55 ` pault at gcc dot gnu dot org
  2006-09-18 22:02 ` pault at gcc dot gnu dot org
  6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-18  6:55 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from pault at gcc dot gnu dot org  2006-09-18 06:55 -------
Fixed on trunk and 4.1

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=28817


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

* [Bug fortran/28817] [gfortran] problems with -Wunused
  2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
                   ` (5 preceding siblings ...)
  2006-09-18  6:55 ` pault at gcc dot gnu dot org
@ 2006-09-18 22:02 ` pault at gcc dot gnu dot org
  6 siblings, 0 replies; 8+ messages in thread
From: pault at gcc dot gnu dot org @ 2006-09-18 22:02 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from pault at gcc dot gnu dot org  2006-09-18 22:02 -------
Subject: Bug 28817

Author: pault
Date: Mon Sep 18 22:02:24 2006
New Revision: 117038

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=117038
Log:
2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * iresolve.c (resolve_spread): Build shape for result if the
        source shape is available and dim and ncopies are constants.

        PR fortran/28817
        PR fortran/21918
        * trans-decl.c (generate_local_decl): Change from 'warning' to
        'gfc_warning' to have line numbers correctly reported.

2006-09-18  Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/29060
        * gfortran.dg/spread_shape_1.f90: New test.

Modified:
    branches/gcc-4_1-branch/gcc/fortran/ChangeLog
    branches/gcc-4_1-branch/gcc/testsuite/ChangeLog
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/spread_shape_1.f90


-- 


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


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

end of thread, other threads:[~2006-09-18 22:02 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-08-23  8:28 [Bug fortran/28817] New: [gfortran] problems with -Wunused martin at mpa-garching dot mpg dot de
2006-09-14  9:09 ` [Bug fortran/28817] " paul dot richard dot thomas at cea dot fr
2006-09-16 19:58 ` pault at gcc dot gnu dot org
2006-09-17 12:45 ` patchapp at dberlin dot org
2006-09-18  6:25 ` pault at gcc dot gnu dot org
2006-09-18  6:47 ` pault at gcc dot gnu dot org
2006-09-18  6:55 ` pault at gcc dot gnu dot org
2006-09-18 22:02 ` pault 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).