public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
@ 2006-02-08 21:51 ` fxcoudert at gcc dot gnu dot org
  2006-02-08 21:59 ` tobi at gcc dot gnu dot org
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: fxcoudert at gcc dot gnu dot org @ 2006-02-08 21:51 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from fxcoudert at gcc dot gnu dot org  2006-02-08 21:51 -------
(In reply to comment #4)
> I have updated my patch, copyright assignment is in the works. No need for
> anyone to work on this.

Hum, Tobias, any news on that one? :)


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|2005-12-15 05:02:16         |2006-02-08 21:51:37
               date|                            |


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
  2006-02-08 21:51 ` [Bug fortran/14771] frontend doesn't record parentheses fxcoudert at gcc dot gnu dot org
@ 2006-02-08 21:59 ` tobi at gcc dot gnu dot org
  2006-02-09 15:24 ` tobi at gcc dot gnu dot org
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-08 21:59 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #8 from tobi at gcc dot gnu dot org  2006-02-08 21:59 -------
I'll try to get to it in the next few days, it shouldn't be too hard.


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  BugsThisDependsOn|20894                       |
OtherBugsDependingO|                            |20894
              nThis|                            |


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
  2006-02-08 21:51 ` [Bug fortran/14771] frontend doesn't record parentheses fxcoudert at gcc dot gnu dot org
  2006-02-08 21:59 ` tobi at gcc dot gnu dot org
@ 2006-02-09 15:24 ` tobi at gcc dot gnu dot org
  2006-02-09 15:25 ` tobi at gcc dot gnu dot org
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-09 15:24 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #9 from tobi at gcc dot gnu dot org  2006-02-09 15:24 -------
For the record, Walt Brainerd's testcase from #6 is as follows:
      PROGRAM fc107

! Submitted by Walt Brainerd, The Fortran Company
! GNU Fortran 95 (GCC 4.1.0 20050322 (experimental))
! Windows XP

! Output should be 3

! Output is 999

      INTEGER I
      INTEGER TRYME
      I = 3
      print *, tryme((i),i)
      END

      INTEGER FUNCTION TRYME(RTNME,HITME)
      INTEGER RTNME,HITME
      HITME = 999
      TRYME = RTNME
      END

This together with PR20894 and PR25048 is fixed by a patch I'm about to send,
once I've finished ChangeLogs and testcases.


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |tobi at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-02-08 21:51:37         |2006-02-09 15:24:09
               date|                            |


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2006-02-09 15:24 ` tobi at gcc dot gnu dot org
@ 2006-02-09 15:25 ` tobi at gcc dot gnu dot org
  2006-02-09 15:55 ` tobi at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-09 15:25 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #10 from tobi at gcc dot gnu dot org  2006-02-09 15:25 -------
*** Bug 20894 has been marked as a duplicate of this bug. ***


-- 


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2006-02-09 15:25 ` tobi at gcc dot gnu dot org
@ 2006-02-09 15:55 ` tobi at gcc dot gnu dot org
  2006-02-09 16:00 ` tobi at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-09 15:55 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #11 from tobi at gcc dot gnu dot org  2006-02-09 15:55 -------
Adding Walt Brainerd, as requested in
<http://gcc.gnu.org/ml/fortran/2005-04/msg00344.html>.

Patch posted here: <http://gcc.gnu.org/ml/fortran/2006-02/msg00172.html>


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |walt at fortran dot com


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2006-02-09 15:55 ` tobi at gcc dot gnu dot org
@ 2006-02-09 16:00 ` tobi at gcc dot gnu dot org
  2006-02-10  0:10 ` tobi at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-09 16:00 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #12 from tobi at gcc dot gnu dot org  2006-02-09 16:00 -------
*** Bug 25048 has been marked as a duplicate of this bug. ***


