On Thu, Sep 28, 2017 at 3:57 PM, Dominique d'Humières <domi...@lps.ens.fr> wrote: > Hi all, > > In the PR there was some consensus to turn the warnings into errors. > > This is what the patch does: the warnings are kept with -std=legacy, errors > are emitted otherwise. > I am sure it may exist a better solution, but I did not find it. > > While regtesting I have found several regressions on top of > gfortran.dg/warn_argument_mismatch_1.f90 > > At this point, I had two options: use -std=legacy everywhere or replace the > dg-warning with dg-error. I have chosen the later except for > warn_argument_mismatch_1.f90. > > Regtested on x86_64-apple-darwin16. > > Comments are welcome and I’ll provide the change logs once the dust has > settled.
Thanks for picking this up. I do think your solution is good enough, and I also agree with how you've dealt with the testsuite changes. So from my point of view, Ok for trunk along with a ChangeLog entry. -- Janne Blomqvist