From: Brian Gaeke <gaeke@uiuc.edu>
Date: Wed, 28 Apr 2004 03:59:48 +0000 (+0000)
Subject: Fix thinkos that Chris caught for me.
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=61d3d5c06b279cce1f2b68af4de82f635910578a;p=oota-llvm.git

Fix thinkos that Chris caught for me.
Make pred mutators private.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@13209 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/include/llvm/CodeGen/MachineBasicBlock.h b/include/llvm/CodeGen/MachineBasicBlock.h
index 01bbc11519d..d6e9ce8ed6b 100644
--- a/include/llvm/CodeGen/MachineBasicBlock.h
+++ b/include/llvm/CodeGen/MachineBasicBlock.h
@@ -119,10 +119,10 @@ public:
   /// The Predecessors list of succ is automatically updated.
   ///
   void addSuccessor (MachineBasicBlock *succ) {
-    Successors.push_back (succ);
     assert (std::find (Successors.begin (), Successors.end (), succ)
             == Successors.end ()
             && "Trying to addSuccessor a MBB which is already my successor");
+    Successors.push_back (succ);
     succ->addPredecessor (this);
   }
 
@@ -138,15 +138,16 @@ public:
     Successors.erase (goner);
   }
 
+private:
   /// addPredecessor - Remove pred as a predecessor of this MachineBasicBlock.
   /// Don't do this unless you know what you're doing, because it doesn't
   /// update pred's successors list. Use pred->addSuccessor instead.
   ///
   void addPredecessor (MachineBasicBlock *pred) {
-    Predecessors.push_back (pred);
     assert(std::find (Predecessors.begin (), Predecessors.end (), pred)
            == Predecessors.end ()
            && "Trying to addPredecessor a MBB which is already my predecessor");
+    Predecessors.push_back (pred);
   }
 
   /// removePredecessor - Remove pred as a predecessor of this
@@ -162,6 +163,7 @@ public:
     Predecessors.erase (goner);
   }
 
+public:
   /// getFirstTerminator - returns an iterator to the first terminator
   /// instruction of this basic block. If a terminator does not exist,
   /// it returns end()