Re: [patch, fortran] Fix ICE on invalid with DO steps

2019-09-15 Thread Thomas Koenig
Hi Steve, s/previusly/previously Fixed, committed. Thanks for the review! Do you it gfortran should skip the front-end optimization pass if error(s) have already been reported? On entry ito this pass, you could test for the error count and simply return. I thought about this for doing thi

Re: [patch, fortran] Fix ICE on invalid with DO steps

2019-09-15 Thread Steve Kargl
On Sun, Sep 15, 2019 at 03:55:55PM +0200, Thomas Koenig wrote: > > Regression-tested. OK for all affected branches (trunk, 9 and 8)? > OK with minor spelling fix. > + sgn = mpz_cmp_ui (dl->ext.iterator->step->value.integer, 0); > + /* This can happen, but then the e

[patch, fortran] Fix ICE on invalid with DO steps

2019-09-15 Thread Thomas Koenig
Hello world, the attached patch fixes an ICE on invalid, where the fact that the step in do i = 1, 3, .1 is actually zero slipped through. Regression-tested. OK for all affected branches (trunk, 9 and 8)? Regards Thomas 2019-09-15 Thomas Koenig PR fortran/91550 *