Some minor cleanups. No functionalitical change.
authorBill Wendling <isanbard@gmail.com>
Thu, 28 Jul 2011 07:44:07 +0000 (07:44 +0000)
committerBill Wendling <isanbard@gmail.com>
Thu, 28 Jul 2011 07:44:07 +0000 (07:44 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@136341 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Utils/InlineFunction.cpp

index 5cba7ee823867e33a65928934271beb6bcefbce3..484899f41783998bf7d78dfa5c8e616bd1a89ee8 100644 (file)
@@ -251,11 +251,10 @@ namespace {
     SmallVector<Value*, 8> UnwindDestPHIValues;
 
     // New EH:
-    BasicBlock *OuterResumeDest;
-    BasicBlock *InnerResumeDest;
-    LandingPadInst *CallerLPad;
-    PHINode *InnerEHValuesPHI;
-    BasicBlock *SplitLPad;
+    BasicBlock *OuterResumeDest; //< Destination of the invoke's unwind.
+    BasicBlock *InnerResumeDest; //< Destination for the callee's resume.
+    LandingPadInst *CallerLPad;  //< LandingPadInst associated with the invoke.
+    PHINode *InnerEHValuesPHI;   //< PHI for EH values from landingpad insts.
 
   public:
     InvokeInliningInfo(InvokeInst *II)
@@ -263,10 +262,10 @@ namespace {
         InnerUnwindDest(0), InnerExceptionPHI(0), InnerSelectorPHI(0),
 
         OuterResumeDest(II->getUnwindDest()), InnerResumeDest(0),
-        CallerLPad(0), InnerEHValuesPHI(0), SplitLPad(0) {
-      // If there are PHI nodes in the unwind destination block, we
-      // need to keep track of which values came into them from the
-      // invoke before removing the edge from this block.
+        CallerLPad(0), InnerEHValuesPHI(0) {
+      // If there are PHI nodes in the unwind destination block, we need to keep
+      // track of which values came into them from the invoke before removing
+      // the edge from this block.
       llvm::BasicBlock *InvokeBB = II->getParent();
       BasicBlock::iterator I = OuterUnwindDest->begin();
       for (; isa<PHINode>(I); ++I) {
@@ -276,8 +275,9 @@ namespace {
       }
 
       // FIXME: With the new EH, this if/dyn_cast should be a 'cast'.
-      if (LandingPadInst *LPI = dyn_cast<LandingPadInst>(I))
+      if (LandingPadInst *LPI = dyn_cast<LandingPadInst>(I)) {
         CallerLPad = LPI;
+      }
     }
 
     /// The outer unwind destination is the target of unwind edges
@@ -296,13 +296,6 @@ namespace {
     BasicBlock *getInnerUnwindDest_new();
 
     LandingPadInst *getLandingPadInst() const { return CallerLPad; }
-    BasicBlock *getSplitLandingPad() {
-      if (SplitLPad) return SplitLPad;
-      assert(CallerLPad && "Trying to split a block that isn't a landing pad!");
-      BasicBlock::iterator I = CallerLPad; ++I;
-      SplitLPad = CallerLPad->getParent()->splitBasicBlock(I, "split.lpad");
-      return SplitLPad;
-    }
 
     bool forwardEHResume(CallInst *call, BasicBlock *src);