[PATCH Fortran/Diagnostics] Move Fortran to common diagnostics machinery

Tobias Burnus burnus@net-b.de
Fri Aug 15 13:29:00 GMT 2014


Am 06.08.2014 18:50, schrieb Manuel López-Ibáñez:
> This is the first step for moving Fortran to use the common
> diagnostics machinery. This patch makes Fortran use the common
> machinery for those warnings that don't have a location or a
> controlling option.
> Bootstrapped and regression tested on x86_64-linux-gnu.
> OK?

Sorry for replying belated, I was in vacation and seemingly, a few 
others as well.

The patch looks good to me, except for gfc-diagnostic.def which contains 
the file content twice. With that fixed, the patch is OK. Thanks a lot!

Tobias

> 2014-08-03  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/44054
> c-family/
>      * c-format.c: Handle Fortran flags.
>      * diagnostic.c (build_message_string): Make it extern.
>      * diagnostic.h (build_message_string): Make it extern.
> fortran/
>      * gfortran.h: Define GCC_DIAG_STYLE.
>      (gfc_diagnostics_init,gfc_warning_cmdline): Declare.
>      * trans-array.c: Include gfortran.h before diagnostic-core.h.
>      * trans-expr.c: Likewise.
>      * trans-openmp.c: Likewise.
>      * trans-const.c: Likewise.
>      * trans.c: Likewise.
>      * trans-types.c: Likewise.
>      * f95-lang.c: Likewise.
>      * trans-decl.c: Likewise.
>      * trans-io.c: Likewise.
>      * trans-intrinsic.c: Likewise.
>      * error.c: Include diagnostic.h and diagnostic-color.h.
>      (gfc_diagnostic_build_prefix): New.
>      (gfc_diagnostic_starter): New.
>      (gfc_diagnostic_finalizer): New.
>      (gfc_warning_cmdline): New.
>      (gfc_diagnostics_init): New.
>      * gfc-diagnostic.def: New.
>      * options.c (gfc_init_options): Call gfc_diagnostics_init.
>      (gfc_post_options): Use gfc_warning_cmdline.



More information about the Fortran mailing list