On 01/26/2017 06:12 PM, Gerald Pfeifer wrote: > Hi Martin, > > On Wed, 25 Jan 2017, Martin Liška wrote: >> Following patch documents DO loop changes which were done for upcoming >> GCC 7.1. > > thanks for putting this together. > > Index: htdocs/gcc-7/changes.html > =================================================================== > + <li> > + DO loops with step equal to 1 or -1 generate faster code as they do not > + have a loop preheader. New warning <code>-Wundefined-do-loop</code> > + warns when a loop iterates either to <code>HUGE(i)</code> (with step > equal > + to 1), or to <code>-HUGE(i)</code> (with step equal to -1). Apart from > + that the invalid behaviour can be caught during run-time of a program > with > + <code>-fcheck=do</code>: > > Can you makr up DO as <code>DO</code> since it is a keyword? > > "A new warning..." > > "behavior" (since we use American English, or it also would be > programme ;-). > > "run time" (according to https://gcc.gnu.org/codingconventions.html ) > > +At line 8 of file > /home/marxin/Programming/gcc/gcc/testsuite/gfortran.dg/do_check_12.f90 > +Fortran runtime error: Loop iterates infinitely > + </pre></blockquote> > > Here, can you shorten the filename to > ".../gcc/testsuite/gfortran.dg/do_check_12.f90" ? > > > This is fine with the changes above. > > Thanks, > Gerald >
Hi Gerald. Thanks for review, all notes are fixed. Martin
