public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
@ 2006-11-07  4:35 ` brooks at gcc dot gnu dot org
  2006-11-07  4:50 ` jvdelisle at gcc dot gnu dot org
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: brooks at gcc dot gnu dot org @ 2006-11-07  4:35 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from brooks at gcc dot gnu dot org  2006-11-07 04:35 -------
Would simply having the error message say "operator '.eq.' or '=='" be a
sufficient solution?


-- 

brooks at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |brooks at gcc dot gnu dot
                   |                            |org


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
  2006-11-07  4:35 ` [Bug fortran/17711] Wrong operator name in error message brooks at gcc dot gnu dot org
@ 2006-11-07  4:50 ` jvdelisle at gcc dot gnu dot org
  2006-11-22 14:48 ` tobi at gcc dot gnu dot org
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: jvdelisle at gcc dot gnu dot org @ 2006-11-07  4:50 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from jvdelisle at gcc dot gnu dot org  2006-11-07 04:50 -------
Yes, but I think you could also grab out of the lb the symbol actually used,
though it may not be worth specializing this.  Are there places where we get
errors for .ne. and /= or other similar things?


-- 


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
  2006-11-07  4:35 ` [Bug fortran/17711] Wrong operator name in error message brooks at gcc dot gnu dot org
  2006-11-07  4:50 ` jvdelisle at gcc dot gnu dot org
@ 2006-11-22 14:48 ` tobi at gcc dot gnu dot org
  2006-11-22 19:49 ` 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-11-22 14:48 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from tobi at gcc dot gnu dot org  2006-11-22 14:48 -------
A possible solution is to add entries for the symbolic forms (e.g. '==') of the
intrinsic operators to the enum gfc_intrinsic_op, and then use them
interchangeably, except when matching the operators, and when printing error
messages.


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |tobi at gcc dot gnu dot org


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2006-11-22 14:48 ` tobi at gcc dot gnu dot org
@ 2006-11-22 19:49 ` tobi at gcc dot gnu dot org
  2006-11-23  0:35 ` patchapp at dberlin dot org
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: tobi at gcc dot gnu dot org @ 2006-11-22 19:49 UTC (permalink / raw)
  To: gcc-bugs



-- 

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|2005-12-30 18:46:39         |2006-11-22 19:49:21
               date|                            |


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2006-11-22 19:49 ` tobi at gcc dot gnu dot org
@ 2006-11-23  0:35 ` patchapp at dberlin dot org
  2006-11-29 20:46 ` tobi at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: patchapp at dberlin dot org @ 2006-11-23  0:35 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from patchapp at dberlin dot org  2006-11-23 00:35 -------
Subject: Bug number PR17711

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-11/msg01590.html


-- 


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2006-11-23  0:35 ` patchapp at dberlin dot org
@ 2006-11-29 20:46 ` tobi at gcc dot gnu dot org
  2007-06-29 23:51 ` dfranke 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-11-29 20:46 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from tobi at gcc dot gnu dot org  2006-11-29 20:45 -------
I retracted the patch because it causes regressions with user-defined
operators.  I'm not convinced that this problem is worth doing anything
non-trivial as is required, so I'm un-assigning myself.


-- 

tobi at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|tobi at gcc dot gnu dot org |unassigned at gcc dot gnu
                   |                            |dot org
             Status|ASSIGNED                    |NEW


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (5 preceding siblings ...)
  2006-11-29 20:46 ` tobi at gcc dot gnu dot org
@ 2007-06-29 23:51 ` dfranke at gcc dot gnu dot org
  2007-06-30 16:23 ` patchapp at dberlin dot org
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2007-06-29 23:51 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from dfranke at gcc dot gnu dot org  2007-06-29 23:51 -------
Think I got all the pieces for this one ...


-- 

dfranke at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dfranke at gcc dot gnu dot
                   |                            |org
         AssignedTo|unassigned at gcc dot gnu   |dfranke at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2006-11-22 19:49:21         |2007-06-29 23:51:16
               date|                            |


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (6 preceding siblings ...)
  2007-06-29 23:51 ` dfranke at gcc dot gnu dot org
@ 2007-06-30 16:23 ` patchapp at dberlin dot org
  2007-07-08 21:09 ` dfranke at gcc dot gnu dot org
  2007-07-08 21:13 ` dfranke at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: patchapp at dberlin dot org @ 2007-06-30 16:23 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #8 from patchapp at dberlin dot org  2007-06-30 16:23 -------
Subject: Bug number PR17711

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/2007-06/msg02153.html


-- 


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (7 preceding siblings ...)
  2007-06-30 16:23 ` patchapp at dberlin dot org
@ 2007-07-08 21:09 ` dfranke at gcc dot gnu dot org
  2007-07-08 21:13 ` dfranke at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2007-07-08 21:09 UTC (permalink / raw)
  To: gcc-bugs

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



------- Comment #9 from dfranke at gcc dot gnu dot org  2007-07-08 21:09 -------
Subject: Bug 17711

Author: dfranke
Date: Sun Jul  8 21:08:52 2007
New Revision: 126468

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=126468
Log:
gcc/fortran:
2007-07-08  Daniel Franke  <franke.daniel@gmail.com>
            Tobias Schlüter <tobias.schlueter@physik.uni-muenchen.de>

        PR fortran/17711
        * gfortran.h (gfc_intrinsic_op): Added INTRINSIC_EQ_OS,
        INTRINSIC_NE_OS, INTRINSIC_GT_OS, INTRINSIC_GE_OS, 
        INTRINSIC_LT_OS and INTRINSIC_LE_OS.
        * arith.c (eval_intrinsic, eval_type_intrinsic0): Likewise.
        * arith.h (gfc_eq, gfc_ne, gfc_gt, gfc_ge, gfc_lt, gfc_le):
        Added gfc_intrinsic_op as third argument type.
        * dump-parse-tree.c (gfc_show_expr): Account for new enum values.
        * expr.c (simplify_intrinsic_op, check_intrinsic_op): Likewise.
        * interface.c (check_operator_interface): Likewise.
        (gfc_check_interfaces): Added cross-checks for FORTRAN 77 and 
        Fortran 90 style operators using new enum values.
        (gfc_extend_expr): Likewise.
        (gfc_add_interface): Likewise.
        * match.c (intrinsic_operators): Distinguish FORTRAN 77 style
        operators from Fortran 90 style operators using new enum values.
        * matchexp.c (match_level_4): Account for new enum values.
        * module.c (mio_expr): Likewise.
        * resolve.c (resolve_operator): Deal with new enum values, fix
        inconsistent error messages.
        * trans-expr.c (gfc_conv_expr_op): Account for new enum values.

gcc/testsuite:
2007-07-08  Daniel Franke  <franke.daniel@gmail.com>

        PR fortran/17711
        * gfortran.dg/operator_4.f90: New test.
        * gfortran.dg/operator_5.f90: New test.
        * gfortran.dg/logical_comp.f90: Adjusted error messages.
        * gfortran.dg/module_md5_1.f90: Adjusted MD5 sum.


Added:
    trunk/gcc/testsuite/gfortran.dg/operator_4.f90
    trunk/gcc/testsuite/gfortran.dg/operator_5.f90
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/arith.c
    trunk/gcc/fortran/arith.h
    trunk/gcc/fortran/dump-parse-tree.c
    trunk/gcc/fortran/expr.c
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/interface.c
    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
    trunk/gcc/testsuite/gfortran.dg/logical_comp.f90
    trunk/gcc/testsuite/gfortran.dg/module_md5_1.f90


-- 


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


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

* [Bug fortran/17711] Wrong operator name in error message
       [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
                   ` (8 preceding siblings ...)
  2007-07-08 21:09 ` dfranke at gcc dot gnu dot org
@ 2007-07-08 21:13 ` dfranke at gcc dot gnu dot org
  9 siblings, 0 replies; 11+ messages in thread
From: dfranke at gcc dot gnu dot org @ 2007-07-08 21:13 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #10 from dfranke at gcc dot gnu dot org  2007-07-08 21:13 -------
Fixed in trunk. No regression, no backport. Closing.


-- 

dfranke at gcc dot gnu dot org changed:

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


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


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

* [Bug fortran/17711] Wrong operator name in error message
  2004-09-28 13:09 [Bug fortran/17711] New: " schnetter at aei dot mpg dot de
@ 2004-09-28 13:16 ` pinskia at gcc dot gnu dot org
  0 siblings, 0 replies; 11+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2004-09-28 13:16 UTC (permalink / raw)
  To: gcc-bugs


------- Additional Comments From pinskia at gcc dot gnu dot org  2004-09-28 13:16 -------
Confirmed, I think this is because we don't keep around if the user used .eq. or ==.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|                            |1
           Keywords|                            |diagnostic
   Last reconfirmed|0000-00-00 00:00:00         |2004-09-28 13:16:45
               date|                            |


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


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

end of thread, other threads:[~2007-07-08 21:13 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <bug-17711-7427@http.gcc.gnu.org/bugzilla/>
2006-11-07  4:35 ` [Bug fortran/17711] Wrong operator name in error message brooks at gcc dot gnu dot org
2006-11-07  4:50 ` jvdelisle at gcc dot gnu dot org
2006-11-22 14:48 ` tobi at gcc dot gnu dot org
2006-11-22 19:49 ` tobi at gcc dot gnu dot org
2006-11-23  0:35 ` patchapp at dberlin dot org
2006-11-29 20:46 ` tobi at gcc dot gnu dot org
2007-06-29 23:51 ` dfranke at gcc dot gnu dot org
2007-06-30 16:23 ` patchapp at dberlin dot org
2007-07-08 21:09 ` dfranke at gcc dot gnu dot org
2007-07-08 21:13 ` dfranke at gcc dot gnu dot org
2004-09-28 13:09 [Bug fortran/17711] New: " schnetter at aei dot mpg dot de
2004-09-28 13:16 ` [Bug fortran/17711] " pinskia 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).