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

[Dale Johannesen]

On Jun 1, 2007, at 12:56 PM, Duncan Sands wrote:

> Hi Dale,
>
>> Arrange for only 1 of multiple branches to landing pad to be kept.

That should be "successors", not "branches", but you've gotten past  
that....

> it struck me that this patch seems to assume that if a successor
> is a landing pad then it is necessarily a landing pad for the
> current MBB.

I wasn't intending to assume that; how so?

> But couldn't it be the landing pad for some other
> MBB, and simply a normal successor for the current one?

It could; there is no way to tell from the CFG information.