[llvm-commits] [llvm] r123743 - in /llvm/trunk: include/llvm/Analysis/DominanceFrontier.h lib/VMCore/Dominators.cpp
Cameron Zwarich
zwarich at apple.com
Mon Jan 17 21:44:04 PST 2011
Author: zwarich
Date: Mon Jan 17 23:44:04 2011
New Revision: 123743
URL: http://llvm.org/viewvc/llvm-project?rev=123743&view=rev
Log:
There is no point in verifying an analysis that is never updated.
Modified:
llvm/trunk/include/llvm/Analysis/DominanceFrontier.h
llvm/trunk/lib/VMCore/Dominators.cpp
Modified: llvm/trunk/include/llvm/Analysis/DominanceFrontier.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/DominanceFrontier.h?rev=123743&r1=123742&r2=123743&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Analysis/DominanceFrontier.h (original)
+++ llvm/trunk/include/llvm/Analysis/DominanceFrontier.h Mon Jan 17 23:44:04 2011
@@ -175,8 +175,6 @@
return false;
}
- virtual void verifyAnalysis() const;
-
virtual void getAnalysisUsage(AnalysisUsage &AU) const {
AU.setPreservesAll();
AU.addRequired<DominatorTree>();
Modified: llvm/trunk/lib/VMCore/Dominators.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Dominators.cpp?rev=123743&r1=123742&r2=123743&view=diff
==============================================================================
--- llvm/trunk/lib/VMCore/Dominators.cpp (original)
+++ llvm/trunk/lib/VMCore/Dominators.cpp Mon Jan 17 23:44:04 2011
@@ -120,17 +120,6 @@
INITIALIZE_PASS_END(DominanceFrontier, "domfrontier",
"Dominance Frontier Construction", true, true)
-void DominanceFrontier::verifyAnalysis() const {
- if (!VerifyDomInfo) return;
-
- DominatorTree &DT = getAnalysis<DominatorTree>();
-
- DominanceFrontier OtherDF;
- const std::vector<BasicBlock*> &DTRoots = DT.getRoots();
- OtherDF.calculate(DT, DT.getNode(DTRoots[0]));
- assert(!compare(OtherDF) && "Invalid DominanceFrontier info!");
-}
-
namespace {
class DFCalculateWorkObject {
public:
More information about the llvm-commits
mailing list