This is an archive of the discontinued LLVM Phabricator instance.

[LoopDistribute] Preserve GlobalsAA also in the new PM
ClosedPublic

Authored by davide on Nov 8 2016, 11:51 AM.

Reviewers

eli.friedman
hfinkel

Commits

Summary

Eli, in r281758 you changed LoopDistribute to preserve GlobalsAA, but only when the old PM is running.
This patch does the same for the new PM.

davide retitled this revision from to [LoopDistribute] Preserve GlobalsAA also in the new PM.

davide updated this object.

davide added a reviewer: eli.friedman.

davide added a subscriber: llvm-commits.

LGTM

This revision is now accepted and ready to land.Nov 8 2016, 11:54 AM

Thanks for spotting that; I haven't really been keeping track of the new PM work.

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

lib/

Transforms/

Scalar/

1 line

Show First 20 Lines • Show All 971 Lines • ▼ Show 20 Lines	PreservedAnalyses LoopDistributePass::run(Function &F,
};		};

bool Changed = runImpl(F, &LI, &DT, &SE, &ORE, GetLAA, ProcessAllLoops);		bool Changed = runImpl(F, &LI, &DT, &SE, &ORE, GetLAA, ProcessAllLoops);
if (!Changed)		if (!Changed)
return PreservedAnalyses::all();		return PreservedAnalyses::all();
PreservedAnalyses PA;		PreservedAnalyses PA;
PA.preserve<LoopAnalysis>();		PA.preserve<LoopAnalysis>();
PA.preserve<DominatorTreeAnalysis>();		PA.preserve<DominatorTreeAnalysis>();
		PA.preserve<GlobalsAA>();
return PA;		return PA;
}		}

char LoopDistributeLegacy::ID;		char LoopDistributeLegacy::ID;
static const char ldist_name[] = "Loop Distribution";		static const char ldist_name[] = "Loop Distribution";

INITIALIZE_PASS_BEGIN(LoopDistributeLegacy, LDIST_NAME, ldist_name, false,		INITIALIZE_PASS_BEGIN(LoopDistributeLegacy, LDIST_NAME, ldist_name, false,
false)		false)
Show All 12 Lines