On 09/01/2012 15:45, Tobias Burnus wrote: > On 01/09/2012 03:34 PM, Mikael Morin wrote: >> The issue is that the code handling NULL() doesn't consume the gfc_ss >> struct created for it. Your fix, which advances to the next one anyway >> would work just well, but I think it is slightly cleaner to not create >> the struct in the first place, as it is unused. >> >> I'm currently regtesting the following patch. The testcase is the same >> as your. OK for 4.7/4.6? > > OK. > > For the test case, either take mine or don't forget to enable all "if() > abort"s and to change in the last "! print *,a" the "a" into a "b". > > + if (!arg->expr > + || arg->expr->expr_type == EXPR_NULL) > continue; > > (I'd prefer to have no line break before the "||".) > > Tobias
As this patch was approved for 4.6, I will now backport it and then close the bug, unless someone complains. Mikael
