Remove explicit RNSuccIterator copy assignment in favor of implicit default
authorDavid Blaikie <dblaikie@gmail.com>
Wed, 4 Mar 2015 07:51:50 +0000 (07:51 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Wed, 4 Mar 2015 07:51:50 +0000 (07:51 +0000)
Asserting that the source and destination iterators are from the same
region is unnecessary - there's no reason to disallow reassignment from
any regions, so long as they aren't compared.

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

include/llvm/Analysis/RegionIterator.h

index 0daff58475dd8baa2afeb0560df9486c635593eb..ced58dfabdd56995561f1d8f9b9e0b17f66e68ed 100644 (file)
@@ -145,16 +145,6 @@ public:
     ++*this;
     return tmp;
   }
-
-  inline const Self &operator=(const Self &I) {
-    if (this != &I) {
-      assert(getNode()->getParent() == I.getNode()->getParent()
-             && "Cannot assign iterators of two different regions!");
-      Node = I.Node;
-      BItor = I.BItor;
-    }
-    return *this;
-  }
 };
 
 
@@ -240,16 +230,6 @@ public:
     ++*this;
     return tmp;
   }
-
-  inline const Self &operator=(const Self &I) {
-    if (this != &I) {
-      assert(Node->getParent() == I.Node->getParent()
-             && "Cannot assign iterators to two different regions!");
-      Node = I.Node;
-      Itor = I.Itor;
-    }
-    return *this;
-  }
 };
 
 template<class NodeType, class BlockT, class RegionT>