Abort in redirect_edge_and_branch on vax at line 866

Jan Hubicka jh@suse.cz
Thu Jan 10 01:47:00 GMT 2002


> > The attached patch should do the trick for you.
> > Bootstrapped/regtested i386. OK to install?
> > 
> > Wed Jan  9 11:23:03 CET 2002  Jan Hubicka  <jh@suse.cz>
> > 	* gcse.c (cprop_jump): Delete insn if simplified jump is no-op.
> Be VERY VERY careful here.  Deleting the insn is likely to cause all
I tried to be - to my understanding all information gcse holds at that
point is the global information for each basic block. As I am not
making the global properties worse, I am just improving them, the datastructures
remain to be conservative approximation of the real thing making all other
changes valid.

I tought the point of not deleting the instruction has been to keep CFG
consistent, really, but possibly I do miss something important but even
second pass didn't show something wrong to me.

Honza
> kinds of problems with the datastructures that gcse.c uses.  That was
> the entire point behind deferring deletion of jumps until after gcse.c
> was finished.
> 
> I would like to see an analysis of the code before this patch is installed.
> jeff



More information about the Gcc-bugs mailing list