Re: [patch, fortran] Fix PR 69544

2016-11-01 Thread Dominique d'Humières
gfortran regtested without regression with the patch --- ../_clean/gcc/fortran/match.c 2016-11-01 17:24:32.0 +0100 +++ gcc/fortran/match.c 2016-11-01 16:21:31.0 +0100 @@ -6219,6 +6219,7 @@ match_simple_where (void) c->next = XCNEW (gfc_code); *c->next = new_st; + c->

Re: [patch, fortran] Fix PR 69544

2016-11-01 Thread Thomas Koenig
Hi Dominique, The patch fixes the ICE due to the missing gfc_current_locus in the last one, but the same should be applied to match_simple_where, otherwise one gets an ICE for statements such as if (n==10) where (txt(1:3) /= '' ) y(1:3,i,j) = txt(1:3) I can confirm that the bug exists. B

Re: [patch, fortran] Fix PR 69544

2016-11-01 Thread Dominique d'Humières
Hi Thomas, > the attached, rather simple, patch, fixes a regression where > the locus was not set, leading to an ICE on a warning. Looking for the occurrences of gfc_clear_new_st in fortran/match.c, I have found it in the following procs: gfc_match_if match_simple_forall gfc_match_forall match_s

Re: [patch, fortran] Fix PR 69544

2016-11-01 Thread Steve Kargl
On Tue, Nov 01, 2016 at 11:58:10AM +0100, Thomas Koenig wrote: > > the attached, rather simple, patch, fixes a regression where > the locus was not set, leading to an ICE on a warning. > > Regression-tested on trunk. OK on all affected and open branches > (7/6/5)? > Yes. -- Steve