On 31 January 2012 16:10, Jakub Jelinek <ja...@redhat.com> wrote: > Hi! > > SUBST_LINK is only used in code guarded with #ifdef HAVE_cc0, so this patch > guards that function and macro with the same to avoid warnings. > Committed to trunk as obvious.
Thanks for fixing this. I was out for a while this morning and was getting around to fix this a few minutes back but you beat me to it. Sorry about the breakage in the first place. Ramana > > 2012-01-31 Jakub Jelinek <ja...@redhat.com> > > PR bootstrap/52058 > * combine.c (do_SUBST_LINK, SUBST_LINK): Guard with #ifndef HAVE_cc0. > > --- gcc/combine.c.jj 2012-01-26 09:22:21.000000000 +0100 > +++ gcc/combine.c 2012-01-31 17:06:31.505978890 +0100 > @@ -790,6 +790,7 @@ do_SUBST_MODE (rtx *into, enum machine_m > > #define SUBST_MODE(INTO, NEWVAL) do_SUBST_MODE(&(INTO), (NEWVAL)) > > +#ifndef HAVE_cc0 > /* Similar to SUBST, but NEWVAL is a LOG_LINKS expression. */ > > static void > @@ -815,7 +816,7 @@ do_SUBST_LINK (struct insn_link **into, > } > > #define SUBST_LINK(oldval, newval) do_SUBST_LINK (&oldval, newval) > - > +#endif > > /* Subroutine of try_combine. Determine whether the replacement patterns > NEWPAT, NEWI2PAT and NEWOTHERPAT are cheaper according to insn_rtx_cost > > Jakub