On Wed, 2014-08-20 at 10:20 +0200, Andreas Schwab wrote:
> David Malcolm <dmalc...@redhat.com> writes:
>
> > @@ -4083,7 +4083,7 @@ cfg_layout_can_duplicate_bb_p (const_basic_block bb)
> > return true;
> > }
> >
> > -rtx
> > +rtx_insn *
> > duplicate_insn_chain (rtx from, rtx to)
> > {
> > rtx insn, next, copy;
> > @@ -4169,7 +4169,7 @@ duplicate_insn_chain (rtx from, rtx to)
> > }
> > insn = NEXT_INSN (last);
> > delete_insn (last);
> > - return insn;
> > + return as_a <rtx_insn *> (insn);
>
> This is wrong, insn may be NULL.
Thanks; and indeed, this broke the bootstrap, as noted in:
https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01964.html
Fixed in r214207:
https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01971.html
Sorry. I'm trying to figure out why this didn't break during my
testing. Patch #60 of the original series contained an equivalent as_a
-> as_a_nullable hunk:
https://gcc.gnu.org/ml/gcc-patches/2014-08/msg00708.html
- albeit with a misleading ChangeLog entry (I'm assuming I messed that
up during a rebase) - but this hadn't been applied yet.
Dave
