From: Chris Lattner Date: Sun, 29 Aug 2010 17:21:35 +0000 (+0000) Subject: now that loop passes don't use DomFrontier, there is no reason X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=937790195ecb644e3fcbae2d70a879748c14b4b8;p=oota-llvm.git now that loop passes don't use DomFrontier, there is no reason for the unroller to pretend it supports updating it. It still has a horrible hack for DomTree. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@112444 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Transforms/Scalar/LoopUnrollPass.cpp b/lib/Transforms/Scalar/LoopUnrollPass.cpp index 4ed2d456872..d0edfa22005 100644 --- a/lib/Transforms/Scalar/LoopUnrollPass.cpp +++ b/lib/Transforms/Scalar/LoopUnrollPass.cpp @@ -62,13 +62,12 @@ namespace { AU.addPreservedID(LoopSimplifyID); AU.addRequiredID(LCSSAID); AU.addPreservedID(LCSSAID); + AU.addPreserved(); // FIXME: Loop unroll requires LCSSA. And LCSSA requires dom info. // If loop unroll does not preserve dom info then LCSSA pass on next // loop will receive invalid dom info. // For now, recreate dom info, if loop is unrolled. AU.addPreserved(); - AU.addPreserved(); - AU.addPreserved(); } }; } @@ -148,12 +147,7 @@ bool LoopUnroll::runOnLoop(Loop *L, LPPassManager &LPM) { return false; // FIXME: Reconstruct dom info, because it is not preserved properly. - DominatorTree *DT = getAnalysisIfAvailable(); - if (DT) { + if (DominatorTree *DT = getAnalysisIfAvailable()) DT->runOnFunction(*F); - DominanceFrontier *DF = getAnalysisIfAvailable(); - if (DF) - DF->runOnFunction(*F); - } return true; }