This is an archive of the discontinued LLVM Phabricator instance.

Add missed PostDominatorTree analysis dependency to GVN hoist pass.
ClosedPublic

Authored by ashlykov on Feb 7 2018, 2:14 AM.

Reviewers

Commits

Summary

GVN hoist pass is using PostDominatorTree analysis, therefore the analysis
should be listed in the pass initialization as a dependency.

LGTM.

This revision is now accepted and ready to land.Feb 7 2018, 5:29 PM

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 1,194 Lines • ▼ Show 20 Lines

	char GVNHoistLegacyPass::ID = 0;			char GVNHoistLegacyPass::ID = 0;

	INITIALIZE_PASS_BEGIN(GVNHoistLegacyPass, "gvn-hoist",			INITIALIZE_PASS_BEGIN(GVNHoistLegacyPass, "gvn-hoist",
	"Early GVN Hoisting of Expressions", false, false)			"Early GVN Hoisting of Expressions", false, false)
	INITIALIZE_PASS_DEPENDENCY(MemoryDependenceWrapperPass)			INITIALIZE_PASS_DEPENDENCY(MemoryDependenceWrapperPass)
	INITIALIZE_PASS_DEPENDENCY(MemorySSAWrapperPass)			INITIALIZE_PASS_DEPENDENCY(MemorySSAWrapperPass)
	INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)			INITIALIZE_PASS_DEPENDENCY(DominatorTreeWrapperPass)
				INITIALIZE_PASS_DEPENDENCY(PostDominatorTreeWrapperPass)
	INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass)			INITIALIZE_PASS_DEPENDENCY(AAResultsWrapperPass)
	INITIALIZE_PASS_END(GVNHoistLegacyPass, "gvn-hoist",			INITIALIZE_PASS_END(GVNHoistLegacyPass, "gvn-hoist",
	"Early GVN Hoisting of Expressions", false, false)			"Early GVN Hoisting of Expressions", false, false)

	FunctionPass *llvm::createGVNHoistPass() { return new GVNHoistLegacyPass(); }			FunctionPass *llvm::createGVNHoistPass() { return new GVNHoistLegacyPass(); }