Re: Questionable code in fixup_reorder_chain

2005-09-23 Thread Eric Botcazou
> I actually believe there was reason for creating the loop (ie > redirecting the edge to anything else than the fallthru egdge > destination) as otherwise we screwed up in > force_nonfallthru_and_redirect and this function (called via > force_nonfallthru) is supposed to redirect the jump back to p

Re: Questionable code in fixup_reorder_chain

2005-09-23 Thread Jan Hubicka
> Hi Jan, > > I think fixup_reorder_chain contains questionable code to cope with a > pathological case: > > /* The degenerated case of conditional jump jumping to the next >instruction can happen on target having jumps with side >effects. > >