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
This commit is contained in:
Cameron Zwarich 2011-01-11 22:14:42 +00:00
parent b08773749a
commit d8c120bbd3

View File

@ -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<DomTreeNode*> DI = df_begin(DT->getRootNode()),
DE = df_end(DT->getRootNode()); DI != DE; ++DI)
Changed |= processBlock(DI->getBlock());