Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Steve Kargl
On Mon, Nov 14, 2016 at 06:21:06PM +0100, Janus Weil wrote: > 2016-11-14 16:10 GMT+01:00 Steve Kargl : > > On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote: > >> > After looking into this a little bit more, I found that the culprit > >> > seems to be 'resolve_procedure_interface', which d

Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Janus Weil
2016-11-14 16:10 GMT+01:00 Steve Kargl : > On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote: >> > After looking into this a little bit more, I found that the culprit >> > seems to be 'resolve_procedure_interface', which does not properly >> > copy the 'class_ok' attribute. I propose the a

Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Steve Kargl
On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote: > > After looking into this a little bit more, I found that the culprit > > seems to be 'resolve_procedure_interface', which does not properly > > copy the 'class_ok' attribute. I propose the attached patch to fix > > this (regtesting righ

Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Janus Weil
> After looking into this a little bit more, I found that the culprit > seems to be 'resolve_procedure_interface', which does not properly > copy the 'class_ok' attribute. I propose the attached patch to fix > this (regtesting right now) ... The regtest finished successfully. Is that patch ok for

Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Janus Weil
2016-11-14 9:56 GMT+01:00 Janus Weil : > Hi Steve, > >> The attach patch allows a procedure with a class result to >> be an actual argument to subprogram where the dummy argument >> expected to be a class. OK to commit? > > that patch actually does not look quite right to me. Does it survive a >

Re: [PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-14 Thread Janus Weil
Hi Steve, > The attach patch allows a procedure with a class result to > be an actual argument to subprogram where the dummy argument > expected to be a class. OK to commit? that patch actually does not look quite right to me. Does it survive a regtest? I think one should rather check why the c

[PATCH] PR fortran/78300 -- class procedure as actual arg

2016-11-13 Thread Steve Kargl
The attach patch allows a procedure with a class result to be an actual argument to subprogram where the dummy argument expected to be a class. OK to commit? 2016-11-14 Steven G. Kargl PR fortran/78300 * resolve.c (resolve_fl_var_and_proc): Allow class procedure an act