summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Transforms
diff options
context:
space:
mode:
authorEli Friedman <eli.friedman@gmail.com>2011-08-16 00:41:37 +0000
committerEli Friedman <eli.friedman@gmail.com>2011-08-16 00:41:37 +0000
commitbd397034560613ce478b306c54879022722f7254 (patch)
tree6bdd55b81df4e0ffba376928e52430aa11290bd0 /llvm/lib/Transforms
parentb8f30de527cc64ae5cfceae4e3f06206440c10ca (diff)
downloadbcm5719-llvm-bd397034560613ce478b306c54879022722f7254.tar.gz
bcm5719-llvm-bd397034560613ce478b306c54879022722f7254.zip
After talking with Bill, it seems like the LandingPad handling here is likely
to be wrong (or at least somewhat suspect). Leave a FIXME for Bill. llvm-svn: 137694
Diffstat (limited to 'llvm/lib/Transforms')
-rw-r--r--llvm/lib/Transforms/Utils/SimplifyCFG.cpp5
1 files changed, 1 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
index 2d60c35afb1..9f2c311601e 100644
--- a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
+++ b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -2266,10 +2266,7 @@ bool SimplifyCFGOpt::SimplifyUnreachable(UnreachableInst *UI) {
!isa<LandingPadInst>(BBI)) {
break;
}
- // 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.
+ // FIXME: Handling of LandingPadInst (landingpad) is suspicious.
}
// Delete this instruction (any uses are guaranteed to be dead)
OpenPOWER on IntegriCloud