[Bug fortran/33502] New: gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[Bug fortran/33502] New: gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[sjc at pobox dot com]

~> gfortran -v
Using built-in specs.
Target: x86_64-unknown-linux-gnu
Configured with: /projects/tob/gcc/configure --enable-languages=c,fortran
--prefix=/projects/tob/gcc-trunk
Thread model: posix
gcc version 4.3.0 20070919 (experimental) [trunk revision 128597] (GCC)
~> cat fm.F
        end
~> gfortran -c -g fm.F
~> gfortran -c -g3 fm.F
Warning: <built-in>:1: Illegal preprocessor directive
Warning: <built-in>:3: Illegal preprocessor directive
Warning: <built-in>:5: Illegal preprocessor directive
Warning: <built-in>:7: Illegal preprocessor directive
Warning: <built-in>:9: Illegal preprocessor directive
Warning: <built-in>:11: Illegal preprocessor directive
Warning: <built-in>:13: Illegal preprocessor directive
Warning: <built-in>:15: Illegal preprocessor directive
Warning: <built-in>:17: Illegal preprocessor directive
Warning: <built-in>:19: Illegal preprocessor directive
Warning: <built-in>:21: Illegal preprocessor directive
Warning: <built-in>:23: Illegal preprocessor directive
Warning: <built-in>:25: Illegal preprocessor directive
Warning: <built-in>:27: Illegal preprocessor directive
Warning: <built-in>:29: Illegal preprocessor directive
Warning: <built-in>:31: Illegal preprocessor directive
Warning: <built-in>:33: Illegal preprocessor directive
Warning: <built-in>:35: Illegal preprocessor directive
Warning: <built-in>:37: Illegal preprocessor directive
Warning: <built-in>:39: Illegal preprocessor directive
Warning: <built-in>:41: Illegal preprocessor directive
Warning: <built-in>:43: Illegal preprocessor directive
Warning: <built-in>:45: Illegal preprocessor directive
Warning: <built-in>:47: Illegal preprocessor directive
Warning: <built-in>:49: Illegal preprocessor directive
Warning: <built-in>:51: Illegal preprocessor directive
Warning: <built-in>:53: Illegal preprocessor directive
Warning: <built-in>:55: Illegal preprocessor directive
Warning: <built-in>:57: Illegal preprocessor directive
Warning: <built-in>:59: Illegal preprocessor directive
Warning: <built-in>:61: Illegal preprocessor directive
Warning: <built-in>:63: Illegal preprocessor directive
Warning: <built-in>:65: Illegal preprocessor directive
Warning: <built-in>:67: Illegal preprocessor directive
Warning: <built-in>:69: Illegal preprocessor directive
Warning: <built-in>:71: Illegal preprocessor directive
Warning: <built-in>:73: Illegal preprocessor directive
Warning: <built-in>:75: Illegal preprocessor directive
Warning: <built-in>:77: Illegal preprocessor directive
Warning: <built-in>:79: Illegal preprocessor directive
Warning: <built-in>:81: Illegal preprocessor directive
Warning: <built-in>:83: Illegal preprocessor directive
Warning: <built-in>:85: Illegal preprocessor directive
Warning: <built-in>:87: Illegal preprocessor directive
Warning: <built-in>:89: Illegal preprocessor directive
Warning: <built-in>:91: Illegal preprocessor directive
Warning: <built-in>:93: Illegal preprocessor directive
Warning: <built-in>:95: Illegal preprocessor directive
Warning: <built-in>:97: Illegal preprocessor directive
Warning: <built-in>:99: Illegal preprocessor directive
Warning: <built-in>:101: Illegal preprocessor directive
Warning: <built-in>:103: Illegal preprocessor directive
Warning: <built-in>:105: Illegal preprocessor directive
Warning: <built-in>:107: Illegal preprocessor directive
Warning: <built-in>:109: Illegal preprocessor directive
Warning: <built-in>:111: Illegal preprocessor directive
Warning: <built-in>:113: Illegal preprocessor directive
Warning: <built-in>:115: Illegal preprocessor directive
Warning: <built-in>:117: Illegal preprocessor directive
Warning: <built-in>:119: Illegal preprocessor directive
Warning: <built-in>:121: Illegal preprocessor directive
Warning: <built-in>:123: Illegal preprocessor directive
Warning: <built-in>:125: Illegal preprocessor directive
Warning: <built-in>:127: Illegal preprocessor directive
Warning: <built-in>:129: Illegal preprocessor directive
Warning: <built-in>:131: Illegal preprocessor directive
Warning: <built-in>:133: Illegal preprocessor directive
Warning: <built-in>:135: Illegal preprocessor directive
Warning: <built-in>:137: Illegal preprocessor directive
Warning: <built-in>:139: Illegal preprocessor directive
Warning: <built-in>:141: Illegal preprocessor directive
Warning: <built-in>:143: Illegal preprocessor directive
Warning: <built-in>:145: Illegal preprocessor directive
Warning: <built-in>:147: Illegal preprocessor directive
Warning: <built-in>:149: Illegal preprocessor directive
Warning: <built-in>:151: Illegal preprocessor directive
Warning: <built-in>:153: Illegal preprocessor directive
Warning: <built-in>:155: Illegal preprocessor directive
Warning: <built-in>:157: Illegal preprocessor directive
Warning: <built-in>:159: Illegal preprocessor directive
Warning: <built-in>:161: Illegal preprocessor directive
Warning: <built-in>:163: Illegal preprocessor directive
Warning: <built-in>:165: Illegal preprocessor directive
Warning: <built-in>:167: Illegal preprocessor directive
Warning: <built-in>:169: Illegal preprocessor directive
Warning: <built-in>:171: Illegal preprocessor directive
Warning: <built-in>:173: Illegal preprocessor directive
Warning: <built-in>:175: Illegal preprocessor directive
Warning: <built-in>:177: Illegal preprocessor directive
Warning: <built-in>:179: Illegal preprocessor directive
Warning: <built-in>:181: Illegal preprocessor directive
Warning: <built-in>:183: Illegal preprocessor directive
Warning: <built-in>:185: Illegal preprocessor directive
Warning: <built-in>:187: Illegal preprocessor directive
Warning: <built-in>:189: Illegal preprocessor directive
Warning: <built-in>:191: Illegal preprocessor directive
Warning: <built-in>:193: Illegal preprocessor directive
Warning: <built-in>:195: Illegal preprocessor directive
Warning: <built-in>:197: Illegal preprocessor directive
Warning: <built-in>:199: Illegal preprocessor directive
Warning: <built-in>:201: Illegal preprocessor directive
Warning: <built-in>:203: Illegal preprocessor directive
Warning: <built-in>:205: Illegal preprocessor directive
Warning: <built-in>:207: Illegal preprocessor directive
Warning: <built-in>:209: Illegal preprocessor directive
Warning: <built-in>:211: Illegal preprocessor directive
Warning: <built-in>:213: Illegal preprocessor directive
Warning: <built-in>:215: Illegal preprocessor directive
Warning: <built-in>:219: Illegal preprocessor directive
Warning: <built-in>:221: Illegal preprocessor directive
Warning: <built-in>:223: Illegal preprocessor directive
Warning: <built-in>:225: Illegal preprocessor directive
Warning: <built-in>:227: Illegal preprocessor directive
Warning: <built-in>:229: Illegal preprocessor directive
Warning: <built-in>:231: Illegal preprocessor directive
Warning: <built-in>:233: Illegal preprocessor directive
Warning: <built-in>:235: Illegal preprocessor directive
Warning: <built-in>:237: Illegal preprocessor directive
Warning: <built-in>:239: Illegal preprocessor directive
Warning: <built-in>:241: Illegal preprocessor directive
Warning: <built-in>:243: Illegal preprocessor directive
Warning: <built-in>:245: Illegal preprocessor directive
Warning: <built-in>:247: Illegal preprocessor directive
Warning: <built-in>:249: Illegal preprocessor directive
Warning: <built-in>:251: Illegal preprocessor directive
Warning: <built-in>:253: Illegal preprocessor directive
Warning: <built-in>:258: Illegal preprocessor directive
Warning: <built-in>:260: Illegal preprocessor directive
Warning: <command-line>:1: Illegal preprocessor directive


