On Jun 1, 2007, at 1:24 PM, Duncan Sands wrote:
> Suppose a successor S1 has isLandingPad true because it is
> a landing pad for some other MBB, and some later successor (S2)
> has isLandingPad true because it is the landing pad for this
> MBB. What will happen? When we get to S1 foundPad is set to
> true and S1 is not deleted. Later we get to S2 and, because
> foundPad is true, fall through to
> } else {
> // Otherwise, this is a superfluous edge, remove it.
> MBB.removeSuccessor(SI);
> MadeChange = true;
> }
> and wrongly delete S2.
Good catch, thanks. I'll fix that.
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
