Hi!
On Thu, Aug 15, 2024 at 08:50:38PM +0200, Jakub Jelinek wrote:
> > whoopsie, I am sorry for that.
> >
> > What I don't get is, why this has not been reported during my bootstrap. I
> > am
> > doing this to bootstrap:
> >
> > LANG=C "${SRCPATH}/configure" \
> > --disable-multilib\
> > --enable-languages=c,fortran,c++\
> > --prefix="${INSTALLPATH}"
> > LANG=C make -j ${NOPARALLEL} bootstrap
> >
> > What is wrong with that?
>
> That should just work and catch it IMHO.
>
> > Er, Jakub, do you do the patch, as you have assigned yourself?
>
> I'm just 40 minutes into bootstrapping/regtesting that patch
> on x86_64-linux and i686-linux, usually bootstrap takes ~ 50 minutes
> and regtest ~ 65 minutes on the latter and ~ 85 minutes + ~ 70 minutes
> on the former, so if you can get it tested faster than that, go ahead and
> commit it, if not, I'll commit it when I'm done with testing.
> It certainly got past the point of the failed bootstraps already.
Here is what I've committed after successful x86_64-linux and i686-linux
bootstraps and regtests:
2024-08-15 Jakub Jelinek <[email protected]>
PR bootstrap/116387
* resolve.cc (resolve_operator): Use %L rather than %%L in format
string.
--- gcc/fortran/resolve.cc.jj 2024-08-15 19:14:25.700837372 +0200
+++ gcc/fortran/resolve.cc 2024-08-15 19:58:04.512851806 +0200
@@ -4646,7 +4646,7 @@ resolve_operator (gfc_expr *e)
}
else
{
- gfc_error ("Inconsistent coranks for operator at %%L and %%L",
+ gfc_error ("Inconsistent coranks for operator at %L and %L",
&op1->where, &op2->where);
return false;
}
Jakub