-- 


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (5 preceding siblings ...)
  2006-02-09 16:00 ` tobi at gcc dot gnu dot org
@ 2006-02-10  0:10 ` tobi at gcc dot gnu dot org
  2006-02-10 18:08 ` tobi at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-10  0:10 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #13 from tobi at gcc dot gnu dot org  2006-02-10 00:10 -------
Subject: Bug 14771

Author: tobi
Date: Fri Feb 10 00:10:47 2006
New Revision: 110819

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110819
Log:
fortran/
2006-02-09  Tobias Schlueter  <tobias.schlueter@physik.uni-muenchen.de>

        PR fortran/14771
        * gfortran.h (gfc_intrinsic_op): Add INTRINSIC_PARENTHESES.
        * dump-parse-tree (gfc_show_expr): Handle INTRINSIC_PARENTHESES.
        * expr.c (simplify_intrinsic_op): Treat INTRINSIC_PARENTHESES as
        if it were INTRINSIC_UPLUS.
        * resolve.c (resolve_operator): Handle INTRINSIC_PARENTHESES.
        * match.c (intrinsic_operators): Add INTRINSIC_PARENTHESES.
        * matchexp.c (match_primary): Record parentheses surrounding
        numeric expressions.
        * module.c (intrinsics): Add INTRINSIC_PARENTHESES for module
        dumping.
        * trans-expr.c (gfc_conv_expr_op): Handle INTRINSIC_PARENTHESES.

testsuite/
2006-02-09  Tobias Schlueter  <tobias.schlueter@physik.uni-muenchen.de>
        Paul Thomas  <pault@gcc.gnu.org>

        PR fortran/14771
        * gfortran.dg/parens_1.f90: New.
        * gfortran.dg/parens_2.f90: New.
        * gfortran.dg/parens_3.f90: New.



Added:
    trunk/gcc/testsuite/gfortran.dg/parens_1.f90
    trunk/gcc/testsuite/gfortran.dg/parens_2.f90
    trunk/gcc/testsuite/gfortran.dg/parens_3.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/dump-parse-tree.c
    trunk/gcc/fortran/expr.c
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/match.c
    trunk/gcc/fortran/matchexp.c
    trunk/gcc/fortran/module.c
    trunk/gcc/fortran/resolve.c
    trunk/gcc/fortran/trans-expr.c
    trunk/gcc/testsuite/ChangeLog


-- 


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (6 preceding siblings ...)
  2006-02-10  0:10 ` tobi at gcc dot gnu dot org
@ 2006-02-10 18:08 ` tobi at gcc dot gnu dot org
  2006-02-12 18:31 ` tobi at gcc dot gnu dot org
  2006-02-12 18:32 ` tobi at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-10 18:08 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #14 from tobi at gcc dot gnu dot org  2006-02-10 18:08 -------
Subject: Bug 14771

Author: tobi
Date: Fri Feb 10 18:08:27 2006
New Revision: 110840

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110840
Log:
fortran/
        PR fortran/14771
        * arith.c (eval_intrinsic): Accept INTRINSIC_PARENTHESES.
        * expr.c (check_intrinsic_op): Likewise.
        * module.c (mio_expr): Likewise.
testsuite/
        PR fortran/14771
        * gfortran.dg/parens_4.f90: New.
        * gfortran.dg/parens_5.f90: New.


Added:
    trunk/gcc/testsuite/gfortran.dg/parens_4.f90
    trunk/gcc/testsuite/gfortran.dg/parens_5.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/arith.c
    trunk/gcc/fortran/expr.c
    trunk/gcc/fortran/module.c
    trunk/gcc/testsuite/ChangeLog


-- 


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (7 preceding siblings ...)
  2006-02-10 18:08 ` tobi at gcc dot gnu dot org
@ 2006-02-12 18:31 ` tobi at gcc dot gnu dot org
  2006-02-12 18:32 ` tobi at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-12 18:31 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #15 from tobi at gcc dot gnu dot org  2006-02-12 18:31 -------
Subject: Bug 14771

Author: tobi
Date: Sun Feb 12 18:31:40 2006
New Revision: 110894

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110894
Log:
fortran/
        Backport r110819 and r110840 from the trunk
        PR fortran/14771
        * gfortran.h (gfc_intrinsic_op): Add INTRINSIC_PARENTHESES.
        * dump-parse-tree (gfc_show_expr): Handle INTRINSIC_PARENTHESES.
        * expr.c (simplify_intrinsic_op): Treat INTRINSIC_PARENTHESES as
        if it were INTRINSIC_UPLUS.
        * resolve.c (resolve_operator): Handle INTRINSIC_PARENTHESES.
        * match.c (intrinsic_operators): Add INTRINSIC_PARENTHESES.
        * matchexp.c (match_primary): Record parentheses surrounding
        numeric expressions.
        * module.c (intrinsics): Add INTRINSIC_PARENTHESES for module
        dumping.
        * trans-expr.c (gfc_conv_expr_op): Handle INTRINSIC_PARENTHESES.

        PR fortran/14771
        * arith.c (eval_intrinsic): Accept INTRINSIC_PARENTHESES.
        * expr.c (check_intrinsic_op): Likewise.
        * module.c (mio_expr): Likewise.
