From: Stepan Dyatkovskiy Date: Wed, 10 Sep 2014 10:08:25 +0000 (+0000) Subject: MergeFunctions: FunctionPtr has been renamed to FunctionNode. X-Git-Url: http://plrg.eecs.uci.edu/git/?p=oota-llvm.git;a=commitdiff_plain;h=e944e11ededd55bc866fd85d4712b1c069fd15ee MergeFunctions: FunctionPtr has been renamed to FunctionNode. It's supposed to store additional pass information for current function here. That was the reason for name change. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@217483 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/IPO/MergeFunctions.cpp b/lib/Transforms/IPO/MergeFunctions.cpp index 7580830576b..05223eb07b8 100644 --- a/lib/Transforms/IPO/MergeFunctions.cpp +++ b/lib/Transforms/IPO/MergeFunctions.cpp @@ -392,15 +392,15 @@ private: DenseMap sn_mapL, sn_mapR; }; -class FunctionPtr { +class FunctionNode { AssertingVH F; const DataLayout *DL; public: - FunctionPtr(Function *F, const DataLayout *DL) : F(F), DL(DL) {} + FunctionNode(Function *F, const DataLayout *DL) : F(F), DL(DL) {} Function *getFunc() const { return F; } void release() { F = 0; } - bool operator<(const FunctionPtr &RHS) const { + bool operator<(const FunctionNode &RHS) const { return (FunctionComparator(DL, F, RHS.getFunc()).compare()) == -1; } }; @@ -1077,7 +1077,7 @@ public: bool runOnModule(Module &M) override; private: - typedef std::set FnTreeType; + typedef std::set FnTreeType; /// A work queue of functions that may have been modified and should be /// analyzed again. @@ -1420,14 +1420,14 @@ void MergeFunctions::mergeTwoFunctions(Function *F, Function *G) { // that was already inserted. bool MergeFunctions::insert(Function *NewFunction) { std::pair Result = - FnTree.insert(FunctionPtr(NewFunction, DL)); + FnTree.insert(FunctionNode(NewFunction, DL)); if (Result.second) { DEBUG(dbgs() << "Inserting as unique: " << NewFunction->getName() << '\n'); return false; } - const FunctionPtr &OldF = *Result.first; + const FunctionNode &OldF = *Result.first; // Don't merge tiny functions, since it can just end up making the function // larger. @@ -1457,7 +1457,7 @@ bool MergeFunctions::insert(Function *NewFunction) { void MergeFunctions::remove(Function *F) { // We need to make sure we remove F, not a function "equal" to F per the // function equality comparator. - FnTreeType::iterator found = FnTree.find(FunctionPtr(F, DL)); + FnTreeType::iterator found = FnTree.find(FunctionNode(F, DL)); size_t Erased = 0; if (found != FnTree.end() && found->getFunc() == F) { Erased = 1;