diff --git a/llvm/lib/Transforms/IPO/Inliner.cpp b/llvm/lib/Transforms/IPO/Inliner.cpp --- a/llvm/lib/Transforms/IPO/Inliner.cpp +++ b/llvm/lib/Transforms/IPO/Inliner.cpp @@ -724,21 +724,31 @@ size_t FirstIndex = 0; }; +class MyPriority { +public: + MyPriority(int size) : size(size) {} + + static bool isMoreDesirable(const MyPriority &S1, const MyPriority &S2) { + return S1.size < S2.size; + } + + static MyPriority evaluate(CallBase *CB) { + Function *Callee = CB->getCalledFunction(); + return MyPriority(Callee->getInstructionCount()); + } + + int size; +}; + +template class PriorityInlineOrder : public InlineOrder> { using T = std::pair; + using HeapT = std::pair; using reference = T &; using const_reference = const T &; - // Return true if S1 is more desirable than S2. - static bool isMoreDesirable(int S1, int S2) { return S1 < S2; } - - static bool cmp(const T &P1, const T &P2) { - return isMoreDesirable(P2.second, P1.second); - } - - int evaluate(CallBase *CB) { - Function *Callee = CB->getCalledFunction(); - return (int)Callee->getInstructionCount(); + static bool cmp(const HeapT &P1, const HeapT &P2) { + return PriorityT::isMoreDesirable(P2.second, P1.second); } // A call site could become less desirable for inlining because of the size @@ -752,9 +762,9 @@ bool Changed = false; do { CallBase *CB = Heap.front().first; - const int PreviousGoodness = Heap.front().second; - const int CurrentGoodness = evaluate(CB); - Changed = isMoreDesirable(PreviousGoodness, CurrentGoodness); + const PriorityT PreviousGoodness = Heap.front().second; + const PriorityT CurrentGoodness = PriorityT::evaluate(CB); + Changed = PriorityT::isMoreDesirable(PreviousGoodness, CurrentGoodness); if (Changed) { std::pop_heap(Heap.begin(), Heap.end(), cmp); Heap.pop_back(); @@ -770,7 +780,7 @@ void push(const T &Elt) override { CallBase *CB = Elt.first; const int InlineHistoryID = Elt.second; - const int Goodness = evaluate(CB); + const PriorityT Goodness = PriorityT::evaluate(CB); Heap.push_back({CB, Goodness}); std::push_heap(Heap.begin(), Heap.end(), cmp); @@ -798,12 +808,16 @@ } void erase_if(function_ref Pred) override { - Heap.erase(std::remove_if(Heap.begin(), Heap.end(), Pred), Heap.end()); + auto PredWrapper = [=](HeapT P) -> bool { + return Pred(std::make_pair(P.first, 0)); + }; + Heap.erase(std::remove_if(Heap.begin(), Heap.end(), PredWrapper), + Heap.end()); std::make_heap(Heap.begin(), Heap.end(), cmp); } private: - SmallVector Heap; + SmallVector Heap; DenseMap InlineHistoryMap; }; @@ -853,9 +867,9 @@ // and eventually they all become too large to inline, rather than // incrementally maknig a single function grow in a super linear fashion. std::unique_ptr>> Calls; - if (InlineEnablePriorityOrder) - Calls = std::make_unique(); - else + if (InlineEnablePriorityOrder) { + Calls = std::make_unique>(); + } else Calls = std::make_unique>>(); assert(Calls != nullptr && "Expected an initialized InlineOrder");