On Jun 1, 2007, at 1:28 PM, Dale Johannesen wrote:
>
> 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.
Should be fixed now.
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