-- 
           Summary: gfortran with .F suffix and -g3 option chokes on
                    preprocessor syntax
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: sjc at pobox dot com


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2007-09-19 18:58 -------
#define have to be understood by the front-end with -g3


-- 

pinskia at gcc dot gnu dot org changed:

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


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #2 from fxcoudert at gcc dot gnu dot org  2007-09-20 08:07 -------
Confirmed. Andrew, what is the front-end expected to do with the #defines? I
suspect that we need to pass them to the middle-end, by calling
debug_hooks->define and debug_hooks->undef (as is done in c-lex.c).


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |fxcoudert at gcc dot gnu dot
                   |                            |org
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
           Keywords|                            |rejects-valid
   Last reconfirmed|0000-00-00 00:00:00         |2007-09-20 08:08:00
               date|                            |


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2007-09-20 09:19 -------
Subject: Re:  gfortran with .F suffix and -g3 option chokes on preprocessor
syntax

On 20 Sep 2007 08:08:00 -0000, fxcoudert at gcc dot gnu dot org
<gcc-bugzilla@gcc.gnu.org> wrote:
>
>
> ------- Comment #2 from fxcoudert at gcc dot gnu dot org  2007-09-20 08:07 -------
> Confirmed. Andrew, what is the front-end expected to do with the #defines? I
> suspect that we need to pass them to the middle-end, by calling
> debug_hooks->define and debug_hooks->undef (as is done in c-lex.c).


