diff options
| author | Bill Wendling <isanbard@gmail.com> | 2011-08-16 20:41:17 +0000 |
|---|---|---|
| committer | Bill Wendling <isanbard@gmail.com> | 2011-08-16 20:41:17 +0000 |
| commit | 23b49ba6d55972fdffdfd4ed37edcf067bb60f3c (patch) | |
| tree | a06d52566e3d17e82ba02c391ea17f7e4630b1d5 /lib/Transforms/Utils | |
| parent | 578071a08734a4eb76484ba0a8d9f10b6e322184 (diff) | |
| download | llvm-23b49ba6d55972fdffdfd4ed37edcf067bb60f3c.tar.gz llvm-23b49ba6d55972fdffdfd4ed37edcf067bb60f3c.tar.bz2 llvm-23b49ba6d55972fdffdfd4ed37edcf067bb60f3c.tar.xz | |
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
Diffstat (limited to 'lib/Transforms/Utils')
| -rw-r--r-- | lib/Transforms/Utils/SimplifyCFG.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/Transforms/Utils/SimplifyCFG.cpp b/lib/Transforms/Utils/SimplifyCFG.cpp index 9f2c311601..2d60c35afb 100644 --- a/lib/Transforms/Utils/SimplifyCFG.cpp +++ b/lib/Transforms/Utils/SimplifyCFG.cpp @@ -2266,7 +2266,10 @@ bool SimplifyCFGOpt::SimplifyUnreachable(UnreachableInst *UI) { !isa<LandingPadInst>(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) |