From d8c120bbd3ed19439c02ce629f312b163302b5dd Mon Sep 17 00:00:00 2001 From: Cameron Zwarich Date: Tue, 11 Jan 2011 22:14:42 +0000 Subject: [PATCH] Dial back the speculative fix for PR8954 a bit, so that we only recompute dominators once at the beginning of GVN instead of once per iteration. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@123278 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Scalar/GVN.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp index 279211d4080..014341b867d 100644 --- a/lib/Transforms/Scalar/GVN.cpp +++ b/lib/Transforms/Scalar/GVN.cpp @@ -1754,6 +1754,9 @@ bool GVN::runOnFunction(Function& F) { unsigned Iteration = 0; + // FIXME: Remove this when PR8954 is fixed. + DT->DT->recalculate(F); + while (ShouldContinue) { DEBUG(dbgs() << "GVN iteration: " << Iteration << "\n"); ShouldContinue = iterateOnFunction(F); @@ -2011,7 +2014,6 @@ bool GVN::iterateOnFunction(Function &F) { RE = RPOT.end(); RI != RE; ++RI) Changed |= processBlock(*RI); #else - DT->DT->recalculate(F); for (df_iterator DI = df_begin(DT->getRootNode()), DE = df_end(DT->getRootNode()); DI != DE; ++DI) Changed |= processBlock(DI->getBlock());