I have just committed the updated patch to trunk as r269658. If anyone thinks this should be backported to 8-branch, please let me know.
Cheers, Janus Am Di., 12. März 2019 um 13:00 Uhr schrieb Janus Weil <ja...@gcc.gnu.org>: > > Hi Steve, > > > > Technically the ICE is a regression, but since it happens on invalid > > > code only, backporting is not essential IMHO (but might still be > > > useful). Ok for trunk? And 8-branch? > > > > Looks good to me with a minor suggestion. See below. > > thanks for the comments. Updated patch attached. > > > > You may want to wait a bit or ping Paul to give him > > a chance to peek at the patch. > > I'm putting Paul in CC and will wait until tomorrow night before committing. > > > > > if (gfc_match_char (')') == MATCH_YES) > > > - goto ok; > > > + { > > > + if (typeparam) > > > + { > > > + gfc_error_now ("A parameter name is required at %C"); > > > > Can you insert "type" so the error reads "A type parameter name"? > > Unfortunately, the words parameter has a few too many meanings. > > True. I'm using 'type parameter list' now. > > Cheers, > Janus