diff options
author | Eli Friedman <eli.friedman@gmail.com> | 2011-08-16 00:41:37 +0000 |
---|---|---|
committer | Eli Friedman <eli.friedman@gmail.com> | 2011-08-16 00:41:37 +0000 |
commit | bd397034560613ce478b306c54879022722f7254 (patch) | |
tree | 6bdd55b81df4e0ffba376928e52430aa11290bd0 /llvm/lib/Transforms | |
parent | b8f30de527cc64ae5cfceae4e3f06206440c10ca (diff) | |
download | bcm5719-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.cpp | 5 |
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) |