Yes that should be enough.

Thanks,
Andrew Pinski


-- 


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |fxcoudert at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
      Known to fail|                            |4.3.0 4.2.1 4.1.3
   Last reconfirmed|2007-09-20 08:08:00         |2007-09-20 09:34:21
               date|                            |


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #4 from fxcoudert at gcc dot gnu dot org  2007-09-22 15:03 -------
Subject: Bug 33502

Author: fxcoudert
Date: Sat Sep 22 15:03:24 2007
New Revision: 128671

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=128671
Log:
        PR fortran/33502
        * scanner.c (gfc_advance_line): Call debug_hooks->start_source_file
        and debug_hooks->end_source_file when entering and exiting
        included files.
        (gfc_define_undef_line): New function.
        (load_file): Ignore #define and #undef preprocessor lines
        while reading source files.
        * parse.c (next_statement): Handle #define and #undef
        preprocessor lines.
        (gfc_parse_file): Call debug_hooks->start_source_file and
        debug_hooks->end_source_file for the main source file if
        requested by the debug format.
        * gfortran.h (gfc_define_undef_line): Add prototype.

Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/parse.c
    trunk/gcc/fortran/scanner.c


-- 


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #5 from fxcoudert at gcc dot gnu dot org  2007-09-22 15:04 -------
Fixed.


-- 

fxcoudert 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=33502

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #6 from fxcoudert at gcc dot gnu dot org  2007-09-24 20:28 -------
It would be nice to find a patch that doesn't break bootstrap on plenty of
platforms :)

I'll be on it when I have time, PR 33538 indicates the trouble caused by my
first patch, now reverted.


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #7 from fxcoudert at gcc dot gnu dot org  2007-10-01 00:05 -------
Created an attachment (id=14276)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=14276&action=view)
Updated patch

The attached updated patch seems to fix the issue (and also fixes a problem in
the logic and yet another unrelated problem; I'll explain in my submission
mail). This is currently regtesting with different debug formats.


-- 


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #8 from fxcoudert at gcc dot gnu dot org  2007-10-04 15:04 -------
Subject: Bug 33502

Author: fxcoudert
Date: Thu Oct  4 15:04:09 2007
New Revision: 129011

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=129011
Log:
        PR fortran/33502
        * scanner.c (gfc_advance_line): Call debug_hooks->end_source_file
        and debug_hooks->start_source_file when appropriate, and set
        dbg_emitted.
        (gfc_define_undef_line): New function.
        (load_file): Don't error out on #define and #undef lines.
        * parse.c (next_statement): Call gfc_define_undef_line.
        (gfc_parse_file): Call debug_hooks->start_source_file and
        debug_hooks->end_source_file for the main source file if
        required.
        * gfortran.h (gfc_linebuf): Add dbg_emitted field.
        (gfc_define_undef_line): New prototype.

Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/parse.c
    trunk/gcc/fortran/scanner.c


-- 


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

[Bug fortran/33502] gfortran with .F suffix and -g3 option chokes on preprocessor syntax

[fxcoudert at gcc dot gnu dot org]

------- Comment #9 from fxcoudert at gcc dot gnu dot org  2007-10-04 15:05 -------
Fixed on mainline. Thanks for the bug report!


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED


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