Re: [patch, fortran] RFD: PR 56666 Allow suppression of zero-trip DO loop warning

2013-08-11 Thread Thomas Koenig
Hi Janus, >> OK for trunk? > Looks good to m Committed as rev. 201658; also committed a snippet to the documentation. Thanks for the review! Thomas

Re: [patch, fortran] RFD: PR 56666 Allow suppression of zero-trip DO loop warning

2013-08-11 Thread Janus Weil
2013/8/11 Thomas Koenig : > Hi Janus, > >> So: Ok for trunk from my side. >> >> However, I would prefer to disable the warning by default, but include >> it in -Wall. > > Here's a patch to do just that. > > Regression-tested (hence the changes to the existing test cases :-) > > OK for trunk? Looks

Re: [patch, fortran] RFD: PR 56666 Allow suppression of zero-trip DO loop warning

2013-08-11 Thread Thomas Koenig
Hi Janus, > So: Ok for trunk from my side. > > However, I would prefer to disable the warning by default, but include > it in -Wall. Here's a patch to do just that. Regression-tested (hence the changes to the existing test cases :-) OK for trunk? Once it goes in, I will also update the WWW ch

Re: [patch, fortran] RFD: PR 56666 Allow suppression of zero-trip DO loop warning

2013-08-06 Thread Janus Weil
Hi Thomas, > the rather self-explanatory patch implements the -Wzerotrip > option. The positive form is not really useful, because > the option is on by default (so the default behavior is > not changed). > > The negative form of the option, -Wno-zerotrip, suppresses the > warning. I have also a

[patch, fortran] RFD: PR 56666 Allow suppression of zero-trip DO loop warning

2013-08-03 Thread Thomas Koenig
Hello world, the rather self-explanatory patch implements the -Wzerotrip option. The positive form is not really useful, because the option is on by default (so the default behavior is not changed). The negative form of the option, -Wno-zerotrip, suppresses the warning. I have also added infor