[llvm-commits] CVS: llvm/lib/CodeGen/BranchFolding.cpp

[Dale Johannesen]

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.