testsuite/
        Backport r110819 and r110840 from the trunk
        PR fortran/14771
        * gfortran.dg/parens_1.f90: New.
        * gfortran.dg/parens_2.f90: New.
        * gfortran.dg/parens_3.f90: New.
        * gfortran.dg/parens_4.f90: New.
        * gfortran.dg/parens_5.f90: New.


Added:
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/parens_1.f90
      - copied unchanged from r110819,
trunk/gcc/testsuite/gfortran.dg/parens_1.f90
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/parens_2.f90
      - copied unchanged from r110819,
trunk/gcc/testsuite/gfortran.dg/parens_2.f90
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/parens_3.f90
      - copied unchanged from r110819,
trunk/gcc/testsuite/gfortran.dg/parens_3.f90
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/parens_4.f90
      - copied unchanged from r110840,
trunk/gcc/testsuite/gfortran.dg/parens_4.f90
    branches/gcc-4_1-branch/gcc/testsuite/gfortran.dg/parens_5.f90
      - copied unchanged from r110840,
trunk/gcc/testsuite/gfortran.dg/parens_5.f90
Modified:
    branches/gcc-4_1-branch/gcc/fortran/ChangeLog
    branches/gcc-4_1-branch/gcc/fortran/arith.c
    branches/gcc-4_1-branch/gcc/fortran/dump-parse-tree.c
    branches/gcc-4_1-branch/gcc/fortran/expr.c
    branches/gcc-4_1-branch/gcc/fortran/gfortran.h
    branches/gcc-4_1-branch/gcc/fortran/match.c
    branches/gcc-4_1-branch/gcc/fortran/matchexp.c
    branches/gcc-4_1-branch/gcc/fortran/module.c
    branches/gcc-4_1-branch/gcc/fortran/resolve.c
    branches/gcc-4_1-branch/gcc/fortran/trans-expr.c
    branches/gcc-4_1-branch/gcc/testsuite/ChangeLog


-- 


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
       [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
                   ` (8 preceding siblings ...)
  2006-02-12 18:31 ` tobi at gcc dot gnu dot org
@ 2006-02-12 18:32 ` tobi at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-02-12 18:32 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #16 from tobi at gcc dot gnu dot org  2006-02-12 18:32 -------
Fixed.


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.1.0


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


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

* [Bug fortran/14771] frontend doesn't record parentheses
  2004-03-29 16:25 [Bug fortran/14771] New: gfortran " Tobias dot Schlueter at physik dot uni-muenchen dot de
@ 2005-04-11  0:06 ` tobi at gcc dot gnu dot org
  0 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2005-04-11  0:06 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From tobi at gcc dot gnu dot org  2005-04-11 00:06 -------
Not recording parentheses does have correctness implications, see this thread:
http://gcc.gnu.org/ml/fortran/2005-04/msg00302.html

Upgrading the bug status accordingly.  Looks like I have to revive my patch
sometime soon.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|Tobias dot Schlueter at     |unassigned at gcc dot gnu
                   |physik dot uni-muenchen dot |dot org
                   |de                          |
           Severity|enhancement                 |normal
             Status|ASSIGNED                    |NEW
           Keywords|                            |accepts-invalid, wrong-code


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


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

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

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-14771-7283@http.gcc.gnu.org/bugzilla/>
2006-02-08 21:51 ` [Bug fortran/14771] frontend doesn't record parentheses fxcoudert at gcc dot gnu dot org
2006-02-08 21:59 ` tobi at gcc dot gnu dot org
2006-02-09 15:24 ` tobi at gcc dot gnu dot org
2006-02-09 15:25 ` tobi at gcc dot gnu dot org
2006-02-09 15:55 ` tobi at gcc dot gnu dot org
2006-02-09 16:00 ` tobi at gcc dot gnu dot org
2006-02-10  0:10 ` tobi at gcc dot gnu dot org
2006-02-10 18:08 ` tobi at gcc dot gnu dot org
2006-02-12 18:31 ` tobi at gcc dot gnu dot org
2006-02-12 18:32 ` tobi at gcc dot gnu dot org
2004-03-29 16:25 [Bug fortran/14771] New: gfortran " Tobias dot Schlueter at physik dot uni-muenchen dot de
2005-04-11  0:06 ` [Bug fortran/14771] " tobi 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).