From ae8abf7896cebfa9770cdc44ccff3c77e7bd5844 Mon Sep 17 00:00:00 2001 From: Chandler Carruth Date: Wed, 16 Apr 2014 11:14:28 +0000 Subject: [PATCH] [LCG] Stop playing fast and loose with reference members and assignment. It doesn't work. I'm still cleaning up all the places where I blindly followed this pattern. There are more to come in this code too. As a benefit, this lets the default copy and move operations Just Work. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@206375 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Analysis/LazyCallGraph.h | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/include/llvm/Analysis/LazyCallGraph.h b/include/llvm/Analysis/LazyCallGraph.h index 74b0c8e9184..d5785e740ae 100644 --- a/include/llvm/Analysis/LazyCallGraph.h +++ b/include/llvm/Analysis/LazyCallGraph.h @@ -119,25 +119,18 @@ public: /// \brief Nonce type to select the constructor for the end iterator. struct IsAtEndT {}; - LazyCallGraph &G; + LazyCallGraph *G; NodeVectorImplT::iterator NI; // Build the begin iterator for a node. explicit iterator(LazyCallGraph &G, NodeVectorImplT &Nodes) - : G(G), NI(Nodes.begin()) {} + : G(&G), NI(Nodes.begin()) {} // Build the end iterator for a node. This is selected purely by overload. iterator(LazyCallGraph &G, NodeVectorImplT &Nodes, IsAtEndT /*Nonce*/) - : G(G), NI(Nodes.end()) {} + : G(&G), NI(Nodes.end()) {} public: - iterator(const iterator &Arg) : G(Arg.G), NI(Arg.NI) {} - iterator(iterator &&Arg) : G(Arg.G), NI(std::move(Arg.NI)) {} - iterator &operator=(iterator Arg) { - std::swap(Arg, *this); - return *this; - } - bool operator==(const iterator &Arg) { return NI == Arg.NI; } bool operator!=(const iterator &Arg) { return !operator==(Arg); } @@ -146,7 +139,7 @@ public: return NI->get(); Function *F = NI->get(); - Node *ChildN = G.get(*F); + Node *ChildN = G->get(*F); *NI = ChildN; return ChildN; } -- 2.34.1