On 9/26/19 10:45 AM, Mark Eggleston wrote:
Original thread starts here https://gcc.gnu.org/ml/gcc-patches/2019-09/msg01185.html
OK to commit?

As Steve, I am not really happy about adding yet another option and especially not about legacy features. On the other hand, I see that legacy code is still used.

Having said this, the patch is OK from my side.

Tobias

PS: I was also not that happy about the BOZ changes by Steve, which broke code here – but, fortunately, adding int( ,kind=) around it was sufficient and that code was supposed to be F2003 standard conforming.  I could ping the authors and is now fixed. Still, I wonder how much code broke due to that change; code is not that simple to fix. – But, in general, I am very much in favour in having valid Fortran 2018 code (can be fixed form, old and use old features, that's fine).


gcc/fortran/ChangeLog

    Mark Eggleston  <mark.eggles...@codethink.com>

    * invoke.texi: Add -Wno-overwrite-recursive to list of options. Add
    description of -Wno-overwrite-recursive. Fix typo in description
    of -Winteger-division.
    * lang.opt: Add option -Woverwrite-recursive initialised as on.
    * option.c (gfc_post_options): Output warning only if it is enabled.

gcc/testsuite/ChangeLog

    Mark Eggleston <mark.eggles...@codethink.com>

    * gfortran.dg/no_overwrite_recursive_1.f90: New test.
    * gfortran.dg/no_overwrite_recursive_2.f90: New test.

Reply via email to