Dear Nicolas, This is OK for trunk.
Thanks for the patch. Paul On 19 March 2017 at 14:34, Nicolas Koenig <koeni...@student.ethz.ch> wrote: > Hello Paul, > > there isn't really a reason for that except for not knowing where to put the > error. Attached are the new patch & test case. > > The paperwork is done and I have commit rights. > > Thanks for the kind welcome! > > Nicolas > > Regression tested for x86_64-pc-linux-gnu. > > 2017-03-12 Nicolas Koenig <koeni...@student.ethz.ch> > > PR fortran/39239 > * symbol.c (check_conflict): report an error if an EQUIVALENCE variable > is BIND(C). > > 2017-03-12 Nicolas Koenig <koeni...@student.ethz.ch> > > PR fortran/39239 > * gfortran.dg/equiv_constraint_bind_c.f90: New test. > > > On 03/19/2017 01:02 PM, Paul Richard Thomas wrote: >> >> Hi Nicolas, >> >> Is there some reason that you didn't use symbol.c(check_conflict)? The >> conflict check could be added at line 547. If this results in >> repetitions of the error message, then your patch is OK. Otherwise, I >> would pop it in there. >> >> Do you have commit rights? ie. have you done the FSF paperwork? >> >> Welcome aboard! >> >> Paul >> >> On 19 March 2017 at 00:15, Nicolas Koenig <koeni...@student.ethz.ch> >> wrote: >>> >>> Hello everyone, >>> >>> I submitted this patch a week ago, but I think it got lost. It adds an >>> error >>> if BIND(C) is used with EQUIVALENCE. >>> >>> Nicolas >>> >>> Regression tested for x86_64-pc-linux-gnu. >>> >>> 2017-03-18 Nicolas Koenig <koeni...@student.ethz.ch> >>> >>> PR fortran/39239 >>> * resolve.c (resolve_equivalence): report an error if an >>> equivalence variable is BIND(C). >>> >>> 2017-03-18 Nicolas Koenig <koeni...@student.ethz.ch> >>> >>> PR fortran/39239 >>> * gfortran.dg/equiv_constraint_bind_c.f90: New test. >>> >> >> > -- "If you can't explain it simply, you don't understand it well enough" - Albert Einstein