diff mbox series

[committed] fortran: Fix bootstrap in resolve.cc [PR116387]

Message ID Zr5qh1q2Y/+di39H@tucnak
State New
Headers show
Series [committed] fortran: Fix bootstrap in resolve.cc [PR116387] | expand

Commit Message

Jakub Jelinek Aug. 15, 2024, 8:52 p.m. UTC 
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  <jakub@redhat.com>

	PR bootstrap/116387
	* resolve.cc (resolve_operator): Use %L rather than %%L in format
	string.



	Jakub
diff mbox series

Patch

--- 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;
 	}