Re: [PATCH]: Fix PR target/27863: ICE in check_cfg, at haifa-sched.c:4615

["Vladimir N. Makarov" <vmakarov at redhat dot com>]

Maxim Kuvyrkov wrote:

> Vladimir N. Makarov wrote:
>
> > Richard Guenther wrote:
>
>
> ...
>
> > > I wonder why you tested this on i686 when the PR is for target ia64.
> > > Can you also add the reduced testcase to gcc.c-torture/compile?
> > >
> > >  
> > Maxim, please don't disable bootstrap when submitting a patch to the 
> > main line.  You should check other major platforms too (like x86, 
> > x86_64, and ppc if you have them).
> >
> > I can guess that you put gcc_asserts as in move_insn but 
> > unlink_other_notes is called not for all basic block (some notes has 
> > no bb field).
> >
> > I am afraid you broke not only i64 but other targets too.  So please 
> > fix this as soon as possible.  I'll wait for the patch.
> >
> > Vlad
>
>
> I am sorry for my mistake.  This patch is currently being tested on 
> i386-pc-linux-gnu and ia64-unknown-linux-gnu.
I don't think that this patch will update bb end e.g. when the last insn 
in the bb is NOTE_FUNCTION_END which gives you null bb.