Index: llvm/trunk/lib/Transforms/Scalar/GVNHoist.cpp =================================================================== --- llvm/trunk/lib/Transforms/Scalar/GVNHoist.cpp +++ llvm/trunk/lib/Transforms/Scalar/GVNHoist.cpp @@ -155,7 +155,6 @@ using CHIIt = SmallVectorImpl::iterator; using CHIArgs = iterator_range; -using CHICache = DenseMap>; using OutValuesType = DenseMap>; using InValuesType = DenseMap, 2>>; @@ -767,7 +766,6 @@ ReverseIDFCalculator IDFs(*PDT); OutValuesType OutValue; InValuesType InValue; - CHICache CachedCHIs; for (const auto &R : Ranks) { const SmallVecInsn &V = Map.lookup(R); if (V.size() < 2) @@ -786,6 +784,7 @@ // which currently have dead terminators that are control // dependence sources of a block which is in NewLiveBlocks. IDFs.setDefiningBlocks(VNBlocks); + IDFBlocks.clear(); IDFs.calculate(IDFBlocks); // Make a map of BB vs instructions to be hoisted. @@ -798,8 +797,7 @@ for (unsigned i = 0; i < V.size(); ++i) { CHIArg C = {VN, nullptr, nullptr}; // Ignore spurious PDFs. - if (DT->properlyDominates(IDFB, V[i]->getParent()) && - CachedCHIs[IDFB].insert(V[i]).second) { + if (DT->properlyDominates(IDFB, V[i]->getParent())) { OutValue[IDFB].push_back(C); LLVM_DEBUG(dbgs() << "\nInsertion a CHI for BB: " << IDFB->getName() << ", for Insn: " << *V[i]);