From 23b49ba6d55972fdffdfd4ed37edcf067bb60f3c Mon Sep 17 00:00:00 2001 From: Bill Wendling Date: Tue, 16 Aug 2011 20:41:17 +0000 Subject: [PATCH] I think there was some confusion about what I meant. :-) Replacing the comment. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@137743 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Utils/SimplifyCFG.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/lib/Transforms/Utils/SimplifyCFG.cpp b/lib/Transforms/Utils/SimplifyCFG.cpp index 9f2c311601e..2d60c35afb1 100644 --- a/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/lib/Transforms/Utils/SimplifyCFG.cpp @@ -2266,7 +2266,10 @@ bool SimplifyCFGOpt::SimplifyUnreachable(UnreachableInst *UI) { !isa(BBI)) { break; } - // FIXME: Handling of LandingPadInst (landingpad) is suspicious. + // Note that deleting LandingPad's here is in fact okay, although it + // involves a bit of subtle reasoning. If this inst is a LandingPad, + // all the predecessors of this block will be the unwind edges of Invokes, + // and we can therefore guarantee this block will be erased. } // Delete this instruction (any uses are guaranteed to be dead) -- 2.11